fix: gte 100 exiting with 1

closes #122
simonecorsi · Jan 10, 2023 · 79e70ac · 79e70ac
1 parent c61dac9
commit 79e70ac
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/bin/cmd.js b/bin/cmd.js
@@ -19,7 +19,7 @@ let min = 0;
 if (gte) {
   const [, value] = gte.split("=");
   min = !isNaN(parseInt(value)) ? parseFloat(value) : 0;
-  process.exitCode = result > min ? 0 : 1;
+  process.exitCode = result >= min ? 0 : 1;
   returnVal = "";
 }
 

diff --git a/tests/main.spec.js b/tests/main.spec.js
@@ -1,7 +1,8 @@
-import total from "../src/index.js";
+// eslint-disable-next-line node/no-missing-import
 import test from "ava";
+import { exec as ex } from "child_process";
+import total from "../src/index.js";
 import util from "util";
-import { exec as ex } from "node:child_process";
 
 const exec = util.promisify(ex);
 
@@ -33,6 +34,12 @@ test("Should exit(0) if greater than equal then 90", async (t) => {
   t.is(stderr, "");
 });
 
+test("Should exit(1) if checking for --gte=100", async (t) => {
+  await t.throwsAsync(
+    exec("node ./bin/cmd.js ./tests/fixtures/lcov.info --gte=100")
+  );
+});
+
 test("Should error on file missing", async (t) => {
   await t.throwsAsync(() => exec("node ./bin/cmd.js"));
 });