Skip to content

fix(net): optimze test cases to make coverage stable#5133

Merged
317787106 merged 10 commits intotronprotocol:release_v4.7.2from
317787106:hotfix/stable_testcase
Apr 19, 2023
Merged

fix(net): optimze test cases to make coverage stable#5133
317787106 merged 10 commits intotronprotocol:release_v4.7.2from
317787106:hotfix/stable_testcase

Conversation

@317787106
Copy link

@317787106 317787106 commented Apr 13, 2023

What does this PR do?

  • optimize some test case, so that test case's coverage of same code will stay the same.
  • fix the bug that oom occurs when run all test cases

Why are these changes required?

  • sometimes, the coverage of the same code in one machine may change because of network latency.
  • sometimes, the coverage of the same code in different machines may change.

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link

codecov-commenter commented Apr 13, 2023

Codecov Report

Merging #5133 (7a8ceee) into release_v4.7.2 (cc9a944) will increase coverage by 0.04%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@                 Coverage Diff                  @@
##             release_v4.7.2    #5133      +/-   ##
====================================================
+ Coverage             56.95%   56.99%   +0.04%     
- Complexity             8521     8533      +12     
====================================================
  Files                   836      836              
  Lines                 49602    49603       +1     
  Branches               5518     5518              
====================================================
+ Hits                  28249    28271      +22     
+ Misses                19068    19051      -17     
+ Partials               2285     2281       -4     
Impacted Files Coverage Δ
...rc/main/java/org/tron/core/net/TronNetService.java 87.23% <100.00%> (+0.13%) ⬆️

... and 12 files with indirect coverage changes

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

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.

4 participants