fix: do not strip excess leading dashes on long option names (#21)

pkgjs · Dec 21, 2021 · f848590 · f848590
1 parent 80843da
commit f848590
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -31,9 +31,7 @@ const parseArgs = (
         throw new Error('What are we doing with shortcodes!?!');
       }
 
-      // Any number of leading dashes are allowed
-      // remove all leading dashes
-      arg = arg.replace(/^-+/, '');
+      arg = arg.slice(2); // remove leading --
 
       if (arg.includes('=')) {
         // withValue equals(=) case

diff --git a/test/index.js b/test/index.js
@@ -68,6 +68,17 @@ test('args are passed "withValue" and "multiples"', function (t) {
   t.end()
 })
 
+test('excess leading dashes on options are retained', function(t) {
+  // Enforce a design decision for an edge case.
+  const passedArgs = ['---triple'];
+  const passedOptions = { };
+  const expected = { flags: { '-triple': true}, values: { '-triple': [undefined]}, positionals: [] };
+  const args = parseArgs(passedArgs, passedOptions);
+
+  t.deepEqual(args, expected, 'excess option dashes are retained');
+
+  t.end();
+});
 
 //Test bad inputs