verbose option should check for NaN (#365)

* Fix: vebose option should error if no number is given

* Add tests for verbose option

Author: joshbax189

src/util.js

@@ -212,6 +212,14 @@ async function e_call(argv, script, ...args) {
     return;
   }
 
+  if (argv.verbose !== undefined &&
+    // this must exclude NaN -- yargs default value for numeric type
+    !(argv.verbose >= 0 && argv.verbose <= 5)) {
+    console.warn("invalid value for --verbose option: must be a number between 0 and 5");
+    process.exit(1);
+    return;
+  }
+
   return new Promise(resolve => {
     let _path = el_script(script);
 

test/jest/options.test.js

@@ -18,6 +18,18 @@ describe("options", () => {
     });
   });
 
+  describe("verbose option", () => {
+    it("should error if no number is given", async () => {
+      await expect(ctx.runEask("info -v")).rejects.toMatchObject({ code: 1 });
+    });
+
+    it("should error if the number is omitted before the next option", async () => {
+      await expect(ctx.runEask("info -v --no-color")).rejects.toMatchObject({
+        code: 1,
+      });
+    });
+  });
+
   test.each([
     "-a",
     "--all",