test rename overhaul: step 14 - vm tests

tools/testing/dart/test_runner.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.
 
 /**
  * Classes and methods for executing tests.
  *
  * This module includes:
  * - Managing parallel execution of tests, including timeout checks.
  * - Evaluating the output of each test as pass/fail/crash/timeout.
@@ skipping 60 matching lines @@
   TestInformation info;
 
   TestCase(this.displayName,
            this.commands,
            this.configuration,
            this.completedHandler,
            this.expectedOutcomes,
            [this.isNegative = false,
             this.info = null]) {
     if (!isNegative) {
-      // TODO(sigmund): dissallow NegativeTest once the test rename overhaul is
-      // done.
+      // TODO(sigmund): use only 'negative_test' (and not 'NegativeTest') once
+      // the test rename overhaul is done.
       this.isNegative = displayName.contains("NegativeTest")
           || displayName.contains("negative_test");
     }
 
     // Special command handling. If a special command is specified
     // we have to completely rewrite the command that we are using.
     // We generate a new command-line that is the special command
     // where we replace '@' with the original command.
     var specialCommand = configuration['special-command'];
     if (!specialCommand.isEmpty()) {
@@ skipping 1069 matching lines @@
         // the developer doesn't waste his or her time trying to fix a bunch of
         // tests that appear to be broken but were actually just flakes that
         // didn't get retried because there had already been one failure.
         bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests;
         new RunningProcess(test, allowRetry, this).start();
       }
       _numProcesses++;
     }
   }
 }