On Thu, 7 Sep 2023 16:22:18 GMT, Soumadipta Roy <d...@openjdk.org> wrote:

>> test/jdk/java/util/concurrent/tck/JSR166TestCase.java line 45:
>> 
>>> 43:  * @modules java.management
>>> 44:  * @run junit/othervm/timeout=1000 JSR166TestCase
>>> 45:  * @run junit/othervm/timeout=1000 -Djava.security.manager=allow 
>>> JSR166TestCase
>> 
>> security manager testing is relatively less important.   I would move this 
>> to a lower tier, while moving a test with 
>> -Djsr166.testImplementationDetails=true into tier1
>
> Hi Martin,
> 
> I will raise another commit trying to add summary for the segregations

I still think the test with -Djava.security.manager=allow is relatively low 
value, so would also move it into its own @test id=security-manager

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/15619#discussion_r1325339151

Reply via email to