From 002d8678336afdd645974354639364d139910758 Mon Sep 17 00:00:00 2001
From: Natalie Weizenbaum <nweiz@google.com>
Date: Mon, 6 Nov 2017 16:36:02 -0800
Subject: [PATCH] Remove the deserialize timeout
This can never fire before the load suite timeout. Now that that
timeout actually works, it's redundant.
See #390
---
lib/src/runner/plugin/platform_helpers.dart | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)
diff --git a/lib/src/runner/plugin/platform_helpers.dart b/lib/src/runner/plugin/platform_helpers.dart
index b41e5ba8..fa0c2a10 100644
--- a/lib/src/runner/plugin/platform_helpers.dart
+++ b/lib/src/runner/plugin/platform_helpers.dart
@@ -23,8 +23,6 @@ import '../load_exception.dart';
import '../runner_suite.dart';
import '../runner_test.dart';
-final _deserializeTimeout = new Duration(minutes: 8);
-
/// A helper method for creating a [RunnerSuiteController] containing tests
/// that communicate over [channel].
///
@@ -113,13 +111,7 @@ Future<RunnerSuiteController> deserializeSuite(
});
return new RunnerSuiteController(
- environment,
- suiteConfig,
- await completer.future.timeout(_deserializeTimeout, onTimeout: () {
- throw new ApplicationException(
- "Timed out while loading the test suite.\n"
- "It's likely that there's a missing import or syntax error.");
- }),
+ environment, suiteConfig, await completer.future,
path: path,
platform: platform,
os: currentOS,
--
GitLab