From 48c92e0f9add1761ef79d036bd0410ba79512409 Mon Sep 17 00:00:00 2001 From: Natalie Weizenbaum Date: Mon, 6 Nov 2017 16:39:56 -0800 Subject: [PATCH] Increase the load suite timeout Closes #390 --- lib/src/runner/load_suite.dart | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/lib/src/runner/load_suite.dart b/lib/src/runner/load_suite.dart index 3b867066d..c5d504844 100644 --- a/lib/src/runner/load_suite.dart +++ b/lib/src/runner/load_suite.dart @@ -19,6 +19,14 @@ import 'load_exception.dart'; import 'plugin/environment.dart'; import 'runner_suite.dart'; +/// The timeout for loading a test suite. +/// +/// We want this to be long enough that even a very large application being +/// compiled with dart2js doesn't trigger it, but short enough that it fires +/// before the host kills it. For example, Google's Forge service has a +/// 15-minute timeout. +final _timeout = new Duration(minutes: 12); + /// A [Suite] emitted by a [Loader] that provides a test-like interface for /// loading a test file. /// @@ -129,7 +137,7 @@ class LoadSuite extends Suite implements RunnerSuite { new Group.root([ new LocalTest( name, - new Metadata(timeout: new Timeout(new Duration(minutes: 5))), + new Metadata(timeout: new Timeout(_timeout)), body) ]), path: path,