Refactor groups to pipe them through to the runner.

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';
 
 import '../../test.dart';
 import '../backend/invoker.dart';
 import '../backend/metadata.dart';
+import '../backend/test.dart';
 import '../backend/test_platform.dart';
 import '../utils.dart';
 import 'load_exception.dart';
 import 'runner_suite.dart';
 import 'vm/environment.dart';
 
 /// A [Suite] emitted by a [Loader] that provides a test-like interface for
 /// loading a test file.
 ///
 /// This is used to expose the current status of test loading to the user. It's
 /// important to provide users visibility into what's taking a long time and
 /// where failures occur. And since some tests may be loaded at the same time as
 /// others are run, it's useful to provide that visibility in the form of a test
 /// suite so that it can integrate well into the existing reporting interface
 /// without too much extra logic.
 ///
 /// 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;
 
+  /// 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 => 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
   /// [RunnerSuite]. Its return value is forwarded through [suite], although if
   /// it throws an error that will be forwarded through the suite's test.
   ///
   /// If the the load test is closed before [body] is complete, it will close
   /// the suite returned by [body] once it completes.
   factory LoadSuite(String name, body(), {TestPlatform platform}) {
     var completer = new Completer.sync();
@@ skipping 49 matching lines @@
 
   LoadSuite._(String name, void body(), this.suite, {TestPlatform platform})
       : super(const VMEnvironment(), [
         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)
-      : super(const VMEnvironment(), old.tests, platform: old.platform);
+      : super(const VMEnvironment(), old.entries, 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.
   LoadSuite changeSuite(RunnerSuite change(RunnerSuite suite)) {
     return new LoadSuite._changeSuite(this, suite.then((loadedSuite) {
       if (loadedSuite == null) return null;
       return change(loadedSuite);
     }));
   }
 
   /// 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 tests.single.load(this);
+    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';
   }
 }