zentol commented on a change in pull request #11639: [FLINK-16626][runtime] Prevent REST handler from being closed more than once URL: https://github.com/apache/flink/pull/11639#discussion_r404579710
########## File path: flink-clients/src/test/java/org/apache/flink/client/program/rest/RestClusterClientTest.java ########## @@ -271,6 +273,24 @@ public void testDetachedJobSubmission() throws Exception { } + /** + * Tests that ensure each handler is registered only once. + */ + @Test + public void testDuplicatedHandlerRegistration() throws Exception { Review comment: If we adjust the test name slightly we don't really need the comment (which is always a good thing since if CI fails people only see the test name ;) ) ```suggestion @Test public void testDuplicateHandlerRegistrationIsForbidden() throws Exception { ``` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services