Print out experimental test.py reproduction command

tools/testing/dart/test_options.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.
 
 library test_options_parser;
 
 import "dart:io";
 import "dart:math";
 import "drt_updater.dart";
 import "test_suite.dart";
@@ skipping 444 matching lines @@
 
     // Apply default values for unspecified options.
     for (var option in _options) {
       if (!configuration.containsKey(option.name)) {
         configuration[option.name] = option.defaultValue;
       }
     }
 
     List<Map> expandedConfigs = _expandConfigurations(configuration);
     List<Map> result = expandedConfigs.where(_isValidConfig).toList();
+    for (var config in result) {
+     config['_reproducing_arguments_'] =

[ricow1, 2013/06/20 13:54:50]

why are you prefixing and postfixing it with _

[kustermann, 2013/06/20 14:17:24]

Because all values we put in the configuration object are visible to status
files (i.g. '$runtime').

To mark values which we use internally (and status files should not use), I
prefix/postfix with "_" (conf['_servers_'] is another example where we hold a
list of HttpServer instances [see test.dart]).

+         _constructReproducingCommandArguments(config);
+    }
     return result.isEmpty ? null : result;
   }
 
+  // For printing out reproducing command lines, we don't want to add these
+  // options.
+  Set<String>  _blacklistedOptions = new Set<String>.from([
+    'progress', 'failure-summary', 'step_name', 'report', 'tasks', 'verbose',
+    'time', 'dart', 'drt', 'dartium', 'build_directory', 'append_logs',
+    'write_debug_log', 'local_ip',

[ricow1, 2013/06/20 13:54:50]

maybe add shards and shard?

[kustermann, 2013/06/20 14:17:24]

Good idea. Done.

+  ]);
+
+  List<String> _constructReproducingCommandArguments(Map config) {
+    var arguments = new List<String>();
+    for (var configKey in config.keys) {
+      if (!_blacklistedOptions.contains(configKey)) {
+        for (var option in _options) {
+          var configValue = config[configKey];
+          // We only include entries of [conf] if we find an option for it.
+          if (configKey == option.name && configValue != option.defaultValue) {
+            var isBooleanOption = option.type == 'bool';
+            // Sort by length, so we get the shortest variant.
+            var possibleOptions = new List.from(option.keys);
+            possibleOptions.sort((a, b) => (a.length < b.length ? -1 : 1));
+            var key = possibleOptions[0];
+            if (key.startsWith('--')) {
+              // long version
+              arguments.add(key);
+              if (!isBooleanOption) {
+                arguments.add("$configValue");
+              }
+            } else {
+              // short version
+              assert(key.startsWith('-'));
+              if (!isBooleanOption) {
+                arguments.add("$key$configValue");
+              } else {
+                arguments.add(key);
+              }
+            }
+          }
+        }
+      }
+    }
+    return arguments;
+  }
+
   /**
    * Determine if a particular configuration has a valid combination of compiler
    * and runtime elements.
    */
   bool _isValidConfig(Map config) {
     bool isValid = true;
     List<String> validRuntimes;
     switch (config['compiler']) {
       case 'dart2js':
         // Note: by adding 'none' as a configuration, if the user
@@ skipping 261 matching lines @@
         return option;
       }
     }
     print('Unknown test option $name');
     exit(1);
   }
 
 
   List<_TestOptionSpecification> _options;
 }