[Fix] consider aliases when testing boolean long option value against…

… 'false' Fixes #30.
minimistjs · Apr 8, 2023 · de53490 · de53490
1 parent 2758c33
commit de53490
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -162,7 +162,7 @@ module.exports = function (args, opts) {
 			var m = arg.match(/^--([^=]+)=([\s\S]*)$/);
 			key = m[1];
 			var value = m[2];
-			if (flags.bools[key]) {
+			if (isBooleanKey(key)) {
 				value = value !== 'false';
 			}
 			setArg(key, value, arg);

diff --git a/test/bool.js b/test/bool.js
@@ -167,6 +167,18 @@ test('supplied default for boolean using alias', function (t) {
 		b: true,
 		_: ['moo'],
 	});
+	t.end();
+});
+
+test('boolean and alias --boool=false', function (t) {
+	var parsed = parse(['--boool=false'], {
+		default: {
+			b: true,
+		},
+		alias: { b: 'boool' },
+		boolean: ['b'],
+	});
 
+	t.same(parsed.boool, false);
 	t.end();
 });