Clean up the warning for unknown tags.

lib/src/runner/load_suite.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.load_suite;
 
 import 'dart:async';
 
 import 'package:stack_trace/stack_trace.dart';
 
@@ skipping 21 matching lines @@
 /// A suite is constructed with logic necessary to produce a test suite. As with
 /// a normal test body, this logic isn't run until [LiveTest.run] is called. The
 /// suite itself is returned by [suite] once it's avaialble, but any errors or
 /// prints will be emitted through the running [LiveTest].
 class LoadSuite extends RunnerSuite {
   /// A future that completes to the loaded suite once the suite's test has been
   /// run and completed successfully.
   ///
   /// This will return `null` if the suite is unavailable for some reason (for
   /// example if an error occurred while loading it).
-  final Future<RunnerSuite> suite;
+  Future<RunnerSuite> get suite async => (await _suiteAndZone)?.first;
+
+  /// A future that completes to a pair of [suite] and the load test's [Zone].
+  ///
+  /// This will return `null` if the suite is unavailable for some reason (for
+  /// example if an error occurred while loading it).
+  final Future<Pair<RunnerSuite, Zone>> _suiteAndZone;
 
   /// Returns the test that loads the suite.
   ///
   /// Load suites are guaranteed to only contain one test. This is a utility
   /// method for accessing it directly.
   Test get test => this.group.entries.single as Test;
 
   /// Creates a load suite named [name] on [platform].
   ///
   /// [body] may return either a [RunnerSuite] or a [Future] that completes to a
@@ skipping 11 matching lines @@
       invoke(() async {
         try {
           var suite = await body();
           if (completer.isCompleted) {
             // If the load test has already been closed, close the suite it
             // generated.
             suite.close();
             return;
           }
 
-          completer.complete(suite);
+          completer.complete(new Pair(suite, Zone.current));
           invoker.removeOutstandingCallback();
         } catch (error, stackTrace) {
           registerException(error, stackTrace);
           if (!completer.isCompleted) completer.complete();
         }
       });
 
       // If the test is forcibly closed, exit immediately. It doesn't have any
       // cleanup to do that won't be handled by Loader.close.
       invoker.onClose.then((_) {
@@ skipping 15 matching lines @@
       return new Future.error(exception, stackTrace);
     }, platform: platform);
   }
 
   /// A utility constructor for a load suite that just emits [suite].
   factory LoadSuite.forSuite(RunnerSuite suite) {
     return new LoadSuite("loading ${suite.path}", () => suite,
         platform: suite.platform);
   }
 
-  LoadSuite._(String name, void body(), this.suite, {TestPlatform platform})
+  LoadSuite._(String name, void body(), this._suiteAndZone,
+          {TestPlatform platform})
       : super(const VMEnvironment(), new Group.root([
         new LocalTest(name,
             new Metadata(timeout: new Timeout(new Duration(minutes: 5))),
             body)
       ]), platform: platform);
 
   /// A constructor used by [changeSuite].
-  LoadSuite._changeSuite(LoadSuite old, Future<RunnerSuite> this.suite)
+  LoadSuite._changeSuite(LoadSuite old, this._suiteAndZone)
       : super(const VMEnvironment(), old.group, platform: old.platform);
 
   /// Creates a new [LoadSuite] that's identical to this one, but that
   /// transforms [suite] once it's loaded.
   ///
-  /// If [suite] completes to `null`, [change] won't be run.
+  /// If [suite] completes to `null`, [change] won't be run. [change] is run
+  /// within the load test's zone, so any errors or prints it emits will be
+  /// associated with that test.
   LoadSuite changeSuite(RunnerSuite change(RunnerSuite suite)) {
-    return new LoadSuite._changeSuite(this, suite.then((loadedSuite) {
-      if (loadedSuite == null) return null;
-      return change(loadedSuite);
+    return new LoadSuite._changeSuite(this, _suiteAndZone.then((pair) {
+      if (pair == null) return null;
+
+      var zone = pair.last;
+      return new Pair(zone.runUnaryGuarded(change, pair.first), zone);
     }));
   }
 
   /// Runs the test and returns the suite.
   ///
   /// Rather than emitting errors through a [LiveTest], this just pipes them
   /// through the return value.
   Future<RunnerSuite> getSuite() async {
     var liveTest = await test.load(this);
     liveTest.onPrint.listen(print);
     await liveTest.run();
 
     if (liveTest.errors.isEmpty) return await suite;
 
     var error = liveTest.errors.first;
     await new Future.error(error.error, error.stackTrace);
     throw 'unreachable';
   }
 }