DelayedExecutorTest doesn't call executor.trigger() and so can never pass
-------------------------------------------------------------------------

                 Key: MAGNOLIA-2311
                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2311
             Project: Magnolia
          Issue Type: Bug
    Affects Versions: 3.6.1
            Reporter: Joshua Frankamp
            Assignee: Boris Kraft
         Attachments: DelayedExecutorTest.diff

AFAIK and I could be wrong, the runnable.executed will never be true unless 
executor.trigger is at least called once after sleeping for 100 ms.

Index: DelayedExecutorTest.java
===================================================================
--- DelayedExecutorTest.java    (revision 17249)
+++ DelayedExecutorTest.java    (working copy)
@@ -69,6 +69,7 @@
         }
         assertEquals(false, runnable.executed);
         Thread.sleep(100);
+        executor.trigger();
         assertEquals(true, runnable.executed);
     }
 


-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to