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

Codecov #80 #87

Merged
merged 18 commits into from
Apr 26, 2022
Merged

Codecov #80 #87

merged 18 commits into from
Apr 26, 2022

Conversation

pingww
Copy link
Contributor

@pingww pingww commented Apr 24, 2022

add some unit tests

@codecov-commenter
Copy link

Codecov Report

Merging #87 (294882c) into main (34f499c) will increase coverage by 2.58%.
The diff coverage is 51.85%.

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   30.07%   32.66%   +2.58%     
==========================================
  Files          97       98       +1     
  Lines        4186     4237      +51     
  Branches      660      671      +11     
==========================================
+ Hits         1259     1384     +125     
+ Misses       2682     2593      -89     
- Partials      245      260      +15     
Impacted Files Coverage Δ
.../apache/rocketmq/mqtt/common/util/SpringUtils.java 0.00% <0.00%> (ø)
...a/org/apache/rocketmq/mqtt/cs/starter/Startup.java 0.00% <0.00%> (ø)
...rocketmq/mqtt/cs/session/loop/SessionLoopImpl.java 44.85% <50.00%> (+0.37%) ⬆️
...ocketmq/mqtt/cs/channel/DefaultChannelManager.java 73.77% <66.66%> (+11.27%) ⬆️
...tt/ds/upstream/processor/UnSubscribeProcessor.java 76.47% <66.66%> (+76.47%) ⬆️
...mqtt/ds/upstream/processor/SubscribeProcessor.java 84.21% <70.00%> (+84.21%) ⬆️
...a/org/apache/rocketmq/mqtt/cs/session/Session.java 39.84% <0.00%> (+2.68%) ⬆️
...g/apache/rocketmq/mqtt/cs/channel/ChannelInfo.java 89.89% <0.00%> (+3.03%) ⬆️
...pache/rocketmq/mqtt/ds/meta/FirstTopicManager.java 46.57% <0.00%> (+8.21%) ⬆️
... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e0b5e20...294882c. Read the comment docs.

@tianliuliu tianliuliu merged commit 4f3f127 into apache:main Apr 26, 2022
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.

3 participants