[jdk21u-dev] RFR: 8375080: The tools/jpackage/windows/Win8365790Test.java may fail with ClassNotFoundException: jtreg.SkippedException

[Min Choi]

On Tue, 10 Feb 2026 18:46:45 GMT, Min Choi <duke at openjdk.org> wrote:

> `Win8365790Test` calls `TKit.throwSkippedException()` when the test environment doesn't support Ctrl+C signal delivery.
> 
> In JDK 17/21, `TKit.throwSkippedException()` uses reflection (`Class.forName("jtreg.SkippedException")`) to load the exception class, which requires `jtreg.SkippedException` to be on the classpath.
> 
> public static RuntimeException throwSkippedException(String reason) {
>     trace("Skip the test: " + reason);
>     RuntimeException ex = ThrowingSupplier.toSupplier(
>             () -> (RuntimeException) Class.forName("jtreg.SkippedException").getConstructor(
>                     String.class).newInstance(reason)).get();
>     currentTest.notifySkipped(ex);
>     throw ex;
> }
> 
> 
> The test was missing the jtreg directives to build this class:
> 
> @library /test/lib
> @build jtreg.SkippedException
> 
> 
> This pattern is already used by other jpackage tests that call `throwSkippedException()` (e.g., `SigningAppImageTest.java`).
> 
> Note: JDK 25+ has this fixed differently via [JDK-8352419](https://bugs.openjdk.org/browse/JDK-8352419) which [hardcodes](https://github.com/openjdk/jdk/commit/70e325004536eb9b79f4943f27eed4ac6cec8bd5#diff-409e0d5eb5d024e7613e041c416fe862752cce72f19c80bb585511239fab9399R1144-R1170) `SkippedException` directly in `TKit.java`.

Thank you everyone for reviewing!

I believe @alexeysemenyukoracle's solution is the correct approach. We should backport the relevant parts of https://github.com/openjdk/jdk/pull/24294 that hardcode SkippedException, which will benefit other tests in JDK 21 and 17 that rely on this.

We have two options:
* Integrate this PR (since it's a straightforward fix) and then backport Alexey's fix separately
* Skip this PR and proceed directly to backporting Alexey's fix

I'd appreciate input on which approach to prefer. Both paths will achieve the same end result, so I'm happy to go with whatever makes most sense.

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

PR Comment: https://git.openjdk.org/jdk21u-dev/pull/2606#issuecomment-3898691031