mbien commented on code in PR #5061:
URL: https://github.com/apache/netbeans/pull/5061#discussion_r1064077131


##########
platform/openide.util/test/unit/src/org/openide/util/TaskTest.java:
##########
@@ -260,26 +304,32 @@ public void 
testWaitWithTimeOutReturnsAfterTimeOutWhenTheTaskIsNotComputedAtAll
         
         fail ("Something wrong happened the task should wait for 1000ms but it 
took: " + time + "\n" + log);
     }
+    
+    
//--------------------------------------------------------------------------
     @Test
-    public void 
testWaitOnStrangeTaskThatStartsItsExecutionInOverridenWaitFinishedMethodLikeFolderInstancesDo
 () throws Exception {
+    public void 
waitOnTask_thatStartsItsExecutionWithOverridenWaitFinishedMethod() 

Review Comment:
   @matthiasblaesing @neilcsmith-net are you two also ok with this long-form 
method naming style for tests? I won't want to be the boo man here and block 
this for style reasons or merge something which isn't liked.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to