Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update test #11

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

update test #11

wants to merge 22 commits into from

Conversation

xwm1992
Copy link
Owner

@xwm1992 xwm1992 commented Nov 3, 2023

Fixes ISSUE#.

Motivation

Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.

Modifications

Describe the modifications you've done.

Documentation

  • Does this pull request introduce a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
  • If a feature is not applicable for documentation, explain why?
  • If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Pil0tXia and others added 8 commits October 31, 2023 20:43
* rename common package to constant

* rename Errors enum to Status and Message sub-class to StatusMessage

* Exact 'Types' to first level of classification 'Category'

* remove 'error' in Status javadoc
* c-sdk dirs

* add libs: curl, json-c

* add libs: curl, json-c

* fix .gitmodules
* c-sdk dirs

* c-sdk support tcp protocol

* license

* support for tcp
* c-sdk dirs

* c-sdk example and config

* add config and example

* add license
* c-sdk dirs

* c-sdk makefile

* add makefile

* makefile

* add license
* [ISSUE apache#4519]Rust sdk support CloudEvents

* optimize code

* fix complie error
@codecov-commenter
Copy link

codecov-commenter commented Nov 3, 2023

Codecov Report

Merging #11 (53715a6) into master (55865a6) will increase coverage by 0.20%.
Report is 2 commits behind head on master.
The diff coverage is 5.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff              @@
##             master      #11      +/-   ##
============================================
+ Coverage     16.09%   16.30%   +0.20%     
- Complexity     1550     1592      +42     
============================================
  Files           730      747      +17     
  Lines         28640    28965     +325     
  Branches       2523     2541      +18     
============================================
+ Hits           4611     4723     +112     
- Misses        23581    23793     +212     
- Partials        448      449       +1     
Files Coverage Δ
...va/org/apache/eventmesh/common/utils/NetUtils.java 80.00% <ø> (ø)
...ache/eventmesh/runtime/boot/AbstractTCPServer.java 0.46% <ø> (ø)
...re/protocol/http/processor/SubscribeProcessor.java 0.00% <ø> (ø)
.../core/protocol/http/push/AsyncHTTPPushRequest.java 0.00% <ø> (ø)
...g/apache/eventmesh/runtime/util/EventMeshUtil.java 76.86% <100.00%> (ø)
...entmesh/client/grpc/consumer/SubStreamHandler.java 48.80% <ø> (ø)
...sh/client/http/consumer/EventMeshHttpConsumer.java 0.00% <ø> (ø)
...storage/standalone/broker/model/TopicMetadata.java 58.33% <ø> (ø)
...protocol/http/consumer/HttpClientGroupMapping.java 0.00% <0.00%> (ø)
.../protocol/http/processor/CreateTopicProcessor.java 0.00% <0.00%> (ø)
... and 9 more

... and 18 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

yanrongzhen and others added 5 commits November 4, 2023 18:33
* fix issue 3825

* return

* fix issue 3902

* fix codestyle
* Add unit test for RedisSinkConnector.

* Refactor test case.

* Do some refactor.

* shut down redis server.

* Simplify test cases.
* Add sendcallback for springConnector.

* Do some optimization.

* Remove external dependencies.

* Canonical setter name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants