[
http://jira.magnolia.info/browse/MAGNOLIA-2311?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Gregory Joseph closed MAGNOLIA-2311.
------------------------------------
Resolution: Not an issue
Assignee: Gregory Joseph (was: Boris Kraft)
the trigger() method is called (5 times) in the for loop just above; the
execution will start after ~50ms, so the first assertion on runnable.executed
is false. We then sleep ~100ms, and runnable.executed is then true.
> 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: Gregory Joseph
> Priority: Minor
> 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/
----------------------------------------------------------------
