refactored all of the tests and test fixtures

Author: JamesMessinger

test/fixtures/bin/git

@@ -0,0 +1,15 @@
+'use strict';
+
+const fs = require('fs');
+const del = require('del');
+
+/**
+ * Clean the .tmp directory before each test
+ */
+beforeEach(() => {
+  // Delete the .tmp directory, if it exists
+  del.sync('test/.tmp');
+
+  // Re-create the .tmp directory
+  fs.mkdirSync('test/.tmp');
+});

test/fixtures/clean-temp-dir.js

@@ -0,0 +1,40 @@
+'use strict';
+
+const path = require('path');
+const spawnSync = require('child_process').spawnSync;
+
+const cliPath = path.resolve('bin/bump.js');
+
+module.exports = { exec };
+
+/**
+ * Runs bump with the given arguments.
+ *
+ * @param {string} args - The args to run bump with
+ */
+function exec (args) {
+  // Run bump
+  args = [cliPath].concat(args.split(' '));
+  let output = spawnSync('node', args, {
+    encoding: 'utf8',
+    cwd: path.resolve('test/.tmp'),
+  });
+
+  // Check for errors
+  if (output.error) {
+    throw output.error;
+  }
+
+  // Create an array containing each line of output
+  output.lines = [];
+  if (output.stdout) {
+    output.lines = output.stdout.split('\n');
+
+    let lastLine = output.lines[output.lines.length - 1];
+    if (lastLine === '') {
+      output.lines.pop();
+    }
+  }
+
+  return output;
+}

test/fixtures/cli.js

@@ -1,3 +1,3 @@
 This file is for testing the `--grep` option.
 
-If all goes well, version 1.2.3 and v1.2.3 should both get updated, but version 5.6.7 and v8.9.10 shouldn't.
+If all goes well, version 1.2.3 and v1.2.3 should both get updated, but version 5.6.7 and v8.9.10 should not be changed.

test/fixtures/files/README.md

@@ -1,8 +1,64 @@
-(function() {
-  'use strict';
+'use strict';
 
-  // This file is for testing the --grep option
-  // to make sure v1.2.3 gets replaced correctly
-  var version = '1.2.3';
+const fs = require('fs');
+const path = require('path');
 
-})();
+const srcDir = __dirname;
+const destDir = path.join(__dirname, '../../.tmp');
+
+const files = module.exports = {
+  /**
+   * Creates a file in the "test/.tmp" directory with the given contents
+   *
+   * @param {string} name - The file name (e.g. "package.json")
+   * @param {string|object} [contents] - The file contents
+   */
+  create (name, contents) {
+    if (typeof contents === 'object') {
+      contents = JSON.stringify(contents, null, 2);
+    }
+    fs.writeFileSync(path.join(destDir, name), contents);
+  },
+
+  /**
+   * Copies a file from the "test/fixtures/files" directory to the "test/.tmp" directory.
+   *
+   * @param {string} name - The name of the file to copy (e.g. "README.md", "script1.js")
+   */
+  copy (name) {
+    let contents = fs.readFileSync(path.join(srcDir, name), 'utf8');
+    files.create(name, contents);
+  },
+
+  /**
+   * Reads a file in the "test/.tmp" directory, and returns its contents as a string.
+   *
+   * @param {string} name - The file name (e.g. "README.md", "script1.js")
+   * @returns {object}
+   */
+  text (name) {
+    try {
+      return fs.readFileSync(path.join(destDir, name), 'utf8');
+    }
+    catch (e) {
+      return '';
+    }
+  },
+
+  /**
+   * Parses a JSON file in the "test/.tmp" directory, and returns its contents
+   * as a JavaScript object.
+   *
+   * @param {string} name - The file name (e.g. "package.json")
+   * @returns {object|undefined}
+   */
+  json (name) {
+    try {
+      let json = files.text(name);
+      return JSON.parse(json);
+    }
+    catch (e) {
+      return undefined;
+    }
+  },
+};

test/fixtures/files/index.js

@@ -0,0 +1,9 @@
+(function () {
+  'use strict';
+
+  // This file is for testing the --grep option
+  // to make sure v1.2.3 gets replaced correctly
+  let version = '1.2.3';
+  return version;
+
+}());

test/fixtures/files/script1.js

@@ -1,7 +1,7 @@
-(function() {
+(function () {
   'use strict';
 
   // This file should not get updated when the --grep option is used,
   // because version 3.2.1 and v8.9.10 don't match the old version number
 
-})();
+}());

test/fixtures/files/script.js test/fixtures/files/script2.jstest/fixtures/files/script.js renamed to test/fixtures/files/script2.js

@@ -0,0 +1,11 @@
+#!/usr/bin/env node
+
+/**
+ *
+ * This is a mock of the `git` CLI, for testing purposes.
+ * It just records the CLI arguments in a JSON file.
+ *
+ */
+
+const mocks = require('./index');
+mocks.record('git', process.argv.slice(2));