From fd6baba1373c1dde90511ca15d9cc89aa642fe3e Mon Sep 17 00:00:00 2001 From: Chris Bracken Date: Fri, 9 Feb 2018 21:43:08 -0800 Subject: [PATCH] Allow spaces in SDK path in flutter_tools tests (#14617) The Flutter engine now supports package: and file: imports that resolve to paths on disk that include spaces and other URI-escaped characters. This patch eliminates the restriction that Dart source paths not include %20 (or other URI-escaped characters) in their paths in flutter_tool tests. --- packages/flutter_tools/test/src/common.dart | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/flutter_tools/test/src/common.dart b/packages/flutter_tools/test/src/common.dart index 06bec7c68d2..40d427926ac 100644 --- a/packages/flutter_tools/test/src/common.dart +++ b/packages/flutter_tools/test/src/common.dart @@ -32,7 +32,7 @@ String getFlutterRoot() { scriptUri = platform.script; break; case 'data': - final RegExp flutterTools = new RegExp(r'(file://[^%]*[/\\]flutter_tools[^%]+\.dart)%'); + final RegExp flutterTools = new RegExp(r'(file://[^ ;]*[/\\]flutter_tools[^%]+\.dart)%'); final Match match = flutterTools.firstMatch(platform.script.path); if (match == null) throw invalidScript();