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

[Unit Test] Use matrix build workflow & Add Integration Test #7078

Merged
merged 10 commits into from
Jan 21, 2021

Conversation

AlbumenJ
Copy link
Member

@AlbumenJ AlbumenJ commented Jan 5, 2021

What is the purpose of the change

  • Use matrix build workflow
  • remove 3rd party step dependency due to apache limit
  • add integration test base on dubbo-samples

@codecov-io
Copy link

codecov-io commented Jan 5, 2021

Codecov Report

Merging #7078 (8e509bf) into master (70b6fb3) will increase coverage by 1.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #7078      +/-   ##
============================================
+ Coverage     59.22%   60.25%   +1.03%     
+ Complexity      508      507       -1     
============================================
  Files          1028     1028              
  Lines         41525    41531       +6     
  Branches       6038     6038              
============================================
+ Hits          24593    25025     +432     
+ Misses        14169    13754     -415     
+ Partials       2763     2752      -11     
Impacted Files Coverage Δ Complexity Δ
.../apache/dubbo/remoting/transport/AbstractPeer.java 58.69% <0.00%> (-4.35%) 0.00% <0.00%> (ø%)
...pache/dubbo/remoting/transport/AbstractServer.java 53.75% <0.00%> (-3.75%) 0.00% <0.00%> (ø%)
...e/dubbo/remoting/transport/netty/NettyChannel.java 52.27% <0.00%> (-3.41%) 19.00% <0.00%> (-1.00%)
.../src/main/java/org/apache/dubbo/rpc/RpcStatus.java 71.42% <0.00%> (-1.20%) 0.00% <0.00%> (ø%)
...pache/dubbo/registry/support/AbstractRegistry.java 79.62% <0.00%> (-1.12%) 0.00% <0.00%> (ø%)
...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java 88.57% <0.00%> (-0.96%) 16.00% <0.00%> (-1.00%)
...figcenter/file/FileSystemDynamicConfiguration.java 62.38% <0.00%> (-0.48%) 0.00% <0.00%> (ø%)
...ava/org/apache/dubbo/common/utils/MemberUtils.java 0.00% <0.00%> (ø) 0.00% <0.00%> (ø%)
...apache/dubbo/common/extension/ExtensionLoader.java 75.56% <0.00%> (ø) 0.00% <0.00%> (ø%)
...mon/src/main/java/org/apache/dubbo/common/URL.java 48.95% <0.00%> (+0.10%) 0.00% <0.00%> (ø%)
... and 47 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 70b6fb3...8e509bf. Read the comment docs.

@AlbumenJ AlbumenJ marked this pull request as ready for review January 5, 2021 05:41
#	.github/workflows/unit-test.yml
@AlbumenJ AlbumenJ changed the title [Unit Test] Use matrix build workflow, remove 3rd party dependency [Unit Test] Use matrix build workflow & Add Integration Test Jan 16, 2021
Copy link
Member

@kylixs kylixs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job!

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