Add --checked or --minified to test directory name when generating output.

tools/testing/dart/test_suite.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 enumerating and preparing tests.
  *
  * This library includes:
  *
  * - Creating tests by listing all the Dart files in certain directories,
@@ skipping 1015 matching lines @@
   String createOutputDirectory(Path testPath, String optionsName) {
     Path relative = testPath.relativeTo(TestUtils.dartDir());
     relative = relative.directoryPath.append(relative.filenameWithoutExtension);
     String testUniqueName = relative.toString().replaceAll('/', '_');
     if (!optionsName.isEmpty) {
       testUniqueName = '$testUniqueName-$optionsName';
     }
 
     // Create '[build dir]/generated_tests/$compiler-$runtime/$testUniqueName',
     // including any intermediate directories that don't exist.
+    // If the tests are run in checked or minified mode we add that to the
+    // '$compile-$runtime' directory name.
+    var checked = configuration['checked'] ? '-checked' : '';
+    var minified = configuration['minified'] ? '-minified' : '';

[ngeoffray, 2012/11/20 10:32:28]

Is configuration the list of flags passed to the compiler? Should you use that
list then, instead of explicitly using some of the flags?

[ricow1, 2012/11/26 13:27:23]

No it is not, configuration is a ton of stuff

+    var dirName = "${configuration['compiler']}-${configuration['runtime']}"
+                  "$checked$minified";
     var generatedTestPath = Strings.join([
         buildDir,
         'generated_tests',
-        "${configuration['compiler']}-${configuration['runtime']}",
+        dirName,
         testUniqueName
     ], '/');
 
     TestUtils.mkdirRecursive(new Path('.'), new Path(generatedTestPath));
     return new File(generatedTestPath).fullPathSync().replaceAll('\\', '/');
   }
 
   String get scriptType {
     switch (configuration['compiler']) {
       case 'none':
@@ skipping 621 matching lines @@
  * $pass tests are expected to pass
  * $failOk tests are expected to fail that we won't fix
  * $fail tests are expected to fail that we should fix
  * $crash tests are expected to crash that we should fix
  * $timeout tests are allowed to timeout
  * $compileErrorSkip tests are skipped on browsers due to compile-time error
 """;
     print(report);
    }
 }