Add command support to args.

pkg/args/test/command_test.dart

Index: pkg/args/test/command_test.dart
diff --git a/pkg/args/test/command_test.dart b/pkg/args/test/command_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..4efdde483e7c893f5136cf99aaf348cfdc433d6f
--- /dev/null
+++ b/pkg/args/test/command_test.dart
@@ -0,0 +1,193 @@
+// 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 command_test;
+
+import 'package:unittest/unittest.dart';
+import 'package:args/args.dart';
+
+main() {
+  group('ArgParser.addCommand()', () {
+    test('returns an ArgParser for the command', () {

[nweiz, 2013/01/11 00:06:06]

This seems kind of pedantic. I don't think it's worth testing this when you're
already testing all sorts of ArgParser behavior.

[Bob Nystrom, 2013/01/11 17:59:49]

Removed. I'm never sure whether or not "nominality" is part of a library's
public API guarantees.

+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+      expect(command, new isInstanceOf<ArgParser>());
+    });
+
+    test('throws on a duplicate command name', () {
+      var parser = new ArgParser();
+      parser.addCommand('install');
+      throwsIllegalArg(() => parser.addCommand('install'));
+    });
+  });
+
+  group('ArgParser.parse()', () {
+    test('parses a command', () {
+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['install']);
+
+      expect(args.command, isNotNull);

[nweiz, 2013/01/11 00:06:06]

This seems redundant with the expect below.

[Bob Nystrom, 2013/01/11 17:59:49]

Removed. Made since at the time since this was the only test that passed at
first. :)

+      expect(args.command.name, equals('install'));
+      expect(args.rest, isEmpty);
+    });
+
+    test('parses a command option', () {
+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+      command.addOption('path');
+
+      var args = parser.parse(['install', '--path', 'some/path']);
+      expect(args.command['path'], equals('some/path'));
+    });
+
+    test('parses a parent solo option before the command', () {
+      var parser = new ArgParser();
+      parser.addOption('mode', abbr: 'm');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['-m', 'debug', 'install']);
+      expect(args['mode'], equals('debug'));
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent solo option after the command', () {
+      var parser = new ArgParser();
+      parser.addOption('mode', abbr: 'm');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['install', '-m', 'debug']);
+      expect(args['mode'], equals('debug'));
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent option before the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('verbose');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['--verbose', 'install']);
+      expect(args['verbose'], isTrue);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent option after the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('verbose');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['install', '--verbose']);
+      expect(args['verbose'], isTrue);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent negated option before the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('verbose', defaultsTo: true);
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['--no-verbose', 'install']);
+      expect(args['verbose'], isFalse);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent negated option after the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('verbose', defaultsTo: true);
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['install', '--no-verbose']);
+      expect(args['verbose'], isFalse);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent abbreviation before the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('debug', abbr: 'd');
+      parser.addFlag('verbose', abbr: 'v');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['-dv', 'install']);
+      expect(args['debug'], isTrue);
+      expect(args['verbose'], isTrue);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('parses a parent abbreviation after the command', () {
+      var parser = new ArgParser();
+      parser.addFlag('debug', abbr: 'd');
+      parser.addFlag('verbose', abbr: 'v');
+      var command = parser.addCommand('install');
+
+      var args = parser.parse(['install', '-dv']);
+      expect(args['debug'], isTrue);
+      expect(args['verbose'], isTrue);
+      expect(args.command.name, equals('install'));
+    });
+
+    test('does not parse a solo command before the command', () {

[nweiz, 2013/01/11 00:06:06]

"solo command" -> "solo command option"

[Bob Nystrom, 2013/01/11 17:59:49]

Done.

+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+      command.addOption('path', abbr: 'p');
+
+      throwsFormat(parser, ['-p', 'foo', 'install']);
+    });
+
+    test('does not parse a command option before the command', () {
+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+      command.addOption('path');
+
+      throwsFormat(parser, ['--path', 'foo', 'install']);
+    });
+
+    test('does not parse a command abbreviation before the command', () {
+      var parser = new ArgParser();
+      var command = parser.addCommand('install');
+      command.addFlag('debug', abbr: 'd');
+      command.addFlag('verbose', abbr: 'v');
+
+      throwsFormat(parser, ['-dv', 'install']);
+    });
+
+    test('assigns collapsed options to the proper command', () {
+      var parser = new ArgParser();
+      parser.addFlag('apple', abbr: 'a');
+      var command = parser.addCommand('cmd');
+      command.addFlag('banana', abbr: 'b');
+      var subcommand = command.addCommand('subcmd');
+      subcommand.addFlag('cherry', abbr: 'c');
+
+      var args = parser.parse(['cmd', 'subcmd', '-abc']);
+      expect(args['apple'], isTrue);
+      expect(args.command.name, equals('cmd'));
+      expect(args.command['banana'], isTrue);
+      expect(args.command.command.name, equals('subcmd'));
+      expect(args.command.command['cherry'], isTrue);
+    });
+
+    test('option is given to innermost command that can take it', () {
+      var parser = new ArgParser();
+      parser.addFlag('verbose');
+      var command = parser.addCommand('cmd');
+      command.addFlag('verbose');
+      var subcommand = command.addCommand('subcmd');
+
+      var args = parser.parse(['cmd', 'subcmd', '--verbose']);
+      expect(args['verbose'], isFalse);
+      expect(args.command.name, equals('cmd'));
+      expect(args.command['verbose'], isTrue);
+      expect(args.command.command.name, equals('subcmd'));
+    });
+  });
+}
+
+throwsIllegalArg(function) {
+  expect(function, throwsArgumentError);
+}
+
+throwsFormat(ArgParser parser, List<String> args) {
+  expect(() => parser.parse(args), throwsFormatException);
+}