fix eslint-airbnb-base linting for tests

SimulatedGREG · SimulatedGREG · commit 491a2cfce264 · 2016-11-01T01:31:02.000-05:00
diff --git a/docs/end-to-end_testing.md b/docs/end-to-end_testing.md
@@ -1,6 +1,6 @@
 # End-to-end testing
 
-electron-vue makes use of [Spectron](http://electron.atom.io/spectron/) and the [Mocha](https://mochajs.org/) test framework for end-to-end testing.
+electron-vue makes use of [Spectron](http://electron.atom.io/spectron/) and the [Mocha](https://mochajs.org/) (with [Chai](http://chaijs.com/)) test framework for end-to-end testing. Mocha & Chai APIs, including `expect`, `should`, and `assert`, are made available in global scope.
 
 ### Running tests
 ```bash
@@ -19,7 +19,7 @@ my-project
 │  │  ├─ specs
 │  │  │  └─ Launch.spec.js
 │  │  ├─ index.js
-│  │  └─ utils.js
+└─ └─ └─ utils.js
 ```
 
 **For the most part, you can ignore `index.js` and focus solely on writing `specs/`.**
diff --git a/docs/unittesting.md b/docs/unittesting.md
@@ -18,7 +18,7 @@ my-project
 │  │  ├─ specs
 │  │  │  └─ LandingPageView.spec.js
 │  │  ├─ index.js
-│  │  └─ karma.conf.js
+└─ └─ └─ karma.conf.js
 ```
 
 **For the most part, you can ignore both `index.js` and `karma.conf.js` and focus solely on writing `specs/`.**
@@ -33,4 +33,4 @@ This is the entry file used by `karma-webpack`. The purpose of this file is to g
 Here you will find the actual `karma` configuration, set up with spec/coverage reporters. Further customization can be made in accordance to the [official karma documentation](http://karma-runner.github.io/1.0/config/configuration-file.html).
 
 ### Mocking dependencies
-electron-vue comes with [`inject-loader`](https://github.com/plasticine/inject-loader) installed by default. For usage with Vue component files see [`vue-loader` docs on testing with mocks](http://vue-loader.vuejs.org/en/workflow/testing-with-mocks.html).
+electron-vue comes with [`inject-loader`](https://github.com/plasticine/inject-loader) installed by default. For usage with Vue component files see [`vue-loader` docs on testing with mocks](http://vue-loader.vuejs.org/en/workflow/testing-with-mocks.html).
diff --git a/template/.eslintignore b/template/.eslintignore
@@ -1,3 +1,8 @@
 app/node_modules/**
 app/dist/**
 app/electron.js
+{{#testing unit e2e}}
+test/unit/coverage/**
+test/unit/*.js
+test/e2e/*.js
+{{/testing}}
diff --git a/template/package.json b/template/package.json
@@ -15,7 +15,7 @@
     {{/if}}
     {{#if eslint}}
     "lint": "eslint --ext .js,.vue -f ./node_modules/eslint-friendly-formatter app",
-    "lint:fix": "eslint --ext .js,.vue -f ./node_modules/eslint-friendly-formatter --fix app",
+    "lint:fix": "eslint --ext .js,.vue -f ./node_modules/eslint-friendly-formatter --fix app{{#testing unit e2e}} test{{/testing}}",
     {{/if}}
     "pack": "cross-env NODE_ENV=production webpack -p --progress --colors",
     {{#testing unit e2e}}
diff --git a/template/test/.eslintrc b/template/test/.eslintrc
@@ -0,0 +1,14 @@
+{
+  "env": {
+    "mocha": true
+  },
+  "globals": {
+    "assert": true,
+    "expect": true,
+    "should": true
+  }{{#if eslint}}{{#if_eq eslintConfig 'airbnb'}},
+  "rules": {
+    "func-names": 0,
+    "prefer-arrow-callback": 0
+  }{{/if_eq}}{{/if}}
+}
diff --git a/template/test/unit/.eslintrc b/template/test/unit/.eslintrc
diff --git a/template/test/unit/index.js b/template/test/unit/index.js
@@ -1,5 +1,3 @@
-'use strict'
-
 // require all test files (files that ends with .spec.js)
 const testsContext = require.context('./specs', true, /\.spec$/)
 testsContext.keys().forEach(testsContext)

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,3 @@`
`1`		`-'use strict'`
`2`		`-`
`3`	`1`	`// require all test files (files that ends with .spec.js)`
`4`	`2`	`const testsContext = require.context('./specs', true, /\.spec$/)`
`5`	`3`	`testsContext.keys().forEach(testsContext)`