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

Unified Diff: tools/testing/dart/main.dart

Issue 2908833002: Revert "Revert "Refactor test option parsing code."" (Closed)
Patch Set: Ignore "—failure-summary" so it doesn't break the bots. Created 3 years, 7 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
Index: tools/testing/dart/main.dart
diff --git a/tools/testing/dart/main.dart b/tools/testing/dart/main.dart
index 727cccf18b534913674585848b02ea090066e02e..ff2a0d1264180d49555116e52d204e360e1bbe6a 100644
--- a/tools/testing/dart/main.dart
+++ b/tools/testing/dart/main.dart
@@ -22,8 +22,8 @@
/// `factory StandardTestSuite.forDirectory`.
import "dart:io";
+import "options.dart";
import "test_configurations.dart";
-import "test_options.dart";
import "test_suite.dart";
/// Runs all of the tests specified by the given command line [arguments].
@@ -32,8 +32,8 @@ void main(List<String> arguments) {
TestUtils.setDartDirUri(Platform.script.resolve('../../..'));
// Parse the command line arguments to a configuration.
- var optionsParser = new TestOptionsParser();
- var configurations = optionsParser.parse(arguments);
+ var parser = new OptionsParser();
+ var configurations = parser.parse(arguments);
if (configurations == null || configurations.isEmpty) return;
// Run all of the configured tests.

Powered by Google App Engine
This is Rietveld 408576698