Fix misplaced cache option for the eslint-loader

Author: Lyrkan

lib/config-generator.js

@@ -235,7 +235,6 @@ class ConfigGenerator {
                 loader: 'eslint-loader',
                 exclude: /node_modules/,
                 enforce: 'pre',
-                cache: true,
                 options: eslintLoaderUtil.getOptions(this.webpackConfig)
             });
         }

lib/loaders/eslint.js

@@ -21,6 +21,7 @@ module.exports = {
         loaderFeatures.ensurePackagesExist('eslint');
 
         const eslintLoaderOptions = {
+            cache: true,
             parser: 'babel-eslint',
             emitWarning: true,
             'import/resolver': {

test/loaders/eslint.js

@@ -28,7 +28,7 @@ describe('loaders/eslint', () => {
         config.enableEslintLoader();
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(3);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
     });
 
     it('getOptions() with extra options', () => {
@@ -39,7 +39,7 @@ describe('loaders/eslint', () => {
 
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(5);
         expect(actualOptions.extends).to.equal('airbnb');
     });
 
@@ -51,7 +51,7 @@ describe('loaders/eslint', () => {
 
         const actualOptions = eslintLoader.getOptions(config);
 
-        expect(Object.keys(actualOptions)).to.have.lengthOf(3);
+        expect(Object.keys(actualOptions)).to.have.lengthOf(4);
         expect(actualOptions.emitWarning).to.equal(false);
     });