test: fix test for #33

fixtures/sandbox/package.json

@@ -2,9 +2,15 @@
   "name": "sandbox",
   "private": true,
   "dependencies": {
-    "enzyme": "3.0.0",
-    "enzyme-adapter-react-16": "1.1.1",
-    "react": "16.0.0",
-    "react-dom": "16.0.0"
+    "angular": "^1.7.2",
+    "enzyme": "^3.0.0",
+    "enzyme-adapter-react-16": "^1.1.1",
+    "eslint": "^4.19.1",
+    "eslint-config-airbnb": "^17.0.0",
+    "eslint-plugin-import": "^2.12.0",
+    "eslint-plugin-jsx-a11y": "^6.0.3",
+    "eslint-plugin-react": "^7.9.1",
+    "react": "^16.0.0",
+    "react-dom": "^16.0.0"
   }
 }

src/cli.js

@@ -76,6 +76,18 @@ if (program.args.length === 0) {
   process.exit(9);
 }
 
+// Make sure we're installing no more than one package
+if (program.args.length > 1) {
+  console.log(
+    `${
+      C.errorText
+    } Please specify ONE package at a time to install with peerDeps.`
+  );
+  // An exit code of "9" indicates an invalid argument
+  // See https://nodejs.org/api/process.html#process_exit_codes
+  process.exit(9);
+}
+
 // The first argument after the options is the name of the package
 const packageString = program.args[0];
 

src/cli.test.js

@@ -10,7 +10,9 @@ import test from "tape";
 function spawnCli(extraArgs) {
   return spawn(
     "node",
-    ["--require", "babel-register", "cli.js"].concat(extraArgs),
+    ["--require", "babel-register", path.join(__dirname, "cli.js")].concat(
+      extraArgs
+    ),
     { cwd: path.join(__dirname, "..", "fixtures", "sandbox") }
   );
 }
@@ -48,7 +50,11 @@ test("errors when more than one package is provided", t => {
     // We should be able to do t.equal(code, 1), but earlier Node versions
     // handle uncaught exceptions differently so we can't (0.10 returns 8,
     // 0.12 returns 9).
-    t.notEqual(code, 0, `errored, exit code was ${code}`);
+    t.notEqual(
+      code,
+      0,
+      `errored, exit code was ${code}.`
+    );
     t.end();
   });
 });
@@ -87,10 +93,15 @@ test("adds an explicit `--no-save` when using `--silent` with NPM", t => {
 });
 
 test("installs packages correctly even if package name ends with '-0'", t => {
-  const cli = spawnCli("enzyme-adapter-react-16@1.1.1");
-  cli.on("exit", code => {
-    t.equal(code, 0, `errored, exit code was ${code}`);
-    t.end();
+  const cli = spawnCli(["enzyme-adapter-react-16@1.1.1"]);
+    cli.on("exit", code => {
+      t.equal(
+        code,
+        0,
+        `errored, exit code was ${code}.`
+      );
+      t.end();
+    });
   });
 });