support zero as flag value (lukeed#6)

jimmyolo · lukeed · commit e3f239d423fb · 2018-05-07T09:59:51.000+02:00
* add isNum function

* simplify zero-value fix
diff --git a/lib/index.js b/lib/index.js
@@ -21,7 +21,8 @@ function toBool(any) {
 }
 
 function toNum(any) {
-	return (!isBool(any) && Number(any)) || any;
+	let x = Number(any);
+	return !isBool(any) && (x * 0 === 0) ? x : any;
 }
 
 function getAlibi(names, arr) {
diff --git a/test/index.js b/test/index.js
@@ -25,6 +25,8 @@ test('comprehensive', t => {
 			'--bool',
 			'--no-meep',
 			'--multi=baz',
+			'--number=-123',
+			'--zeronum=0',
 			'--',
 			'--not-a-flag',
 			'eek'
@@ -41,6 +43,8 @@ test('comprehensive', t => {
 			multi: ['quux', 'baz'],
 			meep: false,
 			name: 'meowmers',
+			number: -123,
+			zeronum: 0,
 			_: ['bare', '--not-a-flag', 'eek']
 		}
 	);

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,8 @@ function toBool(any) {`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`function toNum(any) {`
`24`		`- return (!isBool(any) && Number(any)) \|\| any;`
	`24`	`+ let x = Number(any);`
	`25`	`+ return !isBool(any) && (x * 0 === 0) ? x : any;`
`25`	`26`	`}`
`26`	`27`
`27`	`28`	`function getAlibi(names, arr) {`