8358633: Test ThreadPoolExecutorTest::testTimedInvokeAnyNullTimeUnit is broken by JDK-8347491

Viktor Klang · Viktor Klang · commit 782bbca439cd · 2025-06-05T12:04:57.000Z
Reviewed-by: alanb
diff --git a/test/jdk/java/util/concurrent/tck/ThreadPoolExecutorTest.java b/test/jdk/java/util/concurrent/tck/ThreadPoolExecutorTest.java
@@ -1727,7 +1727,7 @@ public void testTimedInvokeAnyNullTimeUnit() throws Exception {
                 e.invokeAny(l, randomTimeout(), null);
                 shouldThrow();
             } catch (NullPointerException success) {
-                assertEquals("Cannot invoke \"java.util.concurrent.TimeUnit.toNanos(long)\" because \"unit\" is null", success.getMessage());
+                // Do not check the message, as ThreadPoolExecutor does not override invokeAny
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1727,7 +1727,7 @@ public void testTimedInvokeAnyNullTimeUnit() throws Exception {`
`1727`	`1727`	`e.invokeAny(l, randomTimeout(), null);`
`1728`	`1728`	`shouldThrow();`
`1729`	`1729`	`} catch (NullPointerException success) {`
`1730`		`- assertEquals("Cannot invoke \"java.util.concurrent.TimeUnit.toNanos(long)\" because \"unit\" is null", success.getMessage());`
	`1730`	`+ // Do not check the message, as ThreadPoolExecutor does not override invokeAny`
`1731`	`1731`	`}`
`1732`	`1732`	`}`
`1733`	`1733`	`}`