RFR: 8283349 : Robustness improvements to java/util/prefs/AddNodeChangeListener.jar [v2]

[Brent Christian]

On Wed, 23 Mar 2022 17:29:04 GMT, Naoto Sato <naoto at 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