Uses ESLint jquery environment

yeoman · Aug 4, 2015 · 8c3d6f5 · 8c3d6f5
1 parent 53d1e29
commit 8c3d6f5
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/app/index.js b/app/index.js
@@ -127,6 +127,7 @@ module.exports = generators.Base.extend({
         {
           includeSass: this.includeSass,
           includeModernizr: this.includeModernizr,
+          includeJQuery: this.includeBootstrap || this.includeJQuery,
           testFramework: this.options['test-framework'],
           useBabel: this.options['babel']
         }

diff --git a/app/templates/_package.json b/app/templates/_package.json
@@ -47,7 +47,8 @@
   "eslintConfig": {
     "env": {
       "node": true,
-      "browser": true<% if (testFramework === 'mocha') { %>,
+      "browser": true<% if (includeJQuery) { %>,
+      "jquery": true<% } if (testFramework === 'mocha') { %>,
       "mocha": true<% } else if (testFramework === 'jasmine') { %>,
       "jasmine": true<% } %>
     },

diff --git a/test/jquery.js b/test/jquery.js
@@ -19,6 +19,10 @@ describe('jquery', function () {
     it('adds the bower dependency', function () {
       assert.fileContent('bower.json', '"jquery"');
     });
+
+    it('uses the eslint environment', function () {
+      assert.fileContent('package.json', '"jquery"');
+    });
   });
 
   describe('off', function () {
@@ -36,5 +40,9 @@ describe('jquery', function () {
     it('doesn\'t add the bower dependency', function () {
       assert.noFileContent('bower.json', '"jquery"');
     });
+
+    it('doesn\'nt uses the ESLint environment', function () {
+      assert.noFileContent('package.json', '"jquery"');
+    });
   });
 });