Add DirectoryTestSuite.

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index da9cf11dc32fed1174b597f7f898487c59593f6a..e9d77a343233399b55d6c7861607b383db119c6e 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -186,6 +186,7 @@ class TestInformation {
                   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.
@@ -925,6 +926,52 @@ class DartcCompilationTestSuite extends StandardTestSuite {
 }
 
 
+/**
+ * A standard 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.
+ */
+class DirectoryTestSuite extends StandardTestSuite {

[Bill Hesse, 2012/03/28 23:37:30]

I really think we can merge the classes:
Make isTestFile call a closure passed to the constructor.
Default closure for StandardTestSuite is (name)=>name.endsWith("Test.dart");

Then all of DirectoryTestSuite can just be a new constructor for
StandardTestSuite.

This may also allow other subclasses, which only override isTestFile, to be
removed, and to be replaced with instances of StandardTestSuite.

[Bob Nystrom, 2012/03/29 00:10:17]

Excellent idea. Done!

+  factory DirectoryTestSuite(Map configuration, String directory) {
+    final name = directory.substring(directory.lastIndexOf('/') + 1);
+    print(name);

[Bill Hesse, 2012/03/28 23:37:30]

Remove the print statement.

[Bob Nystrom, 2012/03/29 00:10:17]

Done.

+
+    return new DirectoryTestSuite._internal(configuration,
+      name, directory, ['$directory/$name.status']);
+  }
+
+  DirectoryTestSuite._internal(Map configuration,
+                               String suiteName,
+                               String directoryPath,
+                               List<String> statusFilePaths)
+    : super(configuration, suiteName, directoryPath, statusFilePaths);
+
+  bool isTestFile(String filename) => filename.endsWith('_tests.dart');
+}
+
+
 class JUnitTestSuite implements TestSuite {
   Map configuration;
   String suiteName;