On Mon, 22 May 2023 10:09:27 GMT, Alan Bateman <[email protected]> wrote:
>> test/jdk/java/util/concurrent/ThreadPerTaskExecutor/ThreadPerTaskExecutorTest.java
>> line 578:
>>
>>> 576: try {
>>> 577:
>>> scheduleInterruptAt("java.util.concurrent.ThreadPerTaskExecutor.invokeAny");
>>> 578: executor.invokeAny(Set.of(task));
>>
>> Seems strange to test invokeAny when there is only one task?
>
>> Seems strange to test invokeAny when there is only one task?
>
> There are several tests further up that exercise invokeAny with more than one
> task. This one is testing interrupt of invokeAny, we could expect it to test
> interrupting with many tasks if needed.
OK - looks fine to me then.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14072#discussion_r1200349759
