Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(582)

Unified Diff: lib/src/runner/browser/suite.dart

Issue 1422963004: Add RunnerSuite.{on,is}Debugging properties. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Code review changes Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « lib/src/runner/browser/iframe_listener.dart ('k') | lib/src/runner/load_suite.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/runner/browser/suite.dart
diff --git a/lib/src/runner/browser/suite.dart b/lib/src/runner/browser/suite.dart
new file mode 100644
index 0000000000000000000000000000000000000000..075a413e576e8b3db58f9cf1735d870bcd8da404
--- /dev/null
+++ b/lib/src/runner/browser/suite.dart
@@ -0,0 +1,166 @@
+// 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.browser.suite;
+
+import 'dart:async';
+
+import 'package:async/async.dart';
+
+import '../../backend/group.dart';
+import '../../backend/metadata.dart';
+import '../../backend/test.dart';
+import '../../backend/test_platform.dart';
+import '../../util/multi_channel.dart';
+import '../../util/remote_exception.dart';
+import '../../util/stack_trace_mapper.dart';
+import '../../util/stream_channel.dart';
+import '../../utils.dart';
+import '../environment.dart';
+import '../load_exception.dart';
+import '../runner_suite.dart';
+import 'iframe_test.dart';
+
+/// Loads a [RunnerSuite] for a browser.
+///
+/// [channel] should connect to the iframe containing the suite, which should
+/// eventually emit a message containing the suite's test information.
+/// [environment], [path], [platform], and [onClose] are passed to the
+/// [RunnerSuite]. If passed, [mapper] is used to reformat the test's stack
+/// traces.
+Future<RunnerSuite> loadBrowserSuite(StreamChannel channel,
+ Environment environment, String path, {StackTraceMapper mapper,
+ TestPlatform platform, AsyncFunction onClose}) async {
+ // The controller for the returned suite. This is set once we've loaded the
+ // information about the tests in the suite.
+ var controller;
+
+ // A timer that's reset whenever we receive a message from the browser.
+ // Because the browser stops running code when the user is actively debugging,
+ // this lets us detect whether they're debugging reasonably accurately.
+ //
+ // The duration should be short enough that the debugging console is open as
+ // soon as the user is done setting breakpoints, but long enough that a test
+ // doing a lot of synchronous work doesn't trigger a false positive.
+ //
+ // Start this canceled because we don't want it to start ticking until we get
+ // some response from the iframe.
+ var timer = new RestartableTimer(new Duration(seconds: 3), () {
+ controller.setDebugging(true);
+ })..cancel();
+
+ // Even though [channel] is probably a [MultiChannel] already, create a
+ // nested MultiChannel because the iframe will be using a channel wrapped
+ // within the host's channel.
+ var suiteChannel = new MultiChannel(channel.stream.map((message) {
+ // Whenever we get a message, no matter which child channel it's for, we the
+ // browser is still running code which means the using isn't debugging.
+ if (controller != null) {
+ timer.reset();
+ controller.setDebugging(false);
+ }
+
+ return message;
+ }), channel.sink);
+
+ var response = await _getResponse(suiteChannel.stream)
+ .timeout(new Duration(minutes: 1), onTimeout: () {
+ suiteChannel.sink.close();
+ throw new LoadException(
+ path,
+ "Timed out waiting for the test suite to connect.");
+ });
+
+ try {
+ _validateResponse(path, response);
+ } catch (_) {
+ suiteChannel.sink.close();
+ rethrow;
+ }
+
+ controller = new RunnerSuiteController(environment,
+ _deserializeGroup(suiteChannel, response["root"], mapper),
+ platform: platform, path: path,
+ onClose: () {
+ suiteChannel.sink.close();
+ timer.cancel();
+ return onClose == null ? null : onClose();
+ });
+
+ // Start the debugging timer counting down.
+ timer.reset();
+ return controller.suite;
+}
+
+/// Listens for responses from the iframe on [stream].
+///
+/// Returns the serialized representation of the the root group for the suite,
+/// or a response indicating that an error occurred.
+Future<Map> _getResponse(Stream stream) {
+ var completer = new Completer();
+ stream.listen((response) {
+ if (response["type"] == "print") {
+ print(response["line"]);
+ } else if (response["type"] != "ping") {
+ completer.complete(response);
+ }
+ }, onDone: () {
+ if (!completer.isCompleted) completer.complete();
+ });
+
+ return completer.future;
+}
+
+/// Throws an error encoded in [response], if there is one.
+///
+/// [path] is used for the error's metadata.
+Future _validateResponse(String path, Map response) {
+ if (response == null) {
+ throw new LoadException(
+ path, "Connection closed before test suite loaded.");
+ }
+
+ if (response["type"] == "loadException") {
+ throw new LoadException(path, response["message"]);
+ }
+
+ if (response["type"] == "error") {
+ var asyncError = RemoteException.deserialize(response["error"]);
+ return new Future.error(
+ new LoadException(path, asyncError.error),
+ asyncError.stackTrace);
+ }
+
+ return new Future.value();
+}
+
+/// Deserializes [group] into a concrete [Group] class.
+Group _deserializeGroup(MultiChannel suiteChannel, Map group,
+ [StackTraceMapper mapper]) {
+ var metadata = new Metadata.deserialize(group['metadata']);
+ return new Group(group['name'], group['entries'].map((entry) {
+ if (entry['type'] == 'group') {
+ return _deserializeGroup(suiteChannel, entry, mapper);
+ }
+
+ return _deserializeTest(suiteChannel, entry, mapper);
+ }),
+ metadata: metadata,
+ setUpAll: _deserializeTest(suiteChannel, group['setUpAll'], mapper),
+ tearDownAll:
+ _deserializeTest(suiteChannel, group['tearDownAll'], mapper));
+}
+
+/// Deserializes [test] into a concrete [Test] class.
+///
+/// Returns `null` if [test] is `null`.
+Test _deserializeTest(MultiChannel suiteChannel, Map test,
+ [StackTraceMapper mapper]) {
+ if (test == null) return null;
+
+ var metadata = new Metadata.deserialize(test['metadata']);
+ var testChannel = suiteChannel.virtualChannel(test['channel']);
+ return new IframeTest(test['name'], metadata, testChannel,
+ mapper: mapper);
+}
« no previous file with comments | « lib/src/runner/browser/iframe_listener.dart ('k') | lib/src/runner/load_suite.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698