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

Unified Diff: pkg/args/test/parse_all_test.dart

Issue 12545013: Added the continueParsing option to ArgParser. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Done Created 7 years, 6 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 | « pkg/args/lib/src/parser.dart ('k') | sdk/lib/_internal/dartdoc/bin/dartdoc.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/args/test/parse_all_test.dart
diff --git a/pkg/args/test/parse_all_test.dart b/pkg/args/test/parse_all_test.dart
index a921046465786147256074fb58c14bd3cc17ec4e..8498ae672c63da31b5f40868a260970ed279d941 100644
--- a/pkg/args/test/parse_all_test.dart
+++ b/pkg/args/test/parse_all_test.dart
@@ -8,55 +8,56 @@ import 'package:unittest/unittest.dart';
import 'package:args/args.dart';
main() {
- group('ArgParser.parse() starting with a non-option', () {
+ group('ArgParser.parse(allowTrailingOptions: true) '
+ 'starting with a non-option', () {
test('followed by flag', () {
var parser = new ArgParser()..addFlag('flag');
var args = ['A', '--flag'];
- var results = parser.parse(args);
- expect(results['flag'], isFalse);
- expect(results.rest, orderedEquals(args));
+ var resultsAll = parser.parse(args, allowTrailingOptions: true);
+ expect(resultsAll['flag'], isTrue);
+ expect(resultsAll.rest, equals(['A']));
});
test('followed by option', () {
var parser = new ArgParser()..addOption('opt');
var args = ['A', '--opt'];
- var results = parser.parse(args);
- expect(results['opt'], isNull);
- expect(results.rest, orderedEquals(args));
+ expectThrows(parser, args);
});
test('followed by option and value', () {
var parser = new ArgParser()..addOption('opt');
var args = ['A', '--opt', 'V'];
- var results = parser.parse(args);
- expect(results['opt'], isNull);
- expect(results.rest, orderedEquals(args));
+ var resultsAll = parser.parse(args, allowTrailingOptions: true);
+ expect(resultsAll['opt'], equals('V'));
+ expect(resultsAll.rest, equals(['A']));
});
test('followed by unknown flag', () {
var parser = new ArgParser();
var args = ['A', '--xflag'];
- var results = parser.parse(args);
- expect(results.rest, orderedEquals(args));
+
+ expectThrows(parser, args);
});
test('followed by unknown option and value', () {
var parser = new ArgParser();
var args = ['A', '--xopt', 'V'];
- var results = parser.parse(args);
- expect(results.rest, orderedEquals(args));
+
+ expectThrows(parser, args);
});
test('followed by command', () {
var parser = new ArgParser()..addCommand('com');
var args = ['A', 'com'];
- var results = parser.parse(args);
- expect(results.command, isNull);
- expect(results.rest, orderedEquals(args));
+ expectThrows(parser, args);
});
});
}
+expectThrows(ArgParser parser, List<String> args) =>
+ expect(() => parser.parse(args, allowTrailingOptions: true),
+ throwsFormatException,
+ reason: "with allowTrailingOptions: true");
« no previous file with comments | « pkg/args/lib/src/parser.dart ('k') | sdk/lib/_internal/dartdoc/bin/dartdoc.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698