Extend synthetic test timeout (#1107)

* Extend synthetic test timeout

Author: grouma

pkgs/test/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.9.4
+
+* Extend the timeout for synthetic tests, e.g. `tearDownAll`.
+* Depend on the latest `package:test_core`.
+* Depend on the latest `package:test_api`.
+
 ## 1.9.3
 
 * Depend on the latest `package:test_core`.

pkgs/test/pubspec.yaml

@@ -1,5 +1,5 @@
 name: test
-version: 1.9.3
+version: 1.9.4
 author: Dart Team <misc@dartlang.org>
 description: A full featured library for writing and running Dart tests.
 homepage: https://github.com/dart-lang/test/blob/master/pkgs/test
@@ -31,8 +31,8 @@ dependencies:
  web_socket_channel: ^1.0.0
  yaml: ^2.0.0
  # Use an exact version until the test_api and test_core package are stable.
- test_api: 0.2.10
- test_core: 0.2.14
+ test_api: 0.2.11
+ test_core: 0.2.15
 
 dev_dependencies:
  fake_async: ^1.0.0

pkgs/test_api/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.2.11
+
+* Extend the timeout for synthetic tests, e.g. `tearDownAll`.
+
 ## 0.2.10
 
 * Update to latest `package:matcher`. Improves output for instances of private

pkgs/test_api/lib/src/backend/declarer.dart

@@ -58,6 +58,9 @@ class Declarer {
  /// The set-up functions to run once for this group.
  final _setUpAlls = List<Function()>();
 
+ /// The default timeout for synthetic tests.
+ final _timeout = Timeout(Duration(minutes: 12));
+
  /// The trace for the first call to [setUpAll].
  ///
  /// All [setUpAll]s are run in a single logical test, so they can only have
@@ -295,7 +298,8 @@ class Declarer {
  Test get _setUpAll {
  if (_setUpAlls.isEmpty) return null;
 
- return LocalTest(_prefix("(setUpAll)"), _metadata, () {
+ return LocalTest(_prefix("(setUpAll)"), _metadata.change(timeout: _timeout),
+ () {
  return runZoned(() => Future.forEach(_setUpAlls, (setUp) => setUp()),
  // Make the declarer visible to running scaffolds so they can add to
  // the declarer's `tearDownAll()` list.
@@ -309,7 +313,8 @@ class Declarer {
  // dynamically add tear-down code using [addTearDownAll].
  if (_setUpAlls.isEmpty && _tearDownAlls.isEmpty) return null;
 
- return LocalTest(_prefix("(tearDownAll)"), _metadata, () {
+ return LocalTest(
+ _prefix("(tearDownAll)"), _metadata.change(timeout: _timeout), () {
  return runZoned(() {
  return Invoker.current.unclosable(() async {
  while (_tearDownAlls.isNotEmpty) {

pkgs/test_api/pubspec.yaml

@@ -1,5 +1,5 @@
 name: test_api
-version: 0.2.10
+version: 0.2.11
 author: Dart Team <misc@dartlang.org>
 description: A library for writing Dart tests.
 homepage: https://github.com/dart-lang/test/blob/master/pkgs/test_api

pkgs/test_core/CHANGELOG.md

@@ -1,8 +1,9 @@
-## 0.2.15-dev
+## 0.2.15
 
 * Add a `StringSink` argument to reporters to prepare for reporting to a file.
 * Add --test-randomize-ordering-seed` argument to randomize test
 execution order based on a provided seed
+* Depend on the latest `test_api`.
 
 ## 0.2.14
 

pkgs/test_core/pubspec.yaml

@@ -1,5 +1,5 @@
 name: test_core
-version: 0.2.15-dev
+version: 0.2.15
 author: Dart Team <misc@dartlang.org>
 description: A basic library for writing tests and running them on the VM.
 homepage: https://github.com/dart-lang/test/blob/master/pkgs/test_core
@@ -32,7 +32,7 @@ dependencies:
  # properly constrains all features it provides.
  matcher: ">=0.12.6 <0.12.7"
  # Use an exact version until the test_api package is stable.
- test_api: 0.2.10
+ test_api: 0.2.11
 
 dependency_overrides:
  test_api: