Don't set ESLint testing framework environment

This is only needed if no-undef rule is on, even in that case there's a
cleaner way, which is described in the Getting Started guide.

app/templates/gulpfile.js

@@ -43,28 +43,17 @@ gulp.task('scripts', () => {
 function lint(files, options) {
   return gulp.src(files)
     .pipe(reload({stream: true, once: true}))
-    .pipe($.eslint(options))
+    .pipe($.eslint({ fix: true }))
     .pipe($.eslint.format())
     .pipe($.if(!browserSync.active, $.eslint.failAfterError()));
 }
 
 gulp.task('lint', () => {
-  return lint('app/scripts/**/*.js', {
-    fix: true
-  })
+  return lint('app/scripts/**/*.js')
     .pipe(gulp.dest('app/scripts'));
 });
 gulp.task('lint:test', () => {
-  return lint('test/spec/**/*.js', {
-    fix: true,
-    env: {
-<% if (testFramework === 'mocha') { -%>
-      mocha: true
-<% } else if (testFramework === 'jasmine') { -%>
-      jasmine: true
-<% } -%>
-    }
-  })
+  return lint('test/spec/**/*.js')
     .pipe(gulp.dest('test/spec'));
 });
 

docs/README.md

@@ -52,7 +52,27 @@ $ gulp --tasks
 
 ## Gulp plugins
 
-As you might have noticed, gulp plugins (the ones that begin with `gulp-`) don't have to be `require()`'d. They are automatically picked up by [gulp-load-plugins][plugins] and available through the `$` variable.
+Gulp plugins (the ones that begin with `gulp-`) don't have to be `require()`'d. They are automatically picked up by [gulp-load-plugins][plugins] and available through the `$` variable.
+
+## Linting
+
+We use ESLint for linting JavaScript code. You can define rules in your `package.json` under the `"eslint"` field. Alternatively, you can add an `.eslintrc` file to your project root, where you can [configure][eslint-config] ESLint using JavaScript, JSON or YAML.
+
+### The `no-undef` rule and tests
+
+The ESLint rule [`no-undef`] will warn about usage of explicitly undeclared variables and functions. Because our tests use global functions like `describe` and `it` (defined by the testing framework), ESLint will consider those as warnings.
+
+Luckily, the fix is easy—add an `.eslintrc` file to the `test/spec` directory and let ESLint know about your testing framework. For example, if you're using Mocha, add this to `.eslintrc`:
+
+```json
+{
+  "env": {
+    "mocha": true
+  }
+}
+```
+
+Configuration from this `.eslintrc` will merge with your project-wide configuration.
 
 ## Serve
 
@@ -112,12 +132,14 @@ These are a bit tricky, as they can't be automatically injected. Ideally you wou
 
 If there's a problem, it's usually with the `main` field, which wiredep uses to wire up assets. Fortunately you can always [override][override] these fields and send a pull request to that component's repository, fixing their `bower.json` :wink:
 
-[gulp]:       https://github.com/gulpjs/gulp
-[gulp-docs]:  https://github.com/gulpjs/gulp/blob/master/docs/README.md
-[yo]:         https://github.com/yeoman/yo
+[gulp]: https://github.com/gulpjs/gulp
+[gulp-docs]: https://github.com/gulpjs/gulp/blob/master/docs/README.md
+[yo]: https://github.com/yeoman/yo
 [LiveReload]: https://github.com/intesso/connect-livereload
-[plugins]:    https://github.com/jackfranklin/gulp-load-plugins
-[calc]:       https://github.com/postcss/postcss-calc
-[wiredep]:    https://github.com/taptapship/wiredep
-[replace]:    https://github.com/lazd/gulp-replace
-[override]:   https://github.com/taptapship/wiredep#bower-overrides
+[plugins]: https://github.com/jackfranklin/gulp-load-plugins
+[eslint-config]: http://eslint.org/docs/user-guide/configuring
+[`no-undef`]: http://eslint.org/docs/rules/no-undef
+[calc]: https://github.com/postcss/postcss-calc
+[wiredep]: https://github.com/taptapship/wiredep
+[replace]: https://github.com/lazd/gulp-replace
+[override]: https://github.com/taptapship/wiredep#bower-overrides

test/test-framework.js

@@ -11,10 +11,6 @@ describe('test framework', () => {
         .on('end', done);
     });
 
-    it('uses the correct ESLint environment', () => {
-      assert.fileContent('gulpfile.js', 'mocha: true');
-    });
-
     it('generates the expected fixture', () => {
       assert.fileContent('test/index.html', 'mocha');
     });
@@ -28,10 +24,6 @@ describe('test framework', () => {
         .on('end', done);
     });
 
-    it('uses the correct ESLint environment', () => {
-      assert.fileContent('gulpfile.js', 'jasmine: true');
-    });
-
     it('generates the expected fixture', () => {
       assert.fileContent('test/index.html', 'jasmine');
     });