feat: reject in case of error

package.json

@@ -46,11 +46,18 @@
     "@stryker-mutator/jest-runner": "2.1.0",
     "coveralls": "3.0.7",
     "cz-conventional-changelog": "3.0.2",
-    "jest": "24.9.0",
+    "jest": "^24.9.0",
+    "jest-watch-typeahead": "^0.4.0",
     "semantic-release": "15.13.24"
   },
   "dependencies": {
     "enquirer": "^2.3.2",
     "npm": "^6.12.0"
+  },
+  "jest": {
+    "watchPlugins": [
+      "jest-watch-typeahead/filename",
+      "jest-watch-typeahead/testname"
+    ]
   }
 }

src/__snapshots__/index.error.integration.test.js.snap

@@ -0,0 +1,3 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Should throw an error if the project is not found 1`] = `"No NPM project found"`;

src/bin.js

@@ -1,4 +1,4 @@
 #!/usr/bin/env node
 "use strict";
 
-require("./index.js")(process.argv);
+require("./index.js")(process.argv).catch(console.error);

src/index.error.integration.test.js

@@ -0,0 +1,18 @@
+"use strict";
+jest.unmock("npm");
+jest.mock("npm", () => ({
+  load: jest.fn(callback => callback()),
+  config: {
+    sources: {
+      project: {
+        path: undefined
+      }
+    }
+  }
+}));
+
+const main = require("./index.js");
+
+it("Should throw an error if the project is not found", async () => {
+  await expect(main()).rejects.toMatchSnapshot();
+});

src/index.integration.test.js

@@ -2,14 +2,16 @@
 
 const npm = require("npm");
 const main = require("./index.js");
-const { readNpmScripts } = require("./read-npm-scripts.js");
+const { readNpmScripts, npmProjectLogPrefix } = require("./read-npm-scripts.js");
 
 it("Should work", async () => {
+  const spy = jest.spyOn(global.console, "log");
   const scripts = await readNpmScripts();
   const scriptNames = Object.keys(scripts);
   const argv = await main();
 
   expect(npm.load).toBeCalled();
+  expect(spy.mock.calls[0][0].startsWith(npmProjectLogPrefix)).toBe(true);
   expect(npm.run).toBeCalled();
   expect(scriptNames).toContain(npm.run.mock.calls[0][0][0]);
   expect(argv).toBe(npm.run.mock.calls[0][0]);

src/index.js

@@ -15,8 +15,7 @@ const main = argv => {
         npm.run(params);
         resolve(params);
       });
-    })
-    .catch(console.error);
+    });
 };
 
 module.exports = main;

src/read-npm-scripts.js

@@ -2,19 +2,21 @@
 
 const npm = require("npm");
 
+const npmProjectLogPrefix = "NPM project: ";
 const readNpmScripts = () =>
-  new Promise(resolve => {
+  new Promise((resolve, reject) => {
     npm.load(() => {
       if (!npm.config.sources.project.path) {
-        throw new Error("No NPM project found");
+        reject("No NPM project found");
       }
       const packageJsonPath = npm.config.sources.project.path.replace(".npmrc", "package.json");
-      console.log(`NPM project: ${packageJsonPath}`);
+      console.log(npmProjectLogPrefix + packageJsonPath);
       const packageJson = require(packageJsonPath);
       resolve(packageJson.scripts);
     });
   });
 
 module.exports = {
-  readNpmScripts
+  readNpmScripts,
+  npmProjectLogPrefix
 };

test-projects/import-nprr/package.json

@@ -11,6 +11,6 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "nprr": "file:../../nprr-0.0.0-development.tgz"
+    "nprr": "file:../../nprr-1.0.0.tgz"
   }
 }