Improve requireSpaceBeforeBinaryOperators rule

Add check for assignment to requireSpaceBeforeBinaryOperators rule

Author: markelog

lib/rules/require-space-before-binary-operators.js

@@ -38,17 +38,18 @@ module.exports.prototype = {
             }
         });
 
-        // Comma
-        if (operators[ ',']) {
+        // Comma and assignment
+        if (operators[','] || operators['=']) {
             file.iterateTokensByType('Punctuator', function(token, i, tokens) {
-                if (token.value !== ',') {
+                var operator = token.value;
+                if (operator !== ',' && operator !== '=') {
                     return;
                 }
 
                 var prevToken = tokens[i - 1];
                 if (prevToken && prevToken.range[1] === token.range[0]) {
                     errors.add(
-                        'Operator ' + token.value + ' should not stick to preceding expression',
+                        'Operator ' + operator + ' should not stick to preceding expression',
                         token.loc.start
                     );
                 }

test/rules/require-space-before-binary-operators.js

@@ -45,5 +45,13 @@ describe('rules/require-space-before-binary-operators', function() {
         checker.configure({ requireSpaceBeforeBinaryOperators: [','] });
         assert(checker.checkString('({a: 1, b: 2})').getErrorCount() === 1);
     });
+    it('should not report assignment operator for "a = b"', function() {
+        checker.configure({ requireSpaceBeforeBinaryOperators: ['='] });
+        assert(checker.checkString('a = b').isEmpty());
+    });
+    it('should report assignment operator for "a=b"', function() {
+        checker.configure({ requireSpaceBeforeBinaryOperators: ['='] });
+        assert(checker.checkString('a=b').getErrorCount() === 1);
+    });
 
 });