Clean up UnitTestSuite a bit.

client/testing/unittest/unittestsuite.dart

 // Copyright (c) 2011, 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(rnystrom): This code is gradually moving from a Java/JUnit style to
 // something closer to JS/Jasmine. Eventually, the UnitTestSuite class can go
 // away completely (or become private to this library) and the only exposed API
 // will be group()/test()/expect(). Until then, both ways are supported, which
 // is why things look a bit weird in here.
 
@@ skipping 71 matching lines @@
     } catch(var e) {
     // TODO(jacobr): remove this horrible hack to work around dartc bugs.
       window.dynamic.addEventListener("DOMContentLoaded", listener, false);
     }
   }
 
   /** Subclasses should override this method to register tests. */
   void setUpTestSuite() {}
 
   /** Enqueues a synchronous test. */
-  UnitTestSuite addTest(TestFunction body) {
+  void addTest(TestFunction body) {
     test(null, body);
   }
 
   /** Adds the tests defined by the given TestSet to this suite. */
   void addTestSet(TestSet test) {
     test._bindToSuite(this);
     test.setup();
   }
 
   /** Adds the tests defined by the given TestSets to this suite. */
@@ skipping 180 matching lines @@
 
 /**
  * Creates a new async test case with the given description and body. The
  * description will include the descriptions of any surrounding group()
  * calls.
  */
 void asyncTest(String spec, int callbacks, TestFunction body) {
   _ensureActiveSuite();
 
   final testCase = new TestCase(
-    _currentSuite._tests.length + 1, _fullSpec(spec), body, callbacks);
+      _currentSuite._tests.length + 1, _fullSpec(spec), body, callbacks);
   _currentSuite._tests.add(testCase);
 
   if (callbacks < 1) {
     testCase.recordError(
         'Async tests must wait for at least one callback ', '');
   }
 }
 
 /**
  * Creates a new named group of tests. Calls to group() or test() within the
@@ skipping 36 matching lines @@
   if (spec === null) return '$_currentGroup';
   return _currentGroup != '' ? '$_currentGroup $spec' : spec;
 }
 
 /**
  * Lazily creates a UnitTestSuite if there isn't already an active one. Returns
  * whether or not one was created.
  */
 _ensureActiveSuite() {
   if (_currentSuite != null) {
-    return false;
+    return;
   }
 
   _currentSuite = new UnitTestSuite();
-  return true;
 }
 
 /**
  * Wraps an value and provides an "==" operator that can be used to verify that
  * the value matches a given expectation.
  */
 class Expectation {
   final _value;
 
   Expectation(this._value);
@@ skipping 148 matching lines @@
         </tr>""";
     if (stackTrace != null) {
       message +=
           "<tr><td></td><td colspan='2'><pre>${stackTrace}</pre></td></tr>";
     }
     fail = true;
   }
 }
 
 typedef void TestFunction();