test rename overhaul: step 13 - samples

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 15 matching lines @@
 #library("test");
 
 #import("testing/dart/test_runner.dart");
 #import("testing/dart/test_options.dart");
 #import("testing/dart/test_suite.dart");
 
 #import("../client/tests/dartc/test_config.dart");
 #import("../compiler/tests/dartc/test_config.dart");
 #import("../frog/tests/await/test_config.dart");
 #import("../runtime/tests/vm/test_config.dart");
-#import("../samples/tests/samples/test_config.dart");
 #import("../tests/co19/test_config.dart");
 #import("../tests/standalone/test_config.dart");
 
 /**
  * The directories that contain test suites which follow the conventions
  * required by [StandardTestSuite]'s forDirectory constructor.
  * New test suites should follow this convention because it makes it much
  * simpler to add them to test.dart.  Existing test suites should be
  * moved to here, if possible.
 */
 final TEST_SUITE_DIRECTORIES = const [
   'frog/tests/frog',
   'frog/tests/frog_native',
   'frog/tests/leg',
   'frog/tests/leg_only',
+  'samples/tests/samples',

[Bob Nystrom, 2012/04/30 16:18:52]

Not for this patch, but I wonder if we should eventually move these tests to
just "tests/samples".

[Siggi Cherem (dart-lang), 2012/04/30 16:33:02]

yes - or samples/tests/

   'tests/benchmark_smoke',
   'tests/corelib',
   'tests/dom',
   'tests/html',
   'tests/isolate',
   'tests/json',
   'tests/language',
   'tests/utils',
   'utils/tests/css',
   'utils/tests/peg',
@@ skipping 31 matching lines @@
   }
 
   var configurationIterator = configurations.iterator();
   bool enqueueConfiguration(ProcessQueue queue) {
     if (!configurationIterator.hasNext()) {
       return false;
     }
 
     var conf = configurationIterator.next();
     for (String key in selectors.getKeys()) {
-      if (key == 'samples') {
-        queue.addTestSuite(new SamplesTestSuite(conf));
-      } else if (key == 'standalone') {
+      if (key == 'standalone') {
         queue.addTestSuite(new StandaloneTestSuite(conf));
       } else if (key == 'co19') {
         queue.addTestSuite(new Co19TestSuite(conf));
       } else if (conf['runtime'] == 'vm' && key == 'vm') {
         queue.addTestSuite(new VMTestSuite(conf));
         queue.addTestSuite(new VMDartTestSuite(conf));
       } else if (conf['compiler'] == 'dartc' && key == 'dartc') {
         queue.addTestSuite(new ClientDartcTestSuite(conf));
       } else if (conf['compiler'] == 'dartc' && key == 'dartc') {
         queue.addTestSuite(new JUnitDartcTestSuite(conf));
@@ skipping 16 matching lines @@
   // Start process queue.
   var queue = new ProcessQueue(maxProcesses,
                                progressIndicator,
                                startTime,
                                printTiming,
                                enqueueConfiguration,
                                verbose,
                                listTests,
                                keepGeneratedTests);
 }