Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix problem with nondeterministic timer dependent tests #629

Closed
TimLethbridge opened this issue Oct 10, 2015 · 2 comments
Closed

Fix problem with nondeterministic timer dependent tests #629

TimLethbridge opened this issue Oct 10, 2015 · 2 comments

Comments

@TimLethbridge
Copy link
Member

When the server runs slowly we have been getting failures on

cruise.queued.statemachine.test.MultipleQueuedStateMachinesTest (1)
cruise.queued.statemachine.test.QueuedStateMachineTest_timedTransition_1 (1)
cruise.statemachine.test.DoActivityTest (1)

Some of these were already commented out. Others will have to be deactivated then fixed since we have been getting random build fails

Test Failures (2)
cruise.queued.statemachine.test.QueuedStateMachineTest_timedTransition_1 (1)
TestName: processEvents Duration: 2.067

Expected: not <0>
but: was <0>

                            junit.framework.AssertionFailedError: 

Expected: not <0>
but: was <0>
at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
at cruise.queued.statemachine.test.QueuedStateMachineTest_timedTransition_1.processEvents(QueuedStateMachineTest_timedTransition_1.java:59)

cruise.statemachine.test.DoActivityTest (1)
TestName: autoTransitionSyncAfterMultipleDo Duration: 1.941
expected:<[1, 2, [3]]> but was:<[1, 2, []>

                            junit.framework.AssertionFailedError: expected:<[1, 2, [3]]> but was:<[1, 2, []>
at cruise.statemachine.test.DoActivityTest.autoTransitionSyncAfterMultipleDo(DoActivityTest.java:49)
@Nava2
Copy link
Contributor

Nava2 commented Oct 11, 2015

I think this is a duplicate of #591. However, I can actually do my proposed solution. I just forgot about it.

@TimLethbridge
Copy link
Member Author

This is a duplicate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants