Added the continueParsing option to ArgParser.

pkg/args/test/parse_test.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library parse_test;
 
 import 'package:unittest/unittest.dart';
 import 'package:args/args.dart';
 
 main() {
@@ skipping 371 matching lines @@
         parser.addOption('define', defaultsTo: '0', allowMultiple: true);
         var args = parser.parse(['']);
         expect(args['define'], equals(['0']));
       });
     });
 
     group('remaining args', () {
       test('stops parsing args when a non-option-like arg is encountered', () {
         var parser = new ArgParser();
         parser.addFlag('woof');
+        parser.addFlag('neigh');

[Bob Nystrom, 2013/04/22 22:59:03]

Why add neigh to this test?

         parser.addOption('meow');
         parser.addOption('tweet', defaultsTo: 'bird');
 
-        var results = parser.parse(['--woof', '--meow', 'v', 'not', 'option']);
+        var results = parser.parse(['--woof', '--meow', 'v', 'not', 'option',
+            '--tweet', 'devrel', '--neigh', 'end' ]);
         expect(results['woof'], isTrue);
+        expect(results['neigh'], isFalse);
         expect(results['meow'], equals('v'));
         expect(results['tweet'], equals('bird'));
-        expect(results.rest, orderedEquals(['not', 'option']));
+        expect(results.rest, orderedEquals(['not', 'option', '--tweet',
+            'devrel', '--neigh', 'end']));
+      });
+
+      test('with continueParsing, continues parsing when non-option-like'
+        ' arg is encountered', () {
+        var parser = new ArgParser();
+        parser.addFlag('woof');
+        parser.addFlag('neigh');
+        parser.addOption('meow');
+        parser.addOption('tweet', defaultsTo: 'bird');
+
+        var results = parser.parse(['--woof', '--meow', 'v', 'not', 'option',
+            '--tweet', 'devrel', '--neigh', 'end' ],

[Bob Nystrom, 2013/04/22 22:59:03]

This test is pretty complicated. How about simplifying it to just the one thing
being tested, like:

parser.addFlag('woof');

parser.parse(['notoption', 'woof']);

+            continueParsing: true);
+        expect(results['woof'], isTrue);
+        expect(results['neigh'], isTrue);
+        expect(results['meow'], equals('v'));
+        expect(results['tweet'], equals('devrel'));
+        expect(results.rest, orderedEquals(['not', 'option', 'end']));
+      });
+

[Bob Nystrom, 2013/04/22 22:59:03]

Add a test for what happens if continueParsing is true and an argument that
looks like an option but isn't ('--notopt') is encountered.

+      test('with continueParsing, gathers all seen arguments to leaf command',
+          () {
+        var parser = new ArgParser();
+        parser.addFlag('woof');
+        parser.addFlag('neigh');
+        parser.addOption('tweet', defaultsTo: 'bird');
+
+        var command = parser.addCommand('v');
+        command.addCommand('option')
+          ..addOption('tweet', defaultsTo: 'foo')
+          ..addFlag('neigh');

[Bob Nystrom, 2013/04/22 22:59:03]

We should use a consistent style here. How about just ditching the cascades?

+        command.addCommand('end');
+
+        var results = parser.parse(['--woof', 'meow', 'v', 'not', 'option',
+            '--tweet', 'devrel', '--neigh', 'end' ],
+            continueParsing: true);
+        expect(results['woof'], isTrue);
+        expect(results['tweet'], equals('bird'));
+        expect(results.rest, equals([]));
+        expect(results.command.name, equals('v'));
+        expect(results.command.rest, equals([]));
+        expect(results.command.command['neigh'], isTrue);
+        expect(results.command.command.name, equals('option'));
+        expect(results.command.command.rest, orderedEquals(['meow', 'not', 'end']));

[Bob Nystrom, 2013/04/22 22:59:03]

This test is big and hard to read. Split it into a couple of smaller ones:

1. Test that skipped arguments are passed to the leafmost command.
2. Test parsed options are given to the leafmost command when both a command and
its parent define the same option.

Also, I would move these tests to command_test.dart

       });
 
       test('stops parsing at "--"', () {
         var parser = new ArgParser();
         parser.addFlag('woof', defaultsTo: false);
         parser.addOption('meow', defaultsTo: 'kitty');
 
         var results = parser.parse(['--woof', '--', '--meow']);
         expect(results['woof'], isTrue);
         expect(results['meow'], equals('kitty'));
@@ skipping 12 matching lines @@
   });
 }
 
 throwsIllegalArg(function) {
   expect(function, throwsArgumentError);
 }
 
 throwsFormat(ArgParser parser, List<String> args) {
   expect(() => parser.parse(args), throwsFormatException);
 }