1 Reply Latest reply on Jul 16, 2009 12:05 AM by clebert.suconic

    ScheduledDeliveryTest::testDelayedRedeliveryDefault eventual

    clebert.suconic

      there are two assertions on this test:


      assertTrue(time - now >= delay);
      
       // Hudson can introduce a large degree of indeterminism
       assertTrue(time - now + ">" + (delay + 1000), time - now < delay + 1000);
      
      



      The second assertion can fail due to delays such as large Garbage Collections...etc.

      IMO we should just remove it, since the first assertion should be already sufficient for scheduled messages.

      that failure is also causing cascade failures due to something not being consumed or cleared up.