RFR: 8343838: Test EmptyDomainNotificationTest.java fails with ListenerNotFoundException

[David Holmes]

On Fri, 8 Nov 2024 20:46:52 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Looks like EmptyDomainNotificationTest has been working by accident for nearly 20 years.
> Removing notifications when not specifying a domain part of the ObjectName has actually been failing, but calling removeNotificationListener() without a domain has not been sharing the Exception.
> 
> ServerNotifForwarder.removeNotificationListener DOES need to add a domain part to an ObjectName that does not have one.
> 
> ServerNotifForwarder.addNotificationListener was changed in JDK-6238731, which is what this test is testing.
> 
> There is a numeric ID, BUT it still calls removeNotificationListener(ObjectName name, Integer listenerID) it does matter that the ObjectName does not match.
> 
> Maybe the failure to record and throw the Exceptions was the reason it was thought the numeric ID made the domain unnecessary.

This seems reasonable.

As we are approaching 400 failures of this test in the CI I'd like to see it integrated ASAP.

Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21993#pullrequestreview-2425883544