Add DirectoryTestSuite.

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,
  *   and creating [TestCase]s for those files that meet the relevant criteria.
  * - Preparing tests, including copying files and frameworks to temporary
  *   directories, and computing the command line and arguments to be run.
  */
 #library("test_suite");
 
 #import("dart:io");
 #import("dart:builtin");
 #import("dart:isolate");
 #import("status_file_parser.dart");
 #import("test_runner.dart");
 #import("multitest.dart");
 #import("drt_updater.dart");
 
 #source("browser_test.dart");
 
 
+// TODO(rnystrom): Add to dart:core?
+/**
+ * A simple function that tests [arg] and returns `true` or `false`.
+ */
+typedef bool Predicate<T>(T arg);
+
+
 /**
  * A TestSuite represents a collection of tests.  It creates a [TestCase]
  * object for each test to be run, and passes the test cases to a callback.
  *
  * Most TestSuites represent a directory or directory tree containing tests,
  * and a status file containing the expected results when these tests are run.
  */
 interface TestSuite {
   /**
    * Call the callback function onTest with a [TestCase] argument for each
@@ skipping 141 matching lines @@
   bool isNegativeIfChecked;
   bool hasFatalTypeErrors;
   bool hasRuntimeErrors;
   Set<String> multitestOutcome;
 
   TestInformation(this.filename, this.optionsFromFile, this.isNegative,
                   this.isNegativeIfChecked, this.hasFatalTypeErrors,
                   this.hasRuntimeErrors, this.multitestOutcome);
 }
 
+
 /**
  * A standard [TestSuite] implementation that searches for tests in a
  * directory, and creates [TestCase]s that compile and/or run them.
  */
 class StandardTestSuite implements TestSuite {
   Map configuration;
   String suiteName;
   String directoryPath;
   List<String> statusFilePaths;
   Function doTest;
   Function doDone;
   int activeTestGenerators = 0;
   bool listingDone = false;
   TestExpectations testExpectations;
   List<TestInformation> cachedTests;
   final String dartDir;
   Function globalTemporaryDirectory;
+  Predicate<String> isTestFilePredicate;
 
   StandardTestSuite(Map this.configuration,
                     String this.suiteName,
                     String this.directoryPath,
-                    List<String> this.statusFilePaths)
+                    List<String> this.statusFilePaths,
+                    [Predicate<String> this.isTestFilePredicate])
     : dartDir = TestUtils.dartDir();
 
   /**
+   * Creates a test suite whose file organization matches an expected structure.
+   * To use this, your suite should look like:
+   *
+   *     dart/
+   *       path/
+   *         to/
+   *           mytestsuite/
+   *             mytestsuite.status
+   *             example1_tests.dart
+   *             example2_tests.dart
+   *             example3_tests.dart
+   *
+   * The important parts:
+   *
+   * * The leaf directory name is the name of your test suite.
+   * * The status file uses the same name.
+   * * Test files are directly in that directory and end in "_tests.dart".
+   *
+   * If you follow that convention, then you can construct one of these like:
+   *
+   *     new DirectoryTestSuite(configuration, 'path/to/mytestsuite');
+   *
+   * instead of having to create a custom [StandardTestSuite] subclass. In
+   * particular, if you add 'path/to/mytestsuite' to [TEST_SUITE_DIRECTORIES]
+   * in test.dart, this will all be set up for you.
+   */
+  factory StandardTestSuite.forDirectory(
+      Map configuration, String directory) {
+    final name = directory.substring(directory.lastIndexOf('/') + 1);
+
+    return new StandardTestSuite(configuration,
+        name, directory, ['$directory/$name.status'],
+        (filename) => filename.endsWith('_tests.dart'));
+  }
+
+  /**
    * The default implementation assumes a file is a test if
    * it ends in "Test.dart".
    */
-  bool isTestFile(String filename) => filename.endsWith("Test.dart");
+  bool isTestFile(String filename) {
+    // Use the specified predicate, if provided.
+    if (isTestFilePredicate != null) return isTestFilePredicate(filename);
+
+    filename.endsWith("Test.dart");
+  }
 
   bool listRecursively() => false;
 
   String shellPath() => TestUtils.dartShellFileName(configuration);
 
   List<String> additionalOptions(String filename) => [];
 
   void forEachTest(Function onTest, Map testCache, String globalTempDir(),
                    [Function onDone = null]) {
     // If DumpRenderTree is required, and not yet updated, wait for update.
@@ skipping 1011 matching lines @@
  * $noCrash tests are expected to be flaky but not crash
  * $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
 """;
     print(report);
    }
 }