test.py: Run the analyzer on all dart files in the sdk.

dart/tools/testing/dart/test_runner.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 executing tests.
  *
  * This module includes:
  * - Managing parallel execution of tests, including timeout checks.
  * - Evaluating the output of each test as pass/fail/crash/timeout.
@@ skipping 321 matching lines @@
         other is SeleniumTestCommand &&
         super._equal(other) &&
         browser == other.browser &&
         url == other.url;
   }
 }
 
 class AnalysisCommand extends Command {
   final String flavor;
 
+  // If [fileFilter] is given, only errors/warnings reported by the analyzer
+  // for which [fileFilter] returns [:true:] are considered.
+  final Function fileFilter;
+
   AnalysisCommand._(this.flavor,
                     String displayName,
                     String executable,
                     List<String> arguments,
-                    String configurationDir)
+                    String configurationDir,
+                    this.fileFilter)
       : super._(displayName, executable, arguments, configurationDir);
 
   void _buildHashCode(HashCodeBuilder builder) {
     super._buildHashCode(builder);
     builder.add(flavor);
+    builder.add(fileFilter);
   }
 
   bool _equal(Command other) {
     return
         other is AnalysisCommand &&
         super._equal(other) &&
-        flavor == other.flavor;
+        flavor == other.flavor &&
+        fileFilter == other.fileFilter;
   }
 }
 
 class VmCommand extends Command {
   VmCommand._(String executable,
               List<String> arguments,
               String configurationDir)
       : super._("vm", executable, arguments, configurationDir);
 }
 
@@ skipping 58 matching lines @@
                                            String configurationDir) {
     var command =
         new CompilationCommand._(displayName, outputFile, neverSkipCompilation,
                                  bootstrapDependencies, executable, arguments,
                                  configurationDir);
     return _getUniqueCommand(command);
   }
 
   AnalysisCommand getAnalysisCommand(
       String displayName, executable, arguments, String configurationDir,
-      {String flavor: 'dartanalyzer'}) {
+      {String flavor: 'dartanalyzer', Function fileFilter: null}) {
     var command = new AnalysisCommand._(
-        flavor, displayName, executable, arguments, configurationDir);
+        flavor, displayName, executable, arguments, configurationDir,
+        fileFilter);
     return _getUniqueCommand(command);
   }
 
   VmCommand getVmCommand(String executable,
                          List<String> arguments,
                          String configurationDir) {
     var command = new VmCommand._(executable, arguments, configurationDir);
     return _getUniqueCommand(command);
   }
 
@@ skipping 460 matching lines @@
     return !output.contains("PASS");
   }
 }
 
 
 class AnalysisCommandOutputImpl extends CommandOutputImpl {
   // An error line has 8 fields that look like:
   // ERROR|COMPILER|MISSING_SOURCE|file:/tmp/t.dart|15|1|24|Missing source.
   final int ERROR_LEVEL = 0;
   final int ERROR_TYPE = 1;
+  final int FILENAME = 3;
   final int FORMATTED_ERROR = 7;
 
   AnalysisCommandOutputImpl(command,
                             exitCode,
                             timedOut,
                             stdout,
                             stderr,
                             time,
                             compilationSkipped) :
     super(command,
@@ skipping 40 matching lines @@
       return Expectation.STATIC_WARNING;
     }
 
     assert (errors.length == 0 && warnings.length == 0);
     assert (!testCase.info.hasCompileError &&
             !testCase.info.hasStaticWarning);
     return Expectation.PASS;
   }
 
   void parseAnalyzerOutput(List<String> outErrors, List<String> outWarnings) {
+    AnalysisCommand analysisCommand = command;
+    Function fileFilter = analysisCommand.fileFilter;
+    if (fileFilter == null) {
+      // If no filter function was given, we don't filter the output at all.
+      fileFilter = (arg) => true;
+    }
+
     // Parse a line delimited by the | character using \ as an escape charager
     // like:  FOO|BAR|FOO\|BAR|FOO\\BAZ as 4 fields: FOO BAR FOO|BAR FOO\BAZ
     List<String> splitMachineError(String line) {
       StringBuffer field = new StringBuffer();
       List<String> result = [];
       bool escaped = false;
       for (var i = 0 ; i < line.length; i++) {
         var c = line[i];
         if (!escaped && c == '\\') {
           escaped = true;
           continue;
         }
         escaped = false;
         if (c == '|') {
           result.add(field.toString());
           field = new StringBuffer();
           continue;
         }
         field.write(c);
       }
       result.add(field.toString());
       return result;
     }
 
     for (String line in decodeUtf8(super.stderr).split("\n")) {
       if (line.length == 0) continue;
       List<String> fields = splitMachineError(line);
-      if (fields[ERROR_LEVEL] == 'ERROR') {
-        outErrors.add(fields[FORMATTED_ERROR]);
-      } else if (fields[ERROR_LEVEL] == 'WARNING') {
-        outWarnings.add(fields[FORMATTED_ERROR]);
+      // We only consider errors/warnings for files of interest.
+      if (fields.length > FILENAME) {
+        if (fileFilter(fields[FILENAME])) {
+          if (fields[ERROR_LEVEL] == 'ERROR') {
+            outErrors.add(fields[FORMATTED_ERROR]);
+          } else if (fields[ERROR_LEVEL] == 'WARNING') {
+            outWarnings.add(fields[FORMATTED_ERROR]);
+          }
+        }
+        // OK to Skip error output that doesn't match the machine format
       }
-      // OK to Skip error output that doesn't match the machine format
     }
   }
 }
 
 class VmCommandOutputImpl extends CommandOutputImpl
                           with UnittestSuiteMessagesMixin {
   static const DART_VM_EXITCODE_COMPILE_TIME_ERROR = 254;
   static const DART_VM_EXITCODE_UNCAUGHT_EXCEPTION = 255;
 
   VmCommandOutputImpl(Command command, int exitCode, bool timedOut,
@@ skipping 1265 matching lines @@
     }
   }
 
   void eventAllTestsDone() {
     for (var listener in _eventListener) {
       listener.allDone();
     }
     _allDone();
   }
 }