[8u] RFR: 8132376: Add @requires os.family to the client tests with access to internal OS-specific API
Alex Kashchenko
akashche at redhat.com
Mon Jun 15 14:41:33 UTC 2020
On 06/15/2020 01:59 PM, Zhengyu Gu wrote:
> Looks good to me.
Thanks for the review! I've added jdk8u-fix-request label to the issue.
>
> Thanks,
>
> -Zhengyu
>
> On 6/15/20 6:22 AM, Alex Kashchenko wrote:
>> Hi,
>>
>> Please review the backport of JDK-8132376 to 8u.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8132376
>>
>> 9 commit: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/3aef124b3176
>>
>> 8u-dev webrev:
>> http://cr.openjdk.java.net/~akasko/jdk8u/8132376/webrev.00/
>>
>> This change introduces two new Windows-specific tests. 8u patch
>> removes @modules jtreg tag, and adds a type cast to getPeer() call
>> [1][2], that is necessary due to changes introduced in 9 with
>> JDK-8074028.
>>
>> Testing: checked that new tests pass on Windows. Checked that tests
>> marked with "@requires (os.family == "mac")" are selected on macOS.
>>
>>
>> [1] http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/3aef124b3176#l2.73
>> [2] http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/3aef124b3176#l3.76
>>
>
--
-Alex
More information about the jdk8u-dev
mailing list