validateParameterSeparator: notice class methods

Fixes #2289
jscs-dev · Jul 14, 2016 · 47d19e1 · 47d19e1
1 parent 92fa964
commit 47d19e1
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/lib/rules/validate-parameter-separator.js b/lib/rules/validate-parameter-separator.js
@@ -54,7 +54,11 @@ module.exports.prototype = {
         var whitespaceBeforeComma = Boolean(separators.shift());
         var whitespaceAfterComma = Boolean(separators.pop());
 
-        file.iterateNodesByType(['FunctionDeclaration', 'FunctionExpression'], function(node) {
+        file.iterateNodesByType([
+            'FunctionDeclaration',
+            'FunctionExpression',
+            'ClassMethod'
+        ], function(node) {
 
             node.params.forEach(function(paramNode) {
 

diff --git a/test/specs/rules/validate-parameter-separator.js b/test/specs/rules/validate-parameter-separator.js
@@ -51,6 +51,11 @@ describe('rules/validate-parameter-separator', function() {
               .to.have.one.validation.error.from('validateParameterSeparator');
         });
 
+        it('should validate class methods', function() {
+            expect(checker.checkString('class a { constructor(a, b) {} }'))
+              .to.have.one.validation.error.from('validateParameterSeparator');
+        });
+
         it('should report unexpected space for function a(b ,c) {}', function() {
             expect(checker.checkString('function a(b ,c) {}'))
               .to.have.one.validation.error.from('validateParameterSeparator');
@@ -102,6 +107,11 @@ describe('rules/validate-parameter-separator', function() {
               .to.have.one.validation.error.from('validateParameterSeparator');
         });
 
+        it('should validate class methods', function() {
+            expect(checker.checkString('class a { constructor(a , b) {} }'))
+              .to.have.one.validation.error.from('validateParameterSeparator');
+        });
+
         it('should report missing space for function a(b,c) {}', function() {
             expect(checker.checkString('function a(b,c) {}'))
               .to.have.one.validation.error.from('validateParameterSeparator');
@@ -149,6 +159,11 @@ describe('rules/validate-parameter-separator', function() {
               .to.have.one.validation.error.from('validateParameterSeparator');
         });
 
+        it('should validate class methods', function() {
+            expect(checker.checkString('class a { constructor(a , b) {} }'))
+              .to.have.one.validation.error.from('validateParameterSeparator');
+        });
+
         it('should report missing space for function a(b,c) {}', function() {
             expect(checker.checkString('function a(b,c) {}'))
               .to.have.one.validation.error.from('validateParameterSeparator');
@@ -192,6 +207,11 @@ describe('rules/validate-parameter-separator', function() {
               .to.have.one.validation.error.from('validateParameterSeparator');
         });
 
+        it('should validate class methods', function() {
+            expect(checker.checkString('class a { constructor(a, b) {} }'))
+              .to.have.one.validation.error.from('validateParameterSeparator');
+        });
+
         it('should report missing space for function a(b ,c) {}', function() {
             expect(checker.checkString('function a(b ,c) {}'))
               .to.have.one.validation.error.from('validateParameterSeparator');