On Thu, 24 Mar 2022 20:59:20 GMT, Brent Christian <bchri...@openjdk.org> wrote:
>> Please review this change to the java/util/prefs/AddNodeChangeListener.jar >> test. >> >> Although the test specifies "-Djava.util.prefs.userRoot=." to ensure a fresh >> Preferences on each test run, MacOS does not seem to honor this, and still >> stores prefs in Library/. >> >> This test has long suffered intermittent failures. 8255031 added some >> debugging code; as of that change, the test fails fast as soon as an >> expected change event is not received. In particular, if the expected event >> is not received for adding the node, the test exits without removing the >> node. In this way, on Mac, the node can get "stuck" in the preferences of >> the machine. Future runs on the machine will already have the node, no node >> added change event will be generated (because the node was already there), >> the test will continue to fail without removing the node, etc. This matches >> observations on some CI machines. >> >> This change ensures that: >> * the test node is not present before the test >> * the test runs to completion, including removing the test node >> * the test node is not left behind after the test >> >> Thanks. > > 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 Marked as reviewed by lancea (Reviewer). ------------- PR: https://git.openjdk.java.net/jdk/pull/7908