Support @Skip as a suite-level annotation.

lib/src/runner/loader.dart

 // Copyright (c) 2015, 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 test.runner.loader;
 
 import 'dart:async';
 import 'dart:io';
 import 'dart:isolate';
 
@@ skipping 114 matching lines @@
     }
 
     var controller = new StreamController();
     Future.forEach(_platforms, (platform) {
       if (!metadata.testOn.evaluate(platform, os: currentOS)) {
         return new Future.value();
       }
 
       // Don't load a skipped suite.
       if (metadata.skip) {
-        return new Future.value(new Suite([
+        controller.add(new Suite([
           new LocalTest(path, metadata, () {})
         ], path: path, platform: platform.name, metadata: metadata));
+        return new Future.value();

[kevmoo, 2015/04/17 23:56:29]

you can just return here – forEach handles null fine

[nweiz, 2015/04/18 00:07:39]

Done. Future.wait makes me paranoid, because it doesn't.

       }
 
       return new Future.sync(() {
         if (_pubServeUrl != null && !p.isWithin('test', path)) {
           throw new LoadException(path,
               'When using "pub serve", all test files must be in test/.');
         }
 
         if (platform == TestPlatform.vm) return _loadVmFile(path, metadata);
         assert(platform.isBrowser);
@@ skipping 90 matching lines @@
   Future close() {
     for (var isolate in _isolates) {
       isolate.kill();
     }
     _isolates.clear();
 
     if (_browserServerCompleter == null) return new Future.value();
     return _browserServer.then((browserServer) => browserServer.close());
   }
 }