[KAFKA-16791] Add thread detection to ClusterTestExtensions

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 3.9.0
• Assignee: PoAn Yang
• Reporter: Chia-Ping Tsai
• Created: 17/May/24 19:07
• Updated: 23/Jul/24 03:19
• Resolved: 23/Jul/24 03:16

Description

`ClusterTestExtensions` should implement `BeforeAllCallback` and `AfterAllCallback` by `TestUtils.verifyNoUnexpectedThreads`

We can leverage `BeforeEachCallback`/`AfterEachCallback` to implement new AfterEachCallback instead of `verifyNoUnexpectedThreads`. Notice the new thread detection should avoid cascading failure - the thread leak should make specific test case (rather than all subsequent test cases) fail

`ClusterTestExtensions` should implement `BeforeAllCallback` and `AfterAllCallback` to check the thread leak for each "test case"

Issue Links

• is related to: KAFKA-16072 Create Junit 5 extension to detect thread leak
• relates to: KAFKA-17151 Remove waitForCondition when detecting thread leak
• links to: GitHub Pull Request #16427
• links to: GitHub Pull Request #16499