Object Key rules: ignore method syntax

Fixes #1013
Closes gh-1020

Author: Alexej Yaroshevich

lib/rules/disallow-space-after-object-keys.js

@@ -41,7 +41,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateNodesByType('ObjectExpression', function(node) {
             node.properties.forEach(function(property) {
-                if (property.shorthand) {
+                if (property.shorthand || property.method) {
                     return;
                 }
 

lib/rules/disallow-space-before-object-values.js

@@ -41,7 +41,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateNodesByType('ObjectExpression', function(node) {
             node.properties.forEach(function(property) {
-                if (property.shorthand) {
+                if (property.shorthand || property.method) {
                     return;
                 }
 

lib/rules/require-aligned-object-values.js

@@ -67,7 +67,7 @@ module.exports.prototype = {
 
             var maxKeyEndPos = 0;
             var skip = node.properties.some(function(property, index) {
-                if (property.shorthand) {
+                if (property.shorthand || property.method) {
                     return true;
                 }
 

lib/rules/require-quoted-keys-in-objects.js

@@ -44,7 +44,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateNodesByType('ObjectExpression', function(node) {
             node.properties.forEach(function(prop) {
-                if (prop.shorthand) {
+                if (prop.shorthand || prop.method) {
                     return;
                 }
 

lib/rules/require-space-after-object-keys.js

@@ -41,7 +41,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateNodesByType('ObjectExpression', function(node) {
             node.properties.forEach(function(property) {
-                if (property.shorthand) {
+                if (property.shorthand || property.method) {
                     return;
                 }
 

lib/rules/require-space-before-object-values.js

@@ -41,7 +41,7 @@ module.exports.prototype = {
     check: function(file, errors) {
         file.iterateNodesByType('ObjectExpression', function(node) {
             node.properties.forEach(function(property) {
-                if (property.shorthand) {
+                if (property.shorthand || property.method) {
                     return;
                 }
 

test/rules/disallow-space-after-object-keys.js

@@ -39,4 +39,9 @@ describe('rules/disallow-space-after-object-keys', function() {
         checker.configure({ esnext: true });
         assert.equal(checker.checkString('var x = { a : 1, b };').getErrorCount(), 1);
     });
+
+    it('should not report es6-methods. #1013', function() {
+        checker.configure({ esnext: true });
+        assert(checker.checkString('var x = { a() { } };').isEmpty());
+    });
 });

test/rules/disallow-space-before-object-values.js

@@ -51,4 +51,9 @@ describe('rules/disallow-space-before-object-values', function() {
         checker.configure({ esnext: true });
         assert.equal(checker.checkString('var x = { a : 1, b };').getErrorCount(), 1);
     });
+
+    it('should not report es6-methods. #1013', function() {
+        checker.configure({ esnext: true });
+        assert(checker.checkString('var x = { a() { } };').isEmpty());
+    });
 });

test/rules/require-aligned-object-values.js

@@ -46,6 +46,11 @@ describe('rules/require-aligned-object-values', function() {
             );
         });
 
+        it('should not report es6-methods. #1013', function() {
+            checker.configure({ esnext: true });
+            assert(checker.checkString('var x = { a() { } };').isEmpty());
+        });
+
         it('should report invalid alignment', function() {
             assert(
                 checker.checkString(

test/rules/require-quoted-keys-in-objects.js

@@ -67,4 +67,9 @@ describe('rules/require-quoted-keys-in-objects', function() {
         checker.configure({ esnext: true });
         assert(checker.checkString('var x = { a, b };').isEmpty());
     });
+
+    it('should not report es6-methods. #1013', function() {
+        checker.configure({ esnext: true });
+        assert(checker.checkString('var x = { a() { } };').isEmpty());
+    });
 });