Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-2311

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

    XMLWordPrintable

Details

    • Bug
    • Resolution: Not an issue
    • Minor
    • None
    • 3.6.1
    • None
    • None

    Description

      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);
      }

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              gjoseph Magnolia International
              jfrank Julian Frank
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD