Skip to content

Conversation

@forfreeday
Copy link
Contributor

What does this PR do?

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

@codecov-commenter
Copy link

codecov-commenter commented Apr 17, 2023

Codecov Report

Merging #5147 (ff5fb5c) into master (da2673c) will increase coverage by 0.69%.
The diff coverage is 11.76%.

📣 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              @@
##             master    #5147      +/-   ##
============================================
+ Coverage     54.69%   55.38%   +0.69%     
- Complexity     8235     8254      +19     
============================================
  Files           841      833       -8     
  Lines         49717    49271     -446     
  Branches       5459     5460       +1     
============================================
+ Hits          27194    27291      +97     
+ Misses        20384    19841     -543     
  Partials       2139     2139              
Impacted Files Coverage Δ
...main/java/org/tron/core/utils/TransactionUtil.java 44.80% <ø> (-0.32%) ⬇️
...c/main/java/org/tron/core/db/TransactionTrace.java 76.21% <0.00%> (+6.70%) ⬆️
framework/src/main/java/org/tron/core/Wallet.java 39.11% <0.00%> (+1.84%) ⬆️
...ain/java/org/tron/core/services/RpcApiService.java 5.38% <ø> (+0.43%) ⬆️
...ron/core/services/http/FullNodeHttpApiService.java 94.47% <ø> (-0.24%) ⬇️
...on/core/services/http/HttpSelfFormatFieldName.java 0.00% <ø> (ø)
...rc/main/java/org/tron/core/services/http/Util.java 28.51% <ø> (+0.30%) ⬆️
.../services/interfaceOnPBFT/RpcApiServiceOnPBFT.java 18.40% <ø> (+0.95%) ⬆️
...s/interfaceOnSolidity/RpcApiServiceOnSolidity.java 18.09% <ø> (+1.80%) ⬆️
...mework/src/main/java/org/tron/program/Version.java 50.00% <ø> (ø)
... and 2 more

... and 17 files with indirect coverage changes

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

@forfreeday forfreeday merged commit e5e347d into tronprotocol:master Apr 17, 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.

6 participants