Enable frogium component in test.dart.

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 e47c05f82596981f7f7ebb9568e52f7e72bd445c..4cda8542140ed2aef271ed196e62e9a80d2e40cb 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -255,37 +255,38 @@ class StandardTestSuite implements TestSuite {
     }
     if (expectations.contains(SKIP)) return;
 
-    if (configuration['component'] == 'dartium') {
-      enqueueDartiumTest(filename, testName, optionsFromFile,
-                         expectations, isNegative);
-      return;
-    }
-    if (configuration['component'] == 'chromium') {
-      enqueueChromiumTest(filename, testName, optionsFromFile,
-                         expectations, isNegative);
-      return;
-    }
-
-    // Only dartc supports fatal type errors. Enable fatal type
-    // errors with a flag and treat tests that have fatal type
-    // errors as negative.
-    var enableFatalTypeErrors =
-        (info.hasFatalTypeErrors && configuration['component'] == 'dartc');
-    var argumentLists = argumentListsFromFile(filename,
-                                              optionsFromFile,
-                                              enableFatalTypeErrors);
-    isNegative = isNegative ||
-        (configuration['checked'] && info.isNegativeIfChecked) ||
-        enableFatalTypeErrors;
-
-    for (var args in argumentLists) {
-      doTest(new TestCase('$suiteName/$testName',
-                          shellPath(),
-                          args,
-                          configuration,
-                          completeHandler,
-                          expectations,
-                          isNegative));
+    switch (configuration['component']) {
+      case 'dartium':
+        enqueueDartiumTest(filename, testName, optionsFromFile,

[Bill Hesse, 2011/12/22 13:26:02]

I plan to merge Dartium and Chromium/Frogium in the next change.

+                           expectations, isNegative);
+        break;
+      case 'chromium':
+      case 'frogium':
+        enqueueChromiumTest(filename, testName, optionsFromFile,
+                              expectations, isNegative);

[Mads Ager (google), 2011/12/22 15:22:59]

indentation.

+        break;
+      default:
+        // Only dartc supports fatal type errors. Enable fatal type
+        // errors with a flag and treat tests that have fatal type
+        // errors as negative.
+        var enableFatalTypeErrors =
+            (info.hasFatalTypeErrors && configuration['component'] == 'dartc');
+        var argumentLists = argumentListsFromFile(filename,
+                                                  optionsFromFile,
+                                                  enableFatalTypeErrors);
+        isNegative = isNegative ||
+            (configuration['checked'] && info.isNegativeIfChecked) ||
+            enableFatalTypeErrors;
+
+        for (var args in argumentLists) {
+          doTest(new TestCase('$suiteName/$testName',
+                              shellPath(),
+                              args,
+                              configuration,
+                              completeHandler,
+                              expectations,
+                              isNegative));
+        }
     }
   }
 
@@ -454,6 +455,11 @@ class StandardTestSuite implements TestSuite {
 
     String dartWrapperFilename = '${tempDir.path}/test.dart';
     String compiledDartWrapperFilename = '${tempDir.path}/test.js';
+    String domInWrapper = 'dart:dom';

[Mads Ager (google), 2011/12/22 15:22:59]

How about domLibraryImport? I think the name should be a bit more specific.

+    if (configuration['component'] == 'chromium') {
+      domInWrapper = '$dartDir/client/testing/unittest/dom_for_unittest.dart';
+    }
+
     if (!isWebTest) {
       // test.dart will import the dart test directly, if it is a library,
       // or indirectly through test_as_library.dart, if it is not.
@@ -471,7 +477,7 @@ class StandardTestSuite implements TestSuite {
       File file = new File(dartWrapperFilename);
       RandomAccessFile dartWrapper = file.openSync(writable: true);
       dartWrapper.writeStringSync(DartTestWrapper(
-          '$dartDir/client/testing/unittest/dom_for_unittest.dart',
+          domInWrapper,
           '$dartDir/tests/isolate/src/TestFramework.dart',
           dartLibraryFilename));
       dartWrapper.closeSync();
@@ -489,24 +495,40 @@ class StandardTestSuite implements TestSuite {
     htmlTest.closeSync();
 
     for (var vmOptions in optionsFromFile["vmOptions"]) {
-      List<String> dartcArgs = ['--work', tempDir.path];
-      if (configuration['mode'] ==  'release') {
-        dartcArgs.add('--optimize');
+      List<String> compilerArgs;
+      String compilerExecutable = TestUtils.compilerPath(configuration);
+      switch (configuration['component']) {
+        case 'chromium':
+          compilerArgs = ['--work', tempDir.path];
+          if (configuration['mode'] ==  'release') {
+            compilerArgs.add('--optimize');
+          }
+          compilerArgs.addAll(vmOptions);
+          compilerArgs.add('--ignore-unrecognized-flags');
+          compilerArgs.add('--out');
+          compilerArgs.add(compiledDartWrapperFilename);
+          compilerArgs.add(dartWrapperFilename);
+          // TODO(whesse): Add --fatal-type-errors if needed.
+          break;
+        case 'frogium':
+          compilerArgs = ['--libdir=$dartDir/frog/lib',
+                          '--compile-only',
+                          '--out=$compiledDartWrapperFilename'];
+          compilerArgs.addAll(vmOptions);
+          compilerArgs.add(dartWrapperFilename);
+          break;
+        default:
+          Expect.fail('unimplemented component ${configuration['component']}');
       }
-      dartcArgs.addAll(vmOptions);
-      dartcArgs.add('--ignore-unrecognized-flags');
-      dartcArgs.add('--out');
-      dartcArgs.add(compiledDartWrapperFilename);
-      dartcArgs.add(dartWrapperFilename);
-      // TODO(whesse): Add --fatal-type-errors if needed.
+
       var args = ['--no-timeout'];
       args.add(htmlTestBase.fullPathSync());
 
       // Create BrowserTestCase and queue it.
       var testCase = new BrowserTestCase(
           testName,
-          TestUtils.dartcCompilationShellPath(configuration),
-          dartcArgs,
+          compilerExecutable,
+          compilerArgs,
           dumpRenderTreeFilename,
           args,
           configuration,
@@ -522,6 +544,7 @@ class StandardTestSuite implements TestSuite {
       case 'dartium':
         return 'application/dart';
       case 'chromium':
+      case 'frogium':
         return 'text/javascript';
       default:
         Expect.fail('Unimplemented component scriptType');
@@ -534,6 +557,7 @@ class StandardTestSuite implements TestSuite {
       case 'dartium':
         return tempDir.path + 'test.dart';
       case 'chromium':
+      case 'frogium':
         return tempDir.path + 'test.js';
       default:
         Expect.fail('Unimplemented component scriptType');
@@ -546,6 +570,7 @@ class StandardTestSuite implements TestSuite {
       case 'dartium':
         return filename.replaceAll(pathSeparator, '_') + 'dartium.html';
       case 'chromium':
+      case 'frogium':
         return 'test.html';
       default:
         Expect.fail('Unimplemented component scriptType');
@@ -719,7 +744,7 @@ class DartcCompilationTestSuite extends StandardTestSuite {
     }
   }
 
-  String shellPath() => TestUtils.dartcCompilationShellPath(configuration);
+  String shellPath() => TestUtils.compilerPath(configuration);
 
   List<String> additionalOptions() {
     // TODO(ager): potentially register cleanup action to delete the temporary
@@ -770,6 +795,19 @@ class TestUtils {
     }
   }
 
+  static String compilerName(Map configuration) {
+    String postfix =
+        (new Platform().operatingSystem() == 'windows') ? '.exe' : '';
+    switch (configuration['component']) {
+      case 'chromium':

[Bill Hesse, 2011/12/22 13:26:02]

Add case 'dartc' here.

+        return 'compiler/bin/dartc$postfix';
+      case 'frogium':
+        return 'frog/bin/frogsh$postfix';
+      default:
+        throw "Unknown compiler for: ${configuration['component']}";
+    }
+  }
+
   static String dartShellFileName(Map configuration) {
     var name = '${buildDir(configuration)}/${executableName(configuration)}';
     if (!(new File(name)).existsSync()) {
@@ -778,8 +816,8 @@ class TestUtils {
     return name;
   }
 
-  static String dartcCompilationShellPath(Map configuration) {
-    var name = '${buildDir(configuration)}/compiler/bin/dartc';
+  static String compilerPath(Map configuration) {
+    var name = '${buildDir(configuration)}/${compilerName(configuration)}';
     if (!(new File(name)).existsSync()) {
       throw "Executable '$name' does not exist";
     }