This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 214638 - RequestProcessor180386Test.testScheduleRepeatingSanityFixedRate() not reliable
Summary: RequestProcessor180386Test.testScheduleRepeatingSanityFixedRate() not reliable
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 7.3
Hardware: All All
: P3 normal (vote)
Assignee: Jaroslav Tulach
URL:
Keywords: TEST
Depends on:
Blocks: 216483
  Show dependency tree
 
Reported: 2012-06-22 15:49 UTC by Martin Entlicher
Modified: 2012-11-06 10:42 UTC (History)
0 users

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Entlicher 2012-06-22 15:49:51 UTC
http://deadlock.netbeans.org/hudson/job/NB-Core-Build/8543/testReport/junit/org.openide.util/RequestProcessor180386Test/testScheduleRepeatingSanityFixedRate/

junit.framework.AssertionFailedError: expected:<5> but was:<7>
	at org.openide.util.RequestProcessor180386Test.testScheduleRepeatingSanityFixedRate(RequestProcessor180386Test.java:889)

The problem is, that "C" can be executed more than 5 times, if main thread does run in 250ms after the latch is notified.
Maybe it's sufficient to test if c.runCount >= 5.

The similar problem can occur in testScheduleRepeatingSanityFixedDelay().
Comment 1 Martin Entlicher 2012-06-22 15:50:42 UTC
BTW: Maybe there should be "Utilities" component under platform?
Comment 2 Jaroslav Tulach 2012-10-24 07:15:12 UTC
ergonomics#6bfaa30dbd54
Comment 3 Quality Engineering 2012-10-28 01:49:20 UTC
Integrated into 'main-golden', will be available in build *201210280001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)
Changeset: http://hg.netbeans.org/main-golden/rev/6bfaa30dbd54
User: Jaroslav Tulach <jtulach@netbeans.org>
Log: #214638: Changing the assertEquals to assertAtLeast per Martin's suggestion