disallowSpacesInside*Brackets: 100% code coverage

Author: mikesherov

test/rules/disallow-spaces-inside-array-brackets.js

@@ -9,6 +9,14 @@ describe('rules/disallow-spaces-inside-array-brackets', function() {
         checker.registerDefaultRules();
     });
 
+    describe('invalid options', function() {
+        it('should throw when given an number', function() {
+            assert.throws(function() {
+                checker.configure({ disallowSpacesInsideArrayBrackets: 2 });
+            });
+        });
+    });
+
     describe('true value', function() {
         beforeEach(function() {
             checker.configure({ disallowSpacesInsideArrayBrackets: true });
@@ -142,6 +150,15 @@ describe('rules/disallow-spaces-inside-array-brackets', function() {
     });
 
     describe('exceptions', function() {
+        it('should act like "true" when allExcept is false', function() {
+            checker.configure({
+                disallowSpacesInsideArrayBrackets: {
+                    allExcept: false
+                }
+            });
+            assert(checker.checkString('var x = [ 1];').getErrorCount() === 1);
+        });
+
         it('should not report missing space for parentheses', function() {
             checker.configure({
                 disallowSpacesInsideArrayBrackets: {

test/rules/disallow-spaces-inside-brackets.js

@@ -9,6 +9,14 @@ describe('rules/disallow-spaces-inside-brackets', function() {
         checker.registerDefaultRules();
     });
 
+    describe('invalid options', function() {
+        it('should throw when given an number', function() {
+            assert.throws(function() {
+                checker.configure({ disallowSpacesInsideBrackets: 2 });
+            });
+        });
+    });
+
     describe('true value', function() {
         beforeEach(function() {
             checker.configure({ disallowSpacesInsideBrackets: true });
@@ -67,6 +75,15 @@ describe('rules/disallow-spaces-inside-brackets', function() {
     });
 
     describe('exceptions', function() {
+        it('should act like "true" when allExcept is false', function() {
+            checker.configure({
+                disallowSpacesInsideBrackets: {
+                    allExcept: false
+                }
+            });
+            assert(checker.checkString('var x = [ 1];').getErrorCount() === 1);
+        });
+
         it('should not report missing space for parentheses', function() {
             checker.configure({
                 disallowSpacesInsideBrackets: {

test/rules/disallow-spaces-inside-object-brackets.js

@@ -8,6 +8,14 @@ describe('rules/disallow-spaces-inside-object-brackets', function() {
         checker.registerDefaultRules();
     });
 
+    describe('invalid options', function() {
+        it('should throw when given an number', function() {
+            assert.throws(function() {
+                checker.configure({ disallowSpacesInsideObjectBrackets: 2 });
+            });
+        });
+    });
+
     describe('true value', function() {
         beforeEach(function() {
             checker.configure({ disallowSpacesInsideObjectBrackets: true });
@@ -87,6 +95,15 @@ describe('rules/disallow-spaces-inside-object-brackets', function() {
     });
 
     describe('exceptions', function() {
+        it('should act like "true" when allExcept is false', function() {
+            checker.configure({
+                disallowSpacesInsideObjectBrackets: {
+                    allExcept: false
+                }
+            });
+            assert(checker.checkString('var x = { a: 1};').getErrorCount() === 1);
+        });
+
         it('should report for function', function() {
             checker.configure({
                 disallowSpacesInsideObjectBrackets: {