Skip to content

Conversation

@lxcmyf
Copy link
Contributor

@lxcmyf lxcmyf commented Apr 13, 2023

What does this PR do?
optimize Stake2.0 code
Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

317787106 and others added 30 commits August 1, 2022 19:01
…lready exist; if ipv4 and ipv6 not exist, use local ipv4
fix(test): add some test cases of utils
xxo1shine and others added 17 commits April 18, 2023 14:32
feat(backup): add unit tests for backup module
1. check second cache config in unit tests
…_test_check

feat(db): check second cache config in unit tests
feat(net): optimize getBlockChainSummary logic
fix(test): merge unit test optimization branch
…tcase

fix(net): optimze test cases to make coverage stable
@lxcmyf lxcmyf force-pushed the feature/optimize_stake2_code branch 4 times, most recently from 1ca0ecd to abd8a1e Compare April 23, 2023 08:03
@codecov-commenter
Copy link

codecov-commenter commented Apr 23, 2023

Codecov Report

Merging #5134 (a8fc651) into feature/optimize_stake2_code (631c5df) will increase coverage by 4.09%.
The diff coverage is 51.19%.

📣 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                         @@
##             feature/optimize_stake2_code    #5134      +/-   ##
==================================================================
+ Coverage                           54.85%   58.95%   +4.09%     
- Complexity                           8260     8767     +507     
==================================================================
  Files                                 841      836       -5     
  Lines                               49714    49585     -129     
  Branches                             5460     5519      +59     
==================================================================
+ Hits                                27273    29233    +1960     
+ Misses                              20310    18049    -2261     
- Partials                             2131     2303     +172     
Impacted Files Coverage Δ
...main/java/org/tron/core/utils/TransactionUtil.java 44.80% <ø> (-0.32%) ⬇️
...vm/nativecontract/UnDelegateResourceProcessor.java 75.00% <0.00%> (+1.41%) ⬆️
...java/org/tron/core/capsule/TransactionCapsule.java 54.06% <0.00%> (+1.39%) ⬆️
...c/main/java/org/tron/core/db/TransactionTrace.java 76.21% <0.00%> (+6.70%) ⬆️
.../src/main/java/org/tron/common/utils/JsonUtil.java 30.76% <0.00%> (ø)
common/src/main/java/org/tron/core/Constant.java 50.00% <ø> (ø)
.../main/java/org/tron/core/config/args/SeedNode.java 100.00% <ø> (ø)
...n/java/org/tron/common/backup/message/Message.java 40.00% <0.00%> (+40.00%) ⬆️
.../java/org/tron/common/client/WalletGrpcClient.java 0.00% <0.00%> (ø)
...src/main/java/org/tron/core/capsule/utils/RLP.java 25.63% <ø> (+6.72%) ⬆️
... and 60 more

... and 179 files with indirect coverage changes

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

@lxcmyf lxcmyf force-pushed the feature/optimize_stake2_code branch 3 times, most recently from a8b4e3c to f53d1da Compare April 23, 2023 10:11
@lxcmyf lxcmyf force-pushed the feature/optimize_stake2_code branch 2 times, most recently from a3b224d to a8fc651 Compare April 25, 2023 02:53
@halibobo1205 halibobo1205 merged commit 7414c11 into tronprotocol:feature/optimize_stake2_code May 10, 2023
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.