Remove sleeps from #time specs to try to prevent failures #20

Merged
merged 1 commit into from Sep 7, 2012

Conversation

Projects
None yet
1 participant
Owner

reinh commented Sep 7, 2012

This method is coupled to Time.now and it is difficult to see how to
remove this coupling to make it easier to test. This change will
hopefully prevent the test from failing when run on slower systems but
it is still far from ideal.

@reinh reinh Remove sleeps from #time specs to try to prevent failures
This method is coupled to Time.now and it is difficult to see how to
remove this coupling to make it easier to test. This change will
hopefully prevent the test from failing when run on slower systems but
it is still far from ideal.
8ec7c3d
Owner

reinh commented Sep 7, 2012

😖

This is better but not correct and I am wide open to suggestions.

reinh referenced this pull request Sep 7, 2012

Merged

Fix Build Status image #21

@reinh reinh added a commit that referenced this pull request Sep 7, 2012

@reinh reinh Merge pull request #20 from reinh/fix-time-related-test-failure
Remove sleeps from #time specs to try to prevent failures
1dc4613

@reinh reinh merged commit 1dc4613 into master Sep 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment