rpuch commented on a change in pull request #467:
URL: https://github.com/apache/ignite-3/pull/467#discussion_r755806456
##########
File path:
modules/configuration/src/test/java/org/apache/ignite/internal/configuration/notifications/ConfigurationListenerTest.java
##########
@@ -915,4 +957,17 @@ void testGetConfigFromNotificationEventOnUpdate() throws
Exception {
configuration.elements().get(key).str().update(newVal).get(1, SECONDS);
}
+
+ @Test
+ void
listenerGetsNotifiedOnCommonPropertyChangeInPolymorphicBaseConfigUnderNoNamedConfig()
throws Exception {
Review comment:
This is not actually a name because nobody uses test methods for
invocations. This is actually a description of a test. When you read a test
report, you only see the 'method name' and not javadoc, so it seems more useful
to put test case description into a name and not into a javadoc that you can
only see in the code.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
