Added ability to disable logging in mocks, to avoid the memory overhead if you don't need behavior …

tests/lib/unittest/unittest_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // TODO(gram):
 // Unfortunately I can't seem to test anything that involves timeouts, e.g.
 // insufficient callbacks, because the timeout is controlled externally
 // (test.dart?), and we would need to use a shorter timeout for the inner tests
 // so the outer timeout doesn't fire. So I removed all such tests.
 // I'd like to revisit this at some point.
@@ skipping 77 matching lines @@
 }
 
 runTest() {
   port.receive((testName, sendport) {
     configure(_testconfig = new TestConfiguration(sendport));
     if (testName == 'single correct test') {
       test(testName, () => expect(2 + 3, equals(5)));
     } else if (testName == 'single failing test') {
       test(testName, () => expect(2 + 2, equals(5)));
     } else if (testName == 'exception test') {
-      test(testName, () { throw new Exception('fail'); });
+      test(testName, () { throw new Exception('Fail.'); });
     } else if (testName == 'group name test') {
       group('a', () {
         test('a', () {});
         group('b', () {
           test('b', () {});
         });
       });
     } else if (testName == 'setup test') {
       group('a', () {
         setUp(() { _testconfig.setup = 'setup'; });
@@ skipping 142 matching lines @@
         m.getLogs(callsTo(null, 1)).verify(happenedExactly(2));
         m.getLogs(callsTo(null, 2)).verify(happenedExactly(2));
         m.getLogs(null, returning(1)).verify(neverHappened);
         m.getLogs(null, returning(2)).verify(happenedExactly(2));
         m.getLogs(null, returning(4)).verify(happenedExactly(2));
       });
     } else if (testName.startsWith('mock test 10 ')) {
       test(testName, () {
         var m = new Mock();
         m.when(callsTo(matches('^[A-Z]'))).
-            alwaysThrow('Method names must start with lower case');
+            alwaysThrow('Method names must start with lower case.');
         m.test();
       });
     } else if (testName.startsWith('mock test 11 ')) {
       test(testName, () {
         var m = new Mock();
         m.when(callsTo(matches('^[A-Z]'))).
-            alwaysThrow('Method names must start with lower case');
+            alwaysThrow('Method names must start with lower case.');
         m.Test();
       });
+    } else if (testName.startsWith('mock test 12 ')) {
+      test(testName, () {
+        var m = new Mock.custom(enableLogging:false);
+        m.Test();
+        print(m.getLogs(callsTo('Test')).toString());
+      });
     }
   });
 }
 
 void nextTest(int testNum) {
   SendPort sport = spawnFunction(runTest);
   sport.call(tests[testNum]).then((msg) {
     actual.add(msg);
     if (actual.length == expected.length) {
       for (var i = 0; i < tests.length; i++) {
-        test(tests[i], () => expect(actual[i], equals(expected[i])));
+        test(tests[i], () => expect(actual[i].trim(), equals(expected[i])));
       }
     } else {
       nextTest(testNum+1);
     }
   });
 }
 
 main() {
   tests = [
     'single correct test',
     'single failing test',
     'exception test',
     'group name test',
     'setup test',
     'teardown test',
     'setup and teardown test',
     'correct callback test',
     'excess callback test',
     'completion test',
     'mock test 1 (Mock)',
     'mock test 2 (MockList)',
     'mock test 3 (Spy)',
     'mock test 4 (Excess calls)',
     'mock test 5 (No action)',
     'mock test 6 (No matching return)',
     'mock test 7 (No behavior)',
     'mock test 8 (Shared log)',
     'mock test 9 (Null CallMatcher)',
     'mock test 10 (RegExp CallMatcher Good)',
-    'mock test 11 (RegExp CallMatcher Bad)'
+    'mock test 11 (RegExp CallMatcher Bad)',
+    'mock test 12 (No logging)'
   ];
 
   expected = [
     buildStatusString(1, 0, 0, tests[0]),
-    buildStatusString(0, 1, 0, tests[1], message: 'Expected: <5> but: was <4>'),
-    buildStatusString(0, 1, 0, tests[2], message: 'Caught Exception: fail'),
+    buildStatusString(0, 1, 0, tests[1],
+        message: 'Expected: <5> but: was <4>.'),
+    buildStatusString(0, 1, 0, tests[2], message: 'Caught Exception: Fail.'),
     buildStatusString(2, 0, 0, 'a a::a b b'),
     buildStatusString(1, 0, 0, 'a ${tests[4]}', 0, 'setup'),
     buildStatusString(1, 0, 0, 'a ${tests[5]}', 0, '', 'teardown'),
     buildStatusString(1, 0, 0, 'a ${tests[6]}', 0,
         'setup', 'teardown'),
     buildStatusString(1, 0, 0, tests[7], 1),
     buildStatusString(0, 0, 1, tests[8], 1,
-        message: 'Callback called more times than expected (2 > 1)'),
+        message: 'Callback called more times than expected (2 > 1).'),
     buildStatusString(1, 0, 0, tests[9], 10),
     buildStatusString(1, 0, 0, tests[10]),
     buildStatusString(1, 0, 0, tests[11]),
     buildStatusString(1, 0, 0, tests[12]),
     buildStatusString(0, 1, 0, tests[13],
         message: "Expected <null>.'foo'() to be called 1 times but:"
-            " was called 2 times"),
+            " was called 2 times."),
     buildStatusString(0, 1, 0, tests[14],
-        message: 'Caught Exception: No more actions for method foo'),
+        message: 'Caught Exception: No more actions for method foo.'),
     buildStatusString(0, 1, 0, tests[15], message:
-        "Expected <null>.'sum'() to sometimes return <0> but: never did"),
+        "Expected <null>.'sum'() to sometimes return <0> but: never did."),
     buildStatusString(0, 1, 0, tests[16],
-        message: 'Caught Exception: No behavior specified for method bar'),
+        message: 'Caught Exception: No behavior specified for method bar.'),
     buildStatusString(1, 0, 0, tests[17]),
     buildStatusString(1, 0, 0, tests[18]),
     buildStatusString(1, 0, 0, tests[19]),
     buildStatusString(0, 1, 0, tests[20],
-        message:'Caught Method names must start with lower case')
+        message:'Caught Method names must start with lower case.'),
+    buildStatusString(0, 1, 0, tests[21], message:
+      "Caught Exception: Can't retrieve logs when logging was never enabled."),
   ];
 
   actual = [];
 
   nextTest(0);
 }