Split the mocks into separate files for initial setup and testing

Author: JamesMessinger

.mocharc.yml

@@ -3,6 +3,7 @@
 # https://github.com/mochajs/mocha/blob/master/example/config/.mocharc.yml
 
 spec:
+  - test/fixtures/setup-mocks.js
   - test/fixtures/mocha-hooks.js
   - test/specs/**/*.spec.js
 

test/fixtures/mocks/git

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-'use strict';
+"use strict";
 
 /**
  *
@@ -8,5 +8,5 @@
  *
  */
 
-const mocks = require('./index');
-mocks.record('git', process.argv.slice(2));
+const mocks = require("../../utils/mocks");
+mocks.record("git", process.argv.slice(2));

test/fixtures/mocks/npm

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-'use strict';
+"use strict";
 
 /**
  *
@@ -8,5 +8,5 @@
  *
  */
 
-const mocks = require('./index');
-mocks.record('npm', process.argv.slice(2));
+const mocks = require("../../utils/mocks");
+mocks.record("npm", process.argv.slice(2));

test/fixtures/setup-mocks.js

@@ -0,0 +1,30 @@
+"use strict";
+
+const fs = require("fs");
+const path = require("path");
+
+const mocksDir = path.resolve("test", "fixtures", "mocks");
+
+// Ensure that the mock binaries are executable
+fs.chmodSync(path.join(mocksDir, "git"), "0777");
+fs.chmodSync(path.join(mocksDir, "npm"), "0777");
+
+// Inject our mocks directory path into the PATH variable,
+// so that version-bump-prompt runs our mock `git` and `npm` binaries
+// instead of the real ones.
+let otherPaths = getEnvPath();
+process.env.PATH = mocksDir + path.delimiter + otherPaths; // eslint-disable-line no-path-concat
+
+
+/**
+ * Returns the PATH environment variable, case-insensitively
+ */
+function getEnvPath () {
+  let keys = Object.keys(process.env);
+
+  for (let key of keys) {
+    if (key.toUpperCase() === "PATH") {
+      return process.env[key];
+    }
+  }
+}

test/utils/mocks.js

@@ -1,21 +1,7 @@
 "use strict";
 
-const fs = require("fs");
-const path = require("path");
 const files = require("./files");
 
-const mocksDir = path.resolve("test", "fixtures", "mocks");
-
-// Ensure that the mock binaries are executable
-fs.chmodSync(path.join(mocksDir, "git"), "0777");
-fs.chmodSync(path.join(mocksDir, "npm"), "0777");
-
-// Inject our mocks directory path into the PATH variable,
-// so that version-bump-prompt runs our mock `git` and `npm` binaries
-// instead of the real ones.
-let otherPaths = getEnvPath();
-process.env.PATH = mocksDir + path.delimiter + otherPaths; // eslint-disable-line no-path-concat
-
 const mocks = module.exports = {
   /**
    * Returns information about each time `git` was executed.
@@ -70,20 +56,6 @@ const mocks = module.exports = {
   },
 };
 
-
-/**
- * Returns the PATH environment variable, case-insensitively
- */
-function getEnvPath () {
-  let keys = Object.keys(process.env);
-
-  for (let key of keys) {
-    if (key.toUpperCase() === "PATH") {
-      return process.env[key];
-    }
-  }
-}
-
 /**
  * Adds quotes around an argument if it contains whitespace characters
  *