Skip to content

Commit 0f74d7c

Browse files
authored
Use a timeout factor of 3 in the integration test (#3573)
1 parent f47324d commit 0f74d7c

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

test/end2end/dartdoc_integration_test.dart

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,8 @@
22
// for details. All rights reserved. Use of this source code is governed by a
33
// BSD-style license that can be found in the LICENSE file.
44

5-
library dartdoc.dartdoc_integration_test;
5+
@Timeout.factor(3)
6+
library;
67

78
import 'dart:async';
89
import 'dart:io';
@@ -48,7 +49,7 @@ void main() {
4849
contains('package:test_package has no documentable libraries')),
4950
);
5051
await process.shouldExit(0);
51-
}, timeout: Timeout.factor(2));
52+
});
5253

5354
group('invoking dartdoc on a basic package', () {
5455
late String packagePath;
@@ -70,7 +71,7 @@ void main() {
7071
await process.shouldExit(0);
7172
var docs = Directory(path.join(packagePath, 'doc', 'api'));
7273
expect(docs.existsSync(), isFalse);
73-
}, timeout: Timeout.factor(2));
74+
});
7475

7576
test('with --quiet is quiet and does generate docs', () async {
7677
var process = await runDartdoc(
@@ -83,7 +84,7 @@ void main() {
8384
await process.shouldExit(0);
8485
var indexHtml = Directory(path.join(packagePath, 'doc', 'api'));
8586
expect(indexHtml.listSync(), isNotEmpty);
86-
}, timeout: Timeout.factor(2));
87+
});
8788

8889
test('with invalid options return non-zero and print a fatal-error',
8990
() async {
@@ -150,8 +151,8 @@ void main() {
150151
emitsThrough(
151152
'{"level":"WARNING","message":"Found 1 warning and 0 errors."}'));
152153
await process.shouldExit(0);
153-
}, timeout: Timeout.factor(2));
154-
}, timeout: Timeout.factor(8));
154+
});
155+
});
155156

156157
test('with tool errors cause non-zero exit when warnings are off', () async {
157158
// TODO(srawlins): Remove test_package_tool_error and generate afresh.
@@ -171,7 +172,7 @@ void main() {
171172
workingDirectory: packagePath,
172173
);
173174
await process.shouldExit(1);
174-
}, timeout: Timeout.factor(2));
175+
});
175176

176177
test('with missing FLUTTER_ROOT exception reports an error', () async {
177178
// TODO(srawlins): Remove test_package_flutter_plugin and generate afresh.
@@ -185,10 +186,15 @@ void main() {
185186
includeParentEnvironment: false,
186187
);
187188
await expectLater(
188-
process.stderr,
189-
emitsThrough(matches(
190-
'Top level package requires Flutter but FLUTTER_ROOT environment variable not set|'
191-
'test_package_flutter_plugin requires the Flutter SDK, version solving failed')));
189+
process.stderr,
190+
emitsThrough(
191+
matches(
192+
'Top level package requires Flutter but FLUTTER_ROOT environment '
193+
'variable not set|test_package_flutter_plugin requires the Flutter '
194+
'SDK, version solving failed',
195+
),
196+
),
197+
);
192198
await process.shouldExit(1);
193199
});
194200
}

0 commit comments

Comments
 (0)