eslint --fix on test folder

Author: ricardo-devis-agullo

.eslintrc.json

@@ -3,7 +3,8 @@
   "env": {
     "browser": true,
     "es6": true,
-    "node": true
+    "node": true,
+    "mocha": true
   },
   "extends": [
     "eslint:recommended",

test/acceptance/registry-ui.js

@@ -6,7 +6,10 @@ const path = require('path');
 const request = require('minimal-request');
 
 describe('registry (ui interface)', () => {
-  let registry, result, error, headers;
+  let registry;
+  let result;
+  let error;
+  let headers;
 
   const next = done => (e, r, d) => {
     error = e;

test/acceptance/registry.js

@@ -7,7 +7,11 @@ const request = require('minimal-request');
 const _ = require('lodash');
 
 describe('registry', () => {
-  let registry, result, error, headers, status;
+  let registry;
+  let result;
+  let error;
+  let headers;
+  let status;
 
   const oc = require('../../dist/index');
 

test/unit/cli-domain-clean.js

@@ -23,7 +23,8 @@ describe('cli : domain : clean', () => {
 
   describe('when fetching the list of folders to clean', () => {
     describe('happy path', () => {
-      let error, result;
+      let error;
+      let result;
       beforeEach(done => {
         const clean = initialize();
         clean.fetchList('my-components-folder', (err, res) => {
@@ -59,7 +60,8 @@ describe('cli : domain : clean', () => {
 
   describe('when removing the folders to clean', () => {
     describe('happy path', () => {
-      let error, removeMock;
+      let error;
+      let removeMock;
       beforeEach(done => {
         removeMock = sinon.stub().yields(null, 'ok');
         const clean = initialize({ removeMock });
@@ -80,7 +82,8 @@ describe('cli : domain : clean', () => {
     });
 
     describe('fs.remove error', () => {
-      let error, removeMock;
+      let error;
+      let removeMock;
       beforeEach(done => {
         removeMock = sinon.stub().yields(new Error('nope'));
         const clean = initialize({ removeMock });

test/unit/cli-domain-get-components-by-dir.js

@@ -40,7 +40,8 @@ const executeComponentsListingByDir = function (local, callback) {
 
 describe('cli : domain : get-components-by-dir', () => {
   describe('when getting components from dir', () => {
-    let error, result;
+    let error;
+    let result;
     beforeEach(done => {
       const data = initialise();
 
@@ -81,7 +82,8 @@ describe('cli : domain : get-components-by-dir', () => {
   });
 
   describe('when reading a broken package.json', () => {
-    let error, result;
+    let error;
+    let result;
     beforeEach(done => {
       const data = initialise();
 
@@ -109,7 +111,8 @@ describe('cli : domain : get-components-by-dir', () => {
   });
 
   describe('when finds no components', () => {
-    let error, result;
+    let error;
+    let result;
     beforeEach(done => {
       const data = initialise();
 

test/unit/cli-domain-get-mocked-plugins.js

@@ -14,7 +14,8 @@ describe('cli : domain : get-mocked-plugins', () => {
   };
 
   const logMock = { err: _.noop, log: _.noop, ok: _.noop, warn: _.noop };
-  let fsMock, getMockedPlugins;
+  let fsMock;
+  let getMockedPlugins;
 
   const initialise = function (fs, pathJoinStub) {
     fsMock = _.extend(
@@ -120,8 +121,8 @@ describe('cli : domain : get-mocked-plugins', () => {
         }
       };
 
-      const readMock = sinon.stub(),
-        existsMock = sinon.stub();
+      const readMock = sinon.stub();
+      const existsMock = sinon.stub();
 
       readMock.withArgs('/root/components/oc.json').returns(ocJsonComponent);
       readMock.withArgs('/root/oc.json').returns(ocJsonRoot);

test/unit/cli-domain-handle-dependencies-ensure-compiler-is-declared-as-devDependency.js

@@ -6,7 +6,8 @@ describe('cli : domain : handle-dependencies : ensure-compiler-is-declared-as-de
   const ensure =
     require('../../dist/cli/domain/handle-dependencies/ensure-compiler-is-declared-as-devDependency').default;
   describe('when compiler is declared as devDependency', () => {
-    let error, result;
+    let error;
+    let result;
     beforeEach(done => {
       ensure(
         {

test/unit/cli-domain-handle-dependencies-get-compiler.js

@@ -6,7 +6,9 @@ const sinon = require('sinon');
 const _ = require('lodash');
 
 describe('cli : domain : handle-dependencies : get-compiler', () => {
-  let cleanRequireStub, error, installCompilerStub;
+  let cleanRequireStub;
+  let error;
+  let installCompilerStub;
   const execute = (opts, done) => {
     done = done || opts;
     const compilerVersion = opts.compilerVersionEmpty ? '' : '1.2.3';

test/unit/cli-domain-handle-dependencies-install-compiler.js

@@ -5,8 +5,12 @@ const injectr = require('injectr');
 const sinon = require('sinon');
 
 describe('cli : domain : handle-dependencies : install-compiler', () => {
-  let cleanRequireStub, installDependencyMock, isTemplateValidStub, loggerMock;
-  let error, result;
+  let cleanRequireStub;
+  let installDependencyMock;
+  let isTemplateValidStub;
+  let loggerMock;
+  let error;
+  let result;
 
   const initialise = (options, done) => {
     cleanRequireStub = sinon.stub().returns({ theCompiler: true });

test/unit/cli-domain-handle-dependencies-install-missing-dependencies.js

@@ -5,7 +5,8 @@ const injectr = require('injectr');
 const sinon = require('sinon');
 
 describe('cli : domain : handle-dependencies : install-missing-dependencies', () => {
-  let error, logger;
+  let error;
+  let logger;
   const initialise = (options, done) => {
     const { dependencies, stubs } = options;
     logger = {
@@ -32,7 +33,8 @@ describe('cli : domain : handle-dependencies : install-missing-dependencies', ()
   };
 
   describe('when there is no missing dependency', () => {
-    let dependencies, stubs;
+    let dependencies;
+    let stubs;
     beforeEach(done => {
       stubs = {
         getMissingDependencies: sinon.stub().returns([]),
@@ -53,7 +55,8 @@ describe('cli : domain : handle-dependencies : install-missing-dependencies', ()
   });
 
   describe('when there are missing dependencies and install succeeds', () => {
-    let dependencies, stubs;
+    let dependencies;
+    let stubs;
 
     beforeEach(done => {
       stubs = {
@@ -95,7 +98,8 @@ describe('cli : domain : handle-dependencies : install-missing-dependencies', ()
   });
 
   describe('when there are missing dependencies and install fails', () => {
-    let dependencies, stubs;
+    let dependencies;
+    let stubs;
 
     beforeEach(done => {
       stubs = {
@@ -130,7 +134,8 @@ describe('cli : domain : handle-dependencies : install-missing-dependencies', ()
   });
 
   describe('when there are missing dependencies and install succeeds but the dependencies are still not requireable', () => {
-    let dependencies, stubs;
+    let dependencies;
+    let stubs;
 
     beforeEach(done => {
       stubs = {