Add --step_name option to test.dart.

dart/tools/test.dart

 #!/usr/bin/env 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.
 
 /**
  * This file is the entrypoint of the dart test suite.  This suite is used
  * to test:
  *
  *     1. the dart vm
@@ skipping 57 matching lines @@
   var startTime = new Date.now();
   var optionsParser = new TestOptionsParser();
   List<Map> configurations = optionsParser.parse(new Options().arguments);
   if (configurations == null || configurations.length == 0) return;
 
   // Extract global options from first configuration.
   var firstConf = configurations[0];
   Map<String, RegExp> selectors = firstConf['selectors'];
   var maxProcesses = firstConf['tasks'];
   var progressIndicator = firstConf['progress'];
+  var stepName = firstConf['step_name'];

[Bill Hesse, 2012/09/11 10:26:53]

How about BuildbotProgressIndicator.stepName = firstConf['step_name'].  Or a
setter.  Then it doesn't need to be passed as a parameter through two unrelated
class constructors.

   var verbose = firstConf['verbose'];
   var printTiming = firstConf['time'];
   var listTests = firstConf['list'];
 
+  if (progressIndicator == 'buildbot' && stepName !== null) {

[ricow1, 2012/09/11 08:14:38]

I am not particularly happy about having buildbot related stuff in the main
general purpose test script - even if it is only a print statement.

+    print('@@@BUILD_STEP $stepName@@@');
+  }
+
   // Print the configurations being run by this execution of
   // test.dart. However, don't do it if the silent progress indicator
   // is used. This is only needed because of the junit tests.
   if (progressIndicator != 'silent') {
     List output_words = configurations.length > 1 ?
         ['Test configurations:'] : ['Test configuration:'];
     for (Map conf in configurations) {
       List settings =
           ['compiler', 'runtime', 'mode', 'arch'].map((name) => conf[name]);
       if (conf['checked']) settings.add('checked');
@@ skipping 33 matching lines @@
     return true;
   }
 
   // Start process queue.
   var queue = new ProcessQueue(maxProcesses,
                                progressIndicator,
                                startTime,
                                printTiming,
                                enqueueConfiguration,
                                verbose,
-                               listTests);
+                               listTests,
+                               stepName);
 }