Skip to content

fix(test): release resource of testcases gracefully#6437

Merged
kuny0707 merged 18 commits into
tronprotocol:release_v4.8.1from
317787106:hotfix/fix_halt
Sep 5, 2025
Merged

fix(test): release resource of testcases gracefully#6437
kuny0707 merged 18 commits into
tronprotocol:release_v4.8.1from
317787106:hotfix/fix_halt

Conversation

@317787106

@317787106 317787106 commented Sep 3, 2025

Copy link
Copy Markdown
Collaborator

What does this PR do?

  • release resource of testcases to avoid halting, including channel of gRPC, zmq, ExecutorService
  • fix the bug of testcase, such as NullPointException
  • modify testcases those cost too long
  • delete shutdownhook in testcase
  • delete temporary files or dir after testcases

Why are these changes required?

This PR has been tested by:

  • Unit Tests
  • Manual Testing

Follow up

Extra details

Comment thread framework/src/test/java/org/tron/core/CreateCommonTransactionTest.java Outdated
Comment thread framework/src/test/java/org/tron/core/net/BaseNetTest.java
Comment thread framework/src/test/java/org/tron/core/services/DelegationServiceTest.java Outdated
Comment thread framework/src/test/java/org/tron/core/services/WalletApiTest.java Outdated
@kuny0707 kuny0707 merged commit 02f81c6 into tronprotocol:release_v4.8.1 Sep 5, 2025
7 checks passed
@kuny0707 kuny0707 moved this to Done in java-tron Sep 5, 2025
halibobo1205 pushed a commit to halibobo1205/java-tron that referenced this pull request Sep 11, 2025
* delete BaseNet, BaseNetTest

* release resource of gRPC

* optimize testcase to reduce the time cost

* delete db after testcase in tmp dir; short the time cost of some case; optimize some shutdown action

* use shutdownnow instead of shutdown in close Channel
halibobo1205 pushed a commit to halibobo1205/java-tron that referenced this pull request Oct 14, 2025
* delete BaseNet, BaseNetTest

* release resource of gRPC

* optimize testcase to reduce the time cost

* delete db after testcase in tmp dir; short the time cost of some case; optimize some shutdown action

* use shutdownnow instead of shutdown in close Channel
warku123 pushed a commit to warku123/java-tron that referenced this pull request Feb 6, 2026
* delete BaseNet, BaseNetTest

* release resource of gRPC

* optimize testcase to reduce the time cost

* delete db after testcase in tmp dir; short the time cost of some case; optimize some shutdown action

* use shutdownnow instead of shutdown in close Channel
@317787106 317787106 deleted the hotfix/fix_halt branch June 17, 2026 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

9 participants