[fix][test] Fix Thread.getThreadGroup().getName() NPE (#22070)

Technoboy- · web-flow · commit 0b6bd70b8d1e · 2024-02-19T18:09:14.000+02:00
diff --git a/buildtools/src/main/java/org/apache/pulsar/tests/ThreadLeakDetectorListener.java b/buildtools/src/main/java/org/apache/pulsar/tests/ThreadLeakDetectorListener.java
@@ -177,7 +177,8 @@ private static boolean shouldSkipThread(Thread thread) {
             return true;
         }
         // skip Testcontainers threads
-        if (thread.getThreadGroup() != null && "testcontainers".equals(thread.getThreadGroup().getName())) {
+        final ThreadGroup threadGroup = thread.getThreadGroup();
+        if (threadGroup != null && "testcontainers".equals(threadGroup.getName())) {
             return true;
         }
         String threadName = thread.getName();

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,8 @@ private static boolean shouldSkipThread(Thread thread) {`
`177`	`177`	`return true;`
`178`	`178`	`}`
`179`	`179`	`// skip Testcontainers threads`
`180`		`- if (thread.getThreadGroup() != null && "testcontainers".equals(thread.getThreadGroup().getName())) {`
	`180`	`+ final ThreadGroup threadGroup = thread.getThreadGroup();`
	`181`	`+ if (threadGroup != null && "testcontainers".equals(threadGroup.getName())) {`
`181`	`182`	`return true;`
`182`	`183`	`}`
`183`	`184`	`String threadName = thread.getName();`