Chromium Code Reviews| Index: tests/lib/unittest/unittest_test.dart |
| =================================================================== |
| --- tests/lib/unittest/unittest_test.dart (revision 9215) |
| +++ tests/lib/unittest/unittest_test.dart (working copy) |
| @@ -151,12 +151,14 @@ |
| var s = '${m.foo(1,2)}${m.foo(1,1)}${m.foo(9,10)}' |
| '${m.bar(1,1)}${m.foo(1,2)}'; |
| - getLogs(m, callsTo('foo', anything, anything)).verify(calledExactly(4)); |
| - getLogs(m, callsTo('foo', 1, anything)).verify(calledExactly(3)); |
| - getLogs(m, callsTo('foo', 9, anything)).verify(calledOnce); |
| - getLogs(m, callsTo('foo', anything, 2)).verify(calledExactly(2)); |
| - getLogs(m, callsTo('foobar')).verify(neverCalled); |
| - getLogs(m, callsTo('foo', 10, anything)).verify(neverCalled); |
| + m.getLogs(callsTo('foo', anything, anything)).verify(happenedExactly(4)); |
|
Jennifer Messerly
2012/06/29 23:24:09
nit: 80 col
|
| + m.getLogs(callsTo('foo', 1, anything)).verify(happenedExactly(3)); |
| + m.getLogs(callsTo('foo', 9, anything)).verify(happenedOnce); |
| + m.getLogs(callsTo('foo', anything, 2)).verify(happenedExactly(2)); |
| + m.getLogs(callsTo('foobar')).verify(neverHappened); |
| + m.getLogs(callsTo('foo', 10, anything)).verify(neverHappened); |
| + m.getLogs(callsTo('foo'), returning(anyOf('A', 'C'))). |
| + verify(happenedExactly(2)); |
| expect(s, 'ACDEB'); |
| }); |
| } else if (testName == 'mock test 2 (MockList)') { |
| @@ -173,21 +175,21 @@ |
| m.add('foo'); |
| m.add('bar'); |
| - getLogs(m, callsTo('add')).verify(calledExactly(2)); |
| - getLogs(m, callsTo('add', 'foo')).verify(calledOnce); |
| + m.getLogs(callsTo('add')).verify(happenedExactly(2)); |
| + m.getLogs(callsTo('add', 'foo')).verify(happenedOnce); |
| }); |
| } else if (testName == 'mock test 3 (Spy)') { |
| test(testName, () { |
| var p = new FooSpy(); |
| p.sum(1, 2, 3); |
| - getLogs(p, callsTo('sum')).verify(calledOnce); |
| + p.getLogs(callsTo('sum')).verify(happenedOnce); |
| p.sum(2, 2, 2); |
| - getLogs(p, callsTo('sum')).verify(calledExactly(2)); |
| - getLogs(p, callsTo('sum')).verify(sometimeReturned(6)); |
| - getLogs(p, callsTo('sum')).verify(alwaysReturned(6)); |
| - getLogs(p, callsTo('sum')).verify(neverReturned(5)); |
| + p.getLogs(callsTo('sum')).verify(happenedExactly(2)); |
| + p.getLogs(callsTo('sum')).verify(sometimeReturned(6)); |
| + p.getLogs(callsTo('sum')).verify(alwaysReturned(6)); |
| + p.getLogs(callsTo('sum')).verify(neverReturned(5)); |
| p.sum(2, 2, 1); |
| - getLogs(p, callsTo('sum')).verify(sometimeReturned(5)); |
| + p.getLogs(callsTo('sum')).verify(sometimeReturned(5)); |
| }); |
| } else if (testName == 'mock test 4 (Excess calls)') { |
| test(testName, () { |
| @@ -195,9 +197,9 @@ |
| m.when(callsTo('foo')).alwaysReturn(null); |
| m.foo(); |
| m.foo(); |
| - getLogs(m, callsTo('foo')).verify(calledOnce); |
| + m.getLogs(callsTo('foo')).verify(happenedOnce); |
| }); |
| - } else if (testName == 'mock test 5 (No behavior)') { |
| + } else if (testName == 'mock test 5 (No action)') { |
| test(testName, () { |
| var m = new Mock(); |
| m.when(callsTo('foo')).thenReturn(null); |
| @@ -208,8 +210,15 @@ |
| test(testName, () { |
| var p = new FooSpy(); |
| p.sum(1, 2, 3); |
| - getLogs(p, callsTo('sum')).verify(sometimeReturned(0)); |
| + p.getLogs(callsTo('sum')).verify(sometimeReturned(0)); |
| }); |
| + } else if (testName == 'mock test 7 (No behavior)') { |
| + test(testName, () { |
| + var m = new Mock(throwIfNoBehavior:true); |
| + m.when(callsTo('foo')).thenReturn(null); |
| + m.foo(); |
| + m.bar(); |
| + }); |
| } |
| }); |
| } |
| @@ -244,8 +253,9 @@ |
| 'mock test 2 (MockList)', |
| 'mock test 3 (Spy)', |
| 'mock test 4 (Excess calls)', |
| - 'mock test 5 (No behavior)', |
| - 'mock test 6 (No matching return)' |
| + 'mock test 5 (No action)', |
| + 'mock test 6 (No matching return)', |
| + 'mock test 7 (No behavior)' |
| ]; |
| expected = [ |
| @@ -268,9 +278,11 @@ |
| message: 'Expected foo() to be called 1 times but:' |
| ' was called 2 times'), |
| buildStatusString(0, 1, 0, tests[14], |
| - message: 'Caught Exception: No behavior specified for method foo'), |
| + message: 'Caught Exception: No more actions for method foo'), |
| buildStatusString(0, 1, 0, tests[15], |
| - message: 'Expected sum() to sometimes return <0> but: never did') |
| + message: 'Expected sum() to sometimes return <0> but: never did'), |
| + buildStatusString(0, 1, 0, tests[16], |
| + message: 'Caught Exception: No behavior specified for method bar'), |
| ]; |
| actual = []; |