pkg/unittest: more lock-down, cleanup

pkg/unittest/lib/src/test_case.dart

 // Copyright (c) 2013, 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.
 
 part of unittest;
 
 /**
  * Represents the state for an individual unit test.
  *
  * Create by calling [test] or [solo_test].
@@ skipping 39 matching lines @@
 
   /** The group (or groups) under which this test is running. */
   final String currentGroup;
 
   DateTime _startTime;
   DateTime get startTime => _startTime;
 
   Duration _runningTime;
   Duration get runningTime => _runningTime;
 
-  bool enabled = true;
+  bool _enabled = true;
+
+  bool get enabled => _enabled;
 
   bool _doneTeardown = false;
 
   Completer _testComplete;
 
   TestCase._internal(this.id, this.description, this._testFunction)
       : currentGroup = _currentContext.fullName,
         _setUp = _currentContext.testSetup,
         _tearDown = _currentContext.testTeardown;
 
   bool get isComplete => !enabled || result != null;
 
   Function _errorHandler(String stage) => (e, stack) {
     if (stack == null && e is Error) {
       stack = e.stackTrace;
     }
     if (result == null || result == PASS) {
       if (e is TestFailure) {
-        fail("$e", stack);
+        _fail("$e", stack);
       } else {
-        error("$stage failed: Caught $e", stack);
+        _error("$stage failed: Caught $e", stack);
       }
     }
   };
 
   /**
    * Perform any associated [_setUp] function and run the test. Returns
    * a [Future] that can be used to schedule the next test. If the test runs
    * to completion synchronously, or is disabled, null is returned, to
    * tell unittest to schedule the next test immediately.
    */
@@ skipping 61 matching lines @@
       }
     }
     _setResult(testResult, messageText, stack);
     if (_testComplete != null) {
       var t = _testComplete;
       _testComplete = null;
       t.complete(this);
     }
   }
 
-  void pass() {
+  void _pass() {
     _complete(PASS);
   }
 
-  void fail(String messageText, [StackTrace stack]) {
+  void _fail(String messageText, [StackTrace stack]) {
     if (result != null) {
       String newMessage = (result == PASS)
           ? 'Test failed after initially passing: $messageText'
           : 'Test failed more than once: $messageText';
       // TODO(gram): Should we combine the stack with the old one?
       _complete(ERROR, newMessage, stack);
     } else {
       _complete(FAIL, messageText, stack);
     }
   }
 
-  void error(String messageText, [StackTrace stack]) {
+  void _error(String messageText, [StackTrace stack]) {
     _complete(ERROR, messageText, stack);
   }
 
   void _markCallbackComplete() {
     if (--_callbackFunctionsOutstanding == 0 && !isComplete) {
-      pass();
+      _pass();
     }
   }
 
   String toString() => _result != null ? "$description: $result" : description;
 }