Correct error message for "requireSpaceAfterKeywords" rule

qfox · Oct 19, 2014 · 9af1e27 · 9af1e27
1 parent 76be229
commit 9af1e27
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/rules/require-space-after-keywords.js b/lib/rules/require-space-after-keywords.js
@@ -54,7 +54,7 @@ module.exports.prototype = {
                     if (nextToken.type !== 'Punctuator' || nextToken.value !== ';') {
                         errors.add(
                             util.format.apply(null,
-                                diff === 1 ?
+                                diff === 0 ?
                                     [texts[0], token.value] :
                                     [texts[1], diff, token.value]
                             ),

diff --git a/test/rules/require-space-after-keywords.js b/test/rules/require-space-after-keywords.js
@@ -73,7 +73,11 @@ describe('rules/require-space-after-keywords', function() {
 
     it('should trigger different error for comments with more than one space', function() {
         checker.configure({ requireSpaceAfterKeywords: ['if'] });
-        assert(checker.checkString('if(x) { x++; }').getErrorCount() === 1);
+
+        var errors = checker.checkString('if  /**/(x) {}');
+        var error = errors.getErrorList()[0];
+
+        assert(errors.explainError(error).indexOf('Should be one space instead of 2, after "if"') === 0);
     });
 
     it('should report on all spaced keywords if a value of true is supplied', function() {