On Sun, 29 Sep 2024 22:08:28 GMT, Phil Race <[email protected]> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   add multimonitor jtreg key
>
> test/jdk/java/awt/Frame/MultiScreenTest.java line 78:
> 
>> 76:             throw new SkippedException("You have only one monitor in 
>> your system - test passed");
>> 77:         }
>> 78:         MultiScreenTest obj = new MultiScreenTest();
> 
> Although this test will bail if there aren't two monitors, I'd like to see us 
> remember to add @key multimon to such tests.
> Then if someone wants to run multimon testsand ONLY those tests they can 
> specify this key 
> I'm sure we missed a couple of opportunities to do it already but let's not 
> miss this one.
> 
> See 
> https://github.com/openjdk/jdk/blob/master/test/jdk/TEST.ROOT

Ok.. added ` @key multimon `.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21199#discussion_r1780482257

Reply via email to