Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1115)

Unified Diff: test/utils.dart

Issue 960113004: More fixing tests to make bots happy. (Closed) Base URL: https://github.com/dart-lang/dart_style.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/command_line_test.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/utils.dart
diff --git a/test/utils.dart b/test/utils.dart
index 30a273abcde6e5a56589ad8c5ade7c617ed0a0c6..8fba63db529d9c838f5bd0e7e2675e41129319a1 100644
--- a/test/utils.dart
+++ b/test/utils.dart
@@ -22,7 +22,13 @@ ScheduledProcess runFormatter([List<String> args]) {
var formatterPath = p.join(
p.dirname(p.fromUri(Platform.script)), "..", "bin", "format.dart");
- args.insertAll(0, ["--package-root=${Platform.packageRoot}", formatterPath]);
+ args.insert(0, formatterPath);
+
+ // Use the same package root, if there is one.
+ if (Platform.packageRoot.isNotEmpty) {
+ args.insert(0, "--package-root=${Platform.packageRoot}");
+ }
+
return new ScheduledProcess.start(Platform.executable, args);
}
« no previous file with comments | « test/command_line_test.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698