Fix: requireSpacesInForStatement account for parenthesizedExpression

Fixes #1332

Author: hzoo

lib/rules/require-spaces-in-for-statement.js

@@ -41,6 +41,7 @@
  */
 
 var assert = require('assert');
+var TokenCategorizer = require('../token-categorizer');
 
 module.exports = function() {};
 
@@ -60,8 +61,15 @@ module.exports.prototype = {
         file.iterateNodesByType('ForStatement', function(node) {
             if (node.test) {
                 var testToken = file.getFirstNodeToken(node.test);
+                var prevToken = file.getPrevToken(testToken);
+
+                if (prevToken.value === '(' &&
+                    TokenCategorizer.categorizeOpenParen(prevToken, file) === 'ParenthesizedExpression') {
+                    testToken = prevToken;
+                    prevToken = file.getPrevToken(prevToken);
+                }
                 errors.assert.spacesBetween({
-                    token: file.getPrevToken(testToken),
+                    token: prevToken,
                     nextToken: testToken,
                     exactly: 1,
                     message: 'One space required after semicolon'

test/specs/rules/require-spaces-in-for-statement.js

@@ -46,5 +46,10 @@ describe('rules/require-spaces-in-for-statement', function() {
             expect(checker.checkString('for(i=0;i<l;){}'))
               .to.have.one.validation.error.from('requireSpacesInForStatement');
         });
+
+        it('should not report with spaces with parenthesizedExpression #1332', function() {
+            var parenthesizedExpression = 'for (var i = 0; (!reachEnd && (i < elementsToMove)); i++) {}';
+            expect(checker.checkString(parenthesizedExpression)).to.have.no.errors();
+        });
     });
 });