[11u] jtreg.SkippedException?

[Jonathan Gibbons]

Hi Misha, Severin, all,

When we added SkippedException we specifically designed it to avoid any 
dependencies on jtreg.

As far as jtreg is concerned, any class named "jtreg.SkippedException" 
will do. There is not even a requirement
that all tests should use the same definition of jtreg.SkippedException.

It should be sufficient to do the following:

1. Add a definition of jtreg.SkippedException in some test library, and 
use it in tests.
2. Update the requiredVersion for jtreg in TEST.ROOT to be a version 
that handles SkippedException.

Even 2 is somewhat optional, but if you don't do it, the test will just 
fail because of the SkippedException.

-- Jon


On 07/31/2019 09:43 AM, mikhailo.seledtsov at oracle.com wrote:
> Hi Severin,
>
>   There might be a dependency on a specific version of Jtreg to 
> support the jtreg.SkippedException.
>
> Looping in Igor and Jon.
>
>
> Thanks,
>
> Misha
>
> On 7/31/19 9:28 AM, Severin Gehwolf wrote:
>> Hi,
>>
>> During a couple of test related backports to OpenJDK 11u a recurring
>> issue seems to be a need to change a jdk/jdk patch due to
>> jtreg.SkippedException not being present in jdk-updates/jdk11u{,-dev}.
>>
>> I'd propose to get it added to the JDK 11u test trees so as to make
>> test backports easier going forward. While this might add a patch which
>> is JDK 11u only, it removes the need for reviews for more test
>> backports going forward.
>>
>> Note that JDK-8208655 introduced jtreg.SkippedException for JDK 12,
>> which seems too big of a patch for a full backport. I could include
>> SkippedException in one of my pending JDK 11u backports[1][2] or get it
>> added separately. I'm leaning towards a separate JDK 11u-only patch.
>>
>> Thoughts?
>>
>> Thanks,
>> Severin
>>
>> [1] 
>> http://mail.openjdk.java.net/pipermail/jdk-updates-dev/2019-July/001456.html
>> [2] 
>> http://mail.openjdk.java.net/pipermail/jdk-updates-dev/2019-July/001454.html
>>