RFR: 8291651: CleanerTest.java fails with "Cleanable was cleaned"

[Roger Riggs]

On Mon, 29 Aug 2022 17:14:41 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> CleanerTest is failing intermittently on Aarch64, with -Xcomp, and using a VirtualThread for the Cleaner in the test.
>> The extensively relies on references processing and invokes GC on a very short cycle.
>> The existing 10ms delay between GC requests is too short for the changes to reference states to be reliably visible.
>> 
>> The delay between gc requests is extended to 200ms and the tests pass reliably.
>> There are small improvements in diagnostic messages and cleanup of unused imports.
>
> test/jdk/java/lang/ref/CleanerTest.java line 569:
> 
>> 567:         obj = null;
>> 568:         checkCleaned(s.getSemaphore(), true,
>> 569:                 "Object was cleaned using CleanerFactory.cleaner():");
> 
> So the message in case of failure for this case will now be?
> 
> Should have been run: Object was cleaned using CleanerFactory.cleaner():

Yes, (just a typo fix); but the message describes the function of the cleanup
and is prefixed in the error as to whether it was run or not.

-------------

PR: https://git.openjdk.org/jdk/pull/10048