From f68bc1beb206cc058b0301f61d407dd2c6ba7a2d Mon Sep 17 00:00:00 2001 From: Jonah Williams Date: Thu, 13 Jun 2019 17:51:36 -0700 Subject: [PATCH] Dont depend on web SDK unless running tests on chrome (#34457) --- packages/flutter_tools/lib/src/commands/test.dart | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/packages/flutter_tools/lib/src/commands/test.dart b/packages/flutter_tools/lib/src/commands/test.dart index cce7e757e0..28641928b2 100644 --- a/packages/flutter_tools/lib/src/commands/test.dart +++ b/packages/flutter_tools/lib/src/commands/test.dart @@ -21,6 +21,7 @@ import '../test/coverage_collector.dart'; import '../test/event_printer.dart'; import '../test/runner.dart'; import '../test/watcher.dart'; + class TestCommand extends FastFlutterCommand { TestCommand({ bool verboseHelp = false }) { requiresPubspecYaml(); @@ -108,10 +109,15 @@ class TestCommand extends FastFlutterCommand { } @override - Future> get requiredArtifacts async => { - DevelopmentArtifact.universal, - DevelopmentArtifact.web, - }; + Future> get requiredArtifacts async { + final Set results = { + DevelopmentArtifact.universal, + }; + if (argResults['platform'] == 'chrome') { + results.add(DevelopmentArtifact.web); + } + return results; + } @override String get name => 'test';