TestBoost opened a new pull request, #8436: URL: https://github.com/apache/rocketmq/pull/8436
<!-- Please make sure the target branch is right. In most case, the target branch should be `develop`. --> ### Which Issue(s) This PR Fixes <!-- Please ensure that the related issue has already been created, and [link this pull request to that issue using keywords](<https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword>) to ensure automatic closure. --> We are not fixing existing issues in this commit. ### Brief Description <!-- Write a brief description for your pull request to help the maintainer understand the reasons behind your changes. --> Changes proposed in this pull request: We put all the initialization of the mocks and variables into the method body of @BeforeAll and @AfterAll. There are two tests in the test class ConsumeMessageConcurrentlyServiceTest. However, these two tests are not affecting each other. The test runtime can jump from 2.771 s to 1.768 s after applying the changes when run on our machine. ### How Did You Test This Change? <!-- In order to ensure the code quality of Apache RocketMQ, we expect every pull request to have undergone thorough testing. --> We just ran this test class in any order and it passes. We ran this test class 10 times and it also passes. We also ran all tests in client module 10 times, and there are no changes for the test results. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@rocketmq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
