Rename rule and add support for experimentalObjectRestSpread

Author: armano2

docs/rules/component-data.md renamed to docs/rules/no-shared-component-data.md

@@ -1,4 +1,4 @@
-# Enforces that a component data must be a function (component-data)
+# Enforces that a component data must be a function (no-shared-component-data)
 
 When using the data property on a component (i.e. anywhere except on `new Vue`), the value must be a function that returns an object.
 

lib/rules/component-data.js renamed to lib/rules/no-shared-component-data.js

@@ -7,14 +7,6 @@
 const utils = require('../utils')
 
 function create (context) {
-  // variables should be defined here
-
-  // ----------------------------------------------------------------------
-  // Helpers
-  // ----------------------------------------------------------------------
-
-  // any helper functions should go here or else delete this section
-
   // ----------------------------------------------------------------------
   // Public
   // ----------------------------------------------------------------------
@@ -23,9 +15,11 @@ function create (context) {
     utils.executeOnVueComponent(context, (obj) => {
       obj.properties
         .filter(p =>
+          p.type === 'Property' &&
           p.key.type === 'Identifier' &&
           p.key.name === 'data' &&
-          p.value.type !== 'FunctionExpression'
+          p.value.type !== 'FunctionExpression' &&
+          p.value.type !== 'Identifier'
         ).forEach(cp => {
           context.report({
             node: cp.value,

tests/lib/rules/component-data.js renamed to tests/lib/rules/no-shared-component-data.js

@@ -8,16 +8,16 @@
 // Requirements
 // ------------------------------------------------------------------------------
 
-var rule = require('../../../lib/rules/component-data')
+const rule = require('../../../lib/rules/no-shared-component-data')
 
 const RuleTester = require('eslint').RuleTester
 
 // ------------------------------------------------------------------------------
 // Tests
 // ------------------------------------------------------------------------------
 
-var ruleTester = new RuleTester()
-ruleTester.run('component-data', rule, {
+const ruleTester = new RuleTester()
+ruleTester.run('no-shared-component-data', rule, {
 
   valid: [
     {
@@ -67,6 +67,24 @@ ruleTester.run('component-data', rule, {
         }
       `,
       parserOptions: { ecmaVersion: 6, sourceType: 'module' }
+    },
+    {
+      filename: 'test.vue',
+      code: `
+        export default {
+          ...foo
+        }
+      `,
+      parserOptions: { ecmaVersion: 6, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}
+    },
+    {
+      filename: 'test.vue',
+      code: `
+        export default {
+          data
+        }
+      `,
+      parserOptions: { ecmaVersion: 6, sourceType: 'module' }
     }
   ],