Parse comma-separated multiple values.

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';
 import 'utils.dart';
 
@@ skipping 181 matching lines @@
       test('for absent, allowMultiple, options are invoked with value '
           'as an empty list.', () {
         var a;
         var parser = new ArgParser();
         parser.addOption('a',
             allowMultiple: true, callback: (value) => a = value);
 
         parser.parse([]);
         expect(a, isEmpty);
       });
+
+      test('allowMultiple parses comma-separated strings', () {
+        var a;
+        var parser = new ArgParser();
+        parser.addOption('a',
+            allowMultiple: true, callback: (value) => a = value);
+
+        parser.parse(['--a=v,w', '--a=x']);
+        expect(a, equals(['v', 'w', 'x']));
+      });
+
+      test('allowMultiple with allowed parses comma-separated strings', () {
+        var a;
+        var parser = new ArgParser();
+        parser.addOption('a',
+            allowMultiple: true,
+            allowed: ['v', 'w', 'x'],
+            callback: (value) => a = value);
+
+        parser.parse(['--a=v,w', '--a=x']);
+        expect(a, equals(['v', 'w', 'x']));
+      });
     });
 
     group('abbreviations', () {
       test('are parsed with a preceding "-"', () {
         var parser = new ArgParser();
         parser.addFlag('arg', abbr: 'a');
 
         var args = parser.parse(['-a']);
         expect(args['arg'], isTrue);
       });
@@ skipping 68 matching lines @@
         throwsFormat(parser, ['-f', '-abbr']);
       });
 
       test('throw if the value is not allowed', () {
         var parser = new ArgParser();
         parser.addOption('mode', abbr: 'm', allowed: ['debug', 'release']);
 
         throwsFormat(parser, ['-mprofile']);
       });
 
+      test('throw if a comma-separated value is not allowed', () {
+        var parser = new ArgParser();
+        parser.addOption('mode', abbr: 'm', allowMultiple: true,
+            allowed: ['debug', 'release']);
+
+        throwsFormat(parser, ['-mdebug,profile']);
+      });
+

[Bob Nystrom, 2015/03/03 17:36:26]

Tests for:

","
"a,,b"
",a"
"a,"

[nweiz, 2015/03/03 20:22:18]

I don't think these should throw... multi-value options allow the empty string
in general, so I don't see why they shouldn't allow it with this syntax as well.
I'll add a test case above.

       test('throw if any but the first is not a flag', () {
         var parser = new ArgParser();
         parser.addFlag('apple', abbr: 'a');
         parser.addOption('banana', abbr: 'b'); // Takes an argument.
         parser.addFlag('cherry', abbr: 'c');
 
         throwsFormat(parser, ['-abc']);
       });
 
       test('throw if it has a value but the option is a flag', () {
@@ skipping 142 matching lines @@
         var parser = new ArgParser();
         parser.addFlag('woof');
 
         var results = parser.parse(['--woof', 'stop', '--', 'arg']);
         expect(results['woof'], isTrue);
         expect(results.rest, equals(['stop', '--', 'arg']));
       });
     });
   });
 }