Base test case of a multi-test now insists no static type warnings

tools/testing/dart/multitest.dart

 // Copyright (c) 2011, 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.
 
 #library("multitest");
 
 #import("dart:io");
 #import("test_suite.dart");
 
 // Multitests are Dart test scripts containing lines of the form
@@ skipping 69 matching lines @@
 
   List<String> testTemplate = new List<String>();
   testTemplate.add('// Test created from multitest named $filename.');
   // Create the set of multitests, which will have a new test added each
   // time we see a multitest line with a new key.
   Map<String, List<String>> testsAsLines = new Map<String, List<String>>();
 
   int lineCount = 0;
   for (String line in lines) {
     lineCount++;
-    if (line.contains('///')) {
-      var parts = line.split('///')[1].split(':');
-      var key = parts[0].trim();
-      var rest = parts[1].trim();
-      if (testsAsLines.containsKey(key)) {
-        Expect.equals('continued', rest);
-        testsAsLines[key].add(line);
+    var annotation = new _Annotation.from(line);
+    if (annotation != null) {
+      testsAsLines.putIfAbsent(annotation.key,
+          () => new List<String>.from(testTemplate)).add(line);
+      outcomes.putIfAbsent(annotation.key, 
+          () => new Set<String>());
+      if (annotation.rest == 'continued') {
+        continue;
       } else {
-        (testsAsLines[key] = new List<String>.from(testTemplate)).add(line);
-        List<String> outcomesList = rest.split(',');
-        for (String nextOutcome in outcomesList) {
+        for (String nextOutcome in annotation.outcomesList) {
           nextOutcome = nextOutcome.trim();
-          outcomes.putIfAbsent(key, () => new Set<String>()).add(nextOutcome);  
+          outcomes[annotation.key].add(nextOutcome);  
           if (!validMultitestOutcomes.contains(nextOutcome)) {
             Expect.fail(
-              "Invalid test directive '$nextOutcome' on line ${lineCount}: $rest ");
+              "Invalid test directive '$nextOutcome' on line ${lineCount}: ${annotation.rest} ");

[Bill Hesse, 2012/03/12 16:02:49]

Line too long?
I think they have put concatenation of two adjacent strings in - is it in the
VM?

[zundel, 2012/03/12 20:06:23]

Looks like we need to update the dart executable tools/testing/vm/<os> to get
it.

           }
         }
       }
     } else {
       testTemplate.add(line);
       for (var test in testsAsLines.getValues()) test.add(line);
     }
   }
+
+  // Check that every key (other than the none case) has at least one outcome
+  for (var outcomeKey in outcomes.getKeys()) {
+    if (outcomeKey == "none") {
+      continue;
+    }
+    if (outcomes[outcomeKey].isEmpty()) {

[Bill Hesse, 2012/03/12 16:02:49]

Why not: if outcomeKey != 'none' && outcomes[outcomeKey].isEmpty()

[zundel, 2012/03/12 20:06:23]

Done.

+      Expect.fail("Test ${outcomeKey} has no valid annotated outcomes. Expected one of: ${validMultitestOutcomes.toString()}");

[Bill Hesse, 2012/03/12 16:02:49]

Long line.

+    }
+  }
+
   // Add the template, with no multitest lines, as a test with key 'none'.
   testsAsLines['none'] = testTemplate;
   outcomes['none'] = new Set<String>();
 
   // Copy all the tests into the output map tests, as multiline strings.
   for (String key in testsAsLines.getKeys()) {
     tests[key] =
         Strings.join(testsAsLines[key], line_separator) + line_separator;
   }
 }
 
+// Represents a mutlitest annotation in the special /// comment.
+class _Annotation {
+  String key;
+  String rest;
+  List<String> outcomesList;
+  _Annotation() {}
+  factory _Annotation.from(String line) {
+    if (!line.contains('///')) {
+      return null;
+    }
+    var annotation = new _Annotation();
+    var parts = line.split('///')[1].split(':');
+    annotation.key = parts[0].trim();
+    annotation.rest = parts[1].trim();
+    annotation.outcomesList = annotation.rest.split(',');

[Bill Hesse, 2012/03/12 16:02:49]

annotation.outcomesList = annotation.rest.split(',').map((e)=>e.trim());

or maybe map(String.trim)?  I don't know if that works.  probably not.

Or a local function: Function trim(String e) => e.trim();
parts = line.split .... .map(trim)
outcomesList = .... .split(',').map(trim);

+    return annotation;
+  }
+}
+
 // Find all relative imports and copy them into the dir that contains
 // the generated tests.
 Set<String> _findAllRelativeImports(String topLibrary) {
   Set<String> toSearch = new Set<String>.from([topLibrary]);
   Set<String> foundImports = new HashSet<String>();
   String pathSep = new Platform().pathSeparator();
   int end = topLibrary.lastIndexOf(pathSep);
   String libraryDir = topLibrary.substring(0, end);
 
   // Matches #import( or #source( followed by " or ' followed by anything
@@ skipping 102 matching lines @@
   var split = testDir.split('/');
   var lastComponent = split.removeLast();
   Expect.isTrue(lastComponent == 'src');
   String path = '${generatedTestDir.path}/${split.last()}';
   Directory dir = new Directory(path);
   if (!dir.existsSync()) {
     dir.createSync();
   }
   return path;
 }