Merge pull request #13 from JPeer264/fix/global-test

Fix/global test

Author: JPeer264

package.json

@@ -34,6 +34,15 @@
   "bugs": {
     "url": "https://github.com/JPeer264/node-semantic-git-commit-cli/issues"
   },
+  "keywords": [
+    "semantic",
+    "git",
+    "commits",
+    "cli",
+    "fast",
+    "enhance",
+    "workflow"
+  ],
   "homepage": "https://github.com/JPeer264/node-semantic-git-commit-cli#readme",
   "devDependencies": {
     "ava": "^0.18.2",

test/getConfig.js

@@ -8,6 +8,24 @@ import getConfig from '../lib/getConfig';
 const cwd = process.cwd();
 const homedir = os.homedir();
 const fixtures = path.join(cwd, 'test', 'fixtures');
+const randomString = Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 10);
+
+let globalExist = false;
+
+
+// rename global .sgcrc
+test.before(() => {
+  if (fs.existsSync(path.join(homedir, '.sgcrc'))) {
+    globalExist = true;
+    fs.renameSync(path.join(homedir, '.sgcrc'), path.join(homedir, `.sgcrc.${randomString}.back`));
+  }
+});
+
+test.after(() => {
+  if (globalExist) {
+    fs.renameSync(path.join(homedir, `.sgcrc.${randomString}.back`), path.join(homedir, '.sgcrc'));
+  }
+});
 
 test('read config from a specific path', (t) => {
   t.deepEqual(getConfig(path.join(fixtures, '.sgcrc')), json.readToObjSync(path.join(fixtures, '.sgcrc')));
@@ -20,31 +38,24 @@ test('read config from a .sgcrc_default', (t) => {
 test('read config from package.json', (t) => {
   const sgcrc = json.readToObjSync(path.join(fixtures, '.sgcrc'));
   const packageJson = json.readToObjSync(path.join(cwd, 'package.json'));
-  const randomString = Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 10);
+
   packageJson.sgc = sgcrc;
 
+  // manipulate local package
   fs.renameSync(path.join(cwd, 'package.json'), path.join(cwd, `package.json.${randomString}.back`));
   fs.writeFileSync(path.join(cwd, 'package.json'), JSON.stringify(packageJson));
+
   t.deepEqual(getConfig(), sgcrc);
+
+  // revert local package
   fs.removeSync(path.join(cwd, 'package.json'));
   fs.renameSync(path.join(cwd, `package.json.${randomString}.back`), path.join(cwd, 'package.json'));
 });
 
 test('read global config', (t) => {
-  let globalExist = false;
   const sgcrc = json.readToObjSync(path.join(fixtures, '.sgcrc'));
-  const randomString = Math.random().toString(36).replace(/[^a-z]+/g, '').substr(0, 10);
-
-  if (fs.existsSync(path.join(homedir, '.sgcrc'))) {
-    globalExist = true;
-    fs.renameSync(path.join(homedir, '.sgcrc'), path.join(homedir, `.sgcrc.${randomString}.back`));
-  }
 
   fs.writeFileSync(path.join(homedir, '.sgcrc'), JSON.stringify(sgcrc));
   t.deepEqual(getConfig(), sgcrc);
   fs.removeSync(path.join(homedir, '.sgcrc'));
-
-  if (globalExist) {
-    fs.renameSync(path.join(homedir, `.sgcrc.${randomString}.back`), path.join(homedir, '.sgcrc'));
-  }
 });