On Mon, 21 Jul 2025 11:55:15 GMT, Mikhail Yankelevich
<[email protected]> wrote:
>> sun/security/krb5/config/native/TestDynamicStore.java is commonly run
>> without sudo, which leads to the access related failure.
>> Added a ui popup reminding to run the test with sudo, making instructions
>> clearer and more noticeable
>
> Mikhail Yankelevich has updated the pull request incrementally with one
> additional commit since the last revision:
>
> comments
test/jdk/sun/security/krb5/config/native/TestDynamicStore.java line 65:
> 63: if (!"root".equals(System.getProperty("user.name"))) {
> 64:
> 65: JOptionPane.showMessageDialog(null, """
Is this dialog box necessary if the test will fail with the same message?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26293#discussion_r2454808353