Skip to content

Commit

Permalink
Switch to concurrency of two, to see if that is a good balance. (flut…
Browse files Browse the repository at this point in the history
  • Loading branch information
gspencergoog committed Nov 13, 2017
1 parent e2b9f89 commit 2a26f29
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions packages/flutter_tools/lib/src/test/runner.dart
Expand Up @@ -38,6 +38,16 @@ Future<int> runTests(
testArgs.addAll(<String>['-r', 'json']);
}

if (enableObservatory) {
// (In particular, for collecting code coverage.)

// Turn on concurrency, but just barely. If we just go with full concurrency, then
// individual tests timeout. If we turn it off (=1), then the overall tests timeout.
// This is a lit fuse... Eventually it won't matter what number we put here.
// TODO(gspencer): Fix this: https://github.com/flutter/flutter/issues/10694
testArgs.add('--concurrency=2');
}

for (String name in names) {
testArgs..add('--name')..add(name);
}
Expand Down

0 comments on commit 2a26f29

Please sign in to comment.