On Fri, 30 Jan 2026 16:28:28 GMT, Mikhail Yankelevich
<[email protected]> wrote:
>> test/jdk/sun/security/krb5/config/ConfPlusProp.java line 44:
>>
>>> 42: System.getenv("LOGONSERVER") != null) {
>>> 43: throw new SkippedException(
>>> 44: "Test cannot run in a domain.");
>>
>> This is not a problem at all. No `SkippedException` is needed.
>
> This still skips the whole test so I believe this should throw a
> SkippedException. Alternatively I thought to require os not being Windows.
> What would you prefer to do?
I don't think you should skip it for all windows. The update looks good to me.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28898#discussion_r3156371972
