On Wed, 23 Mar 2022 17:29:04 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> Brent Christian has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add test failure message to 'couldn't remove' RuntimeException
>
> test/jdk/java/util/prefs/AddNodeChangeListener.java line 63:
>
>> 61: } finally {
>> 62: // Make sure test node is not present after the test
>> 63: clearPrefs();
>
> If this call throws a `RuntimeException("Unable to clear...")`, it will
> override the possible `RuntimeException("Failed")` in the `try` block, but I
> think it is OK.
That's true - I hadn't considered that. I think it's OK, too. Even so, I've
added a message to the exception thrown from clearPrefs() to indicate if the
test failed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7908
