Dry cleaning tests ✨

Author: rmariuzzo

.jshintrc

@@ -12,13 +12,13 @@
   "node"    : true,
   "browser" : true,
   "globals" : {
-    "it"         : false,
-    "xit"        : false,
-    "describe"   : false,
-    "xdescribe"  : false,
-    "beforeEach" : false,
-    "afterEach"  : false,
-    "expect"     : false,
-    "spyOn"      : false
+    "it"         : true,
+    "xit"        : true,
+    "describe"   : true,
+    "xdescribe"  : true,
+    "beforeEach" : true,
+    "afterEach"  : true,
+    "expect"     : true,
+    "spyOn"      : true
   }
-}
+}

tests/.jshintrc

@@ -0,0 +1,27 @@
+{
+    "curly": true,
+    "eqeqeq": true,
+    "immed": true,
+    "latedef": true,
+    "newcap": true,
+    "noarg": true,
+    "sub": true,
+    "undef": true,
+    "boss": true,
+    "eqnull": true,
+    "node": true,
+    "browser": true,
+    "globals": {
+        "it": true,
+        "xit": true,
+        "describe": true,
+        "xdescribe": true,
+        "beforeEach": true,
+        "afterEach": true,
+        "expect": true,
+        "spyOn": true,
+        "jasmine": true,
+        "$": true,
+        "loadFixtures": true
+    }
+}

tests/specs/jquery_checkboxes_check_spec.js

@@ -1,30 +1,27 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The `check` method', function() {
+describe('The `check` method', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         loadFixtures('unchecked.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes').check).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         loadFixtures('unchecked.html');
         var el = $('body').checkboxes();
         expect(typeof el.data('checkboxes').check).toBe('function');
     });
 
-    it('should check all checkboxes in context', function() {
+    it('should check all checkboxes in context', function () {
         loadFixtures('unchecked.html');
         var el = $('body').checkboxes('check');
         expect(el.find(':checkbox')).toBeChecked();
     });
 
-    it('should check specified checkbox', function() {
+    it('should check specified checkbox', function () {
         // Check first checkbox.
         loadFixtures('unchecked.html');
         var el = $(':checkbox:first').parent().checkboxes('check');
@@ -33,16 +30,16 @@ describe('The `check` method', function() {
 
         // Check last checkbox.
         loadFixtures('unchecked.html');
-        var el = $(':checkbox:last').parent().checkboxes('check');
+        el = $(':checkbox:last').parent().checkboxes('check');
         expect(el.find(':checkbox:last')).toBeChecked();
         expect(el.find(':checkbox:not(:last)')).toBeChecked();
     });
 
-    it('should check all checkboxes in context, but no disabled ones', function() {
+    it('should check all checkboxes in context, but no disabled ones', function () {
         loadFixtures('unchecked.html');
         var el = $('body').checkboxes('check');
         expect(el.find(':checkbox:disabled')).not.toBeChecked();
         expect(el.find(':checkbox:not(:disabled)')).toBeChecked();
     });
 
-});
+});

tests/specs/jquery_checkboxes_max_spec.js

@@ -1,31 +1,28 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The `max` method', function() {
+describe('The `max` method', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes').max).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(typeof el.data('checkboxes').max).toBe('function');
     });
 
-    it('should limit the number of checked checkboxes in context', function() {
+    it('should limit the number of checked checkboxes in context', function () {
         loadFixtures('unchecked.html');
         var context = $('body');
         context.checkboxes('max', 3);
-        context.find(':checkbox').each(function(i) {
+        context.find(':checkbox').each(function (i) {
             $(this).click();
             expect($(this).prop('checked')).toBe(i < 3); // The first three should be checked.
         });
     });
 
-});
+});

tests/specs/jquery_checkboxes_range_spec.js

@@ -1,21 +1,18 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The `range` method', function() {
+describe('The `range` method', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes').range).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(typeof el.data('checkboxes').range).toBe('function');
     });
 
-});
+});

tests/specs/jquery_checkboxes_spec.js

@@ -1,29 +1,26 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The jQuery checkboxes plugin', function() {
+describe('The jQuery checkboxes plugin', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         expect($.fn.checkboxes).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         expect(typeof $.fn.checkboxes).toBe('function');
     });
 
-    it('should create an instance in $.data', function() {
+    it('should create an instance in $.data', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes')).toBeDefined();
     });
 
-    it('should not create an instance in $.data with empty element set', function() {
+    it('should not create an instance in $.data with empty element set', function () {
         loadFixtures('checked.html');
         var el = $().checkboxes();
         expect(el.data('checkboxes')).not.toBeDefined();
     });
 
-});
+});

tests/specs/jquery_checkboxes_toggle_spec.js

@@ -1,33 +1,30 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The `toggle` method', function() {
+describe('The `toggle` method', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         loadFixtures('mixed.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes').toggle).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         loadFixtures('mixed.html');
         var el = $('body').checkboxes();
         expect(typeof el.data('checkboxes').toggle).toBe('function');
     });
 
-    it('should toggle all checkboxes in context', function() {
+    it('should toggle all checkboxes in context', function () {
         loadFixtures('mixed.html');
         var original = $('body').clone();
         var modified = $('body').checkboxes('toggle');
-        modified.find(':checkbox:not(:disabled)').each(function(i) {
+        modified.find(':checkbox:not(:disabled)').each(function (i) {
             expect($(this).prop('checked')).not.toBe(original.find(':checkbox:not(:disabled)').eq(i).prop('checked'));
         });
     });
 
-    it('should toggle specified checkbox', function() {
+    it('should toggle specified checkbox', function () {
         // Check first checkbox.
         loadFixtures('mixed.html');
         var original = $(':checkbox:first').clone();
@@ -41,13 +38,13 @@ describe('The `toggle` method', function() {
         expect(modified.find(':checkbox:last').prop('checked')).not.toBe(original.prop('checked'));
     });
 
-    it('should toggle all checkboxes in context, but no disabled ones', function() {
+    it('should toggle all checkboxes in context, but no disabled ones', function () {
         loadFixtures('mixed.html');
         var original = $('body').clone();
         var modified = $('body').checkboxes('toggle');
-        modified.find(':checkbox:disabled').each(function(i) {
+        modified.find(':checkbox:disabled').each(function (i) {
             expect($(this).prop('checked')).toBe(original.find(':checkbox:disabled').eq(i).prop('checked'));
         });
     });
 
-});
+});

tests/specs/jquery_checkboxes_uncheck_spec.js

@@ -1,30 +1,27 @@
-/* globals $, jasmine, loadFixtures */
-
-'use strict';
-
 jasmine.getFixtures().fixturesPath = 'tests/fixtures';
 
-describe('The `uncheck` method', function() {
+describe('The `uncheck` method', function () {
+    'use strict';
 
-    it('should exists', function() {
+    it('should exists', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(el.data('checkboxes').uncheck).toBeDefined();
     });
 
-    it('should be a function', function() {
+    it('should be a function', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes();
         expect(typeof el.data('checkboxes').uncheck).toBe('function');
     });
 
-    it('should uncheck all checkboxes in context', function() {
+    it('should uncheck all checkboxes in context', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes('uncheck');
         expect(el.find(':checkbox:not(:disabled)')).not.toBeChecked();
     });
 
-    it('should uncheck specified checkbox', function() {
+    it('should uncheck specified checkbox', function () {
         // Check first checkbox.
         loadFixtures('checked.html');
         var el = $(':checkbox:first').parent().checkboxes('uncheck');
@@ -33,16 +30,16 @@ describe('The `uncheck` method', function() {
 
         // Check last checkbox.
         loadFixtures('checked.html');
-        var el = $(':checkbox:last').parent().checkboxes('uncheck');
+        el = $(':checkbox:last').parent().checkboxes('uncheck');
         expect(el.find(':checkbox:last')).not.toBeChecked();
         expect(el.find(':checkbox:not(:last)')).toBeChecked();
     });
 
-    it('should check all checkboxes in context, but no disabled ones', function() {
+    it('should check all checkboxes in context, but no disabled ones', function () {
         loadFixtures('checked.html');
         var el = $('body').checkboxes('uncheck');
         expect(el.find(':checkbox:disabled')).toBeChecked();
         expect(el.find(':checkbox:not(:disabled)')).not.toBeChecked();
     });
 
-});
+});