Skip to content

[type:test] Improve the unit test coverage.#3831

Merged
yu199195 merged 35 commits intoapache:masterfrom
yunlongn:Improve-test-v2
Aug 29, 2022
Merged

[type:test] Improve the unit test coverage.#3831
yu199195 merged 35 commits intoapache:masterfrom
yunlongn:Improve-test-v2

Conversation

@yunlongn
Copy link
Member

Improve the unit test coverage of shenyu-web, shenyu-loadbalancer.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@codecov-commenter
Copy link

codecov-commenter commented Aug 11, 2022

Codecov Report

Merging #3831 (a6e7ab4) into master (6a08b6c) will increase coverage by 1.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3831      +/-   ##
============================================
+ Coverage     64.74%   65.89%   +1.14%     
- Complexity     6095     6228     +133     
============================================
  Files           898      898              
  Lines         24708    24706       -2     
  Branches       2240     2240              
============================================
+ Hits          15998    16280     +282     
+ Misses         7304     7040     -264     
+ Partials       1406     1386      -20     
Impacted Files Coverage Δ
...enyu/loadbalancer/factory/LoadBalancerFactory.java 100.00% <ø> (+33.33%) ⬆️
...che/shenyu/web/loader/ShenyuPluginPathBuilder.java 100.00% <ø> (+10.00%) ⬆️
...henyu/plugin/grpc/resolver/ShenyuNameResolver.java 62.76% <0.00%> (-1.07%) ⬇️
...enyu/client/tars/TarsServiceBeanEventListener.java 92.59% <0.00%> (+1.23%) ⬆️
...yu/web/forward/ForwardedRemoteAddressResolver.java 100.00% <0.00%> (+4.00%) ⬆️
...henyu/loadbalancer/spi/RoundRobinLoadBalancer.java 92.45% <0.00%> (+11.32%) ⬆️
...a/org/apache/shenyu/web/filter/FallbackFilter.java 100.00% <0.00%> (+14.28%) ⬆️
...java/org/apache/shenyu/web/filter/CrossFilter.java 98.41% <0.00%> (+14.28%) ⬆️
...e/shenyu/loadbalancer/cache/UpstreamCheckTask.java 93.60% <0.00%> (+15.19%) ⬆️
... and 15 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yunlongn yunlongn closed this Aug 11, 2022
@yunlongn yunlongn reopened this Aug 12, 2022
@yu199195 yu199195 merged commit a13a935 into apache:master Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants