Chromium Code Reviews| Index: tests/isolate/multiple_timer_test.dart |
| =================================================================== |
| --- tests/isolate/multiple_timer_test.dart (revision 14114) |
| +++ tests/isolate/multiple_timer_test.dart (working copy) |
| @@ -23,29 +23,29 @@ |
| void timeoutHandler1(Timer timer) { |
| int endTime = (new Date.now()).millisecondsSinceEpoch; |
| - expect((endTime - _startTime1) >= TIMEOUT1); |
| - expect(_order[_message] == 0); |
| + expect(endTime - _startTime1, greaterThanOrEqualTo(TIMEOUT1)); |
| + expect(_order[_message], 0); |
|
Jennifer Messerly
2012/10/26 02:57:05
isZero?
gram
2012/10/26 22:26:27
I chose not to use isZero as we have 1, 2, 3 below
|
| _message++; |
| } |
| void timeoutHandler2(Timer timer) { |
| int endTime = (new Date.now()).millisecondsSinceEpoch; |
| - expect((endTime - _startTime2) >= TIMEOUT2); |
| - expect(_order[_message] == 1); |
| + expect(endTime - _startTime2, greaterThanOrEqualTo(TIMEOUT2)); |
| + expect(_order[_message], 1); |
|
Jennifer Messerly
2012/10/26 02:57:05
equals(1) and below?
gram
2012/10/26 22:26:27
Automagic.
|
| _message++; |
| } |
| void timeoutHandler3(Timer timer) { |
| int endTime = (new Date.now()).millisecondsSinceEpoch; |
| - expect((endTime - _startTime3) >= TIMEOUT3); |
| - expect(_order[_message] == 2); |
| + expect(endTime - _startTime3, greaterThanOrEqualTo(TIMEOUT3)); |
| + expect(_order[_message], 2); |
| _message++; |
| } |
| void timeoutHandler4(Timer timer) { |
| int endTime = (new Date.now()).millisecondsSinceEpoch; |
| - expect((endTime - _startTime4) >= TIMEOUT4); |
| - expect(_order[_message] == 3); |
| + expect(endTime - _startTime4, greaterThanOrEqualTo(TIMEOUT4)); |
| + expect(_order[_message], 3); |
| _message++; |
| } |