Chore: remove deprecated experimentalObjectRestSpread options

mysticatea · michalsnik · commit dadeadc41186 · 2018-07-24T23:43:27.000+02:00
diff --git a/tests/lib/rules/name-property-casing.js b/tests/lib/rules/name-property-casing.js
@@ -16,9 +16,8 @@ const RuleTester = require('eslint').RuleTester
 // ------------------------------------------------------------------------------
 
 const parserOptions = {
-  ecmaVersion: 6,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 const ruleTester = new RuleTester()
diff --git a/tests/lib/rules/no-async-in-computed-properties.js b/tests/lib/rules/no-async-in-computed-properties.js
@@ -12,15 +12,8 @@ const rule = require('../../../lib/rules/no-async-in-computed-properties')
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 6,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
-}
-
-const parserOptions8 = {
-  ecmaVersion: 8,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 // ------------------------------------------------------------------------------
@@ -77,7 +70,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
           }));
         }
       `,
-      parserOptions: parserOptions8
+      parserOptions
     }
   ],
 
@@ -93,7 +86,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
           }
         }
       `,
-      parserOptions: parserOptions8,
+      parserOptions,
       errors: [{
         message: 'Unexpected async function declaration in "foo" computed property.',
         line: 4
@@ -113,7 +106,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {
           }
         }
       `,
-      parserOptions: parserOptions8,
+      parserOptions,
       errors: [{
         message: 'Unexpected async function declaration in "foo" computed property.',
         line: 4
diff --git a/tests/lib/rules/no-dupe-keys.js b/tests/lib/rules/no-dupe-keys.js
@@ -79,7 +79,7 @@ ruleTester.run('no-dupe-keys', rule, {
           },
         }
       `,
-      parserOptions: { ecmaVersion: 8, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}
+      parserOptions: { ecmaVersion: 2018, sourceType: 'module' }
     },
 
     {
diff --git a/tests/lib/rules/no-reserved-keys.js b/tests/lib/rules/no-reserved-keys.js
@@ -12,9 +12,8 @@ const rule = require('../../../lib/rules/no-reserved-keys')
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 7,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 // ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/no-shared-component-data.js b/tests/lib/rules/no-shared-component-data.js
@@ -13,9 +13,8 @@ const rule = require('../../../lib/rules/no-shared-component-data')
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 7,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 // ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/no-side-effects-in-computed-properties.js b/tests/lib/rules/no-side-effects-in-computed-properties.js
@@ -12,9 +12,8 @@ const rule = require('../../../lib/rules/no-side-effects-in-computed-properties'
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 6,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 // ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/order-in-components.js b/tests/lib/rules/order-in-components.js
@@ -10,9 +10,8 @@ const RuleTester = require('eslint').RuleTester
 const ruleTester = new RuleTester()
 
 const parserOptions = {
-  ecmaVersion: 6,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 ruleTester.run('order-in-components', rule, {
diff --git a/tests/lib/rules/prop-name-casing.js b/tests/lib/rules/prop-name-casing.js
@@ -16,9 +16,8 @@ const RuleTester = require('eslint').RuleTester
 // ------------------------------------------------------------------------------
 
 const parserOptions = {
-  ecmaVersion: 6,
-  sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true }
+  ecmaVersion: 2018,
+  sourceType: 'module'
 }
 
 const ruleTester = new RuleTester()
diff --git a/tests/lib/rules/require-default-prop.js b/tests/lib/rules/require-default-prop.js
@@ -11,8 +11,7 @@
 const rule = require('../../../lib/rules/require-default-prop')
 const RuleTester = require('eslint').RuleTester
 const parserOptions = {
-  ecmaVersion: 6,
-  ecmaFeatures: { experimentalObjectRestSpread: true },
+  ecmaVersion: 2018,
   sourceType: 'module'
 }
 
diff --git a/tests/lib/rules/require-prop-types.js b/tests/lib/rules/require-prop-types.js
@@ -31,7 +31,7 @@ ruleTester.run('require-prop-types', rule, {
           }
         }
       `,
-      parserOptions: { ecmaVersion: 6, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}
+      parserOptions: { ecmaVersion: 2018, sourceType: 'module' }
     },
     {
       filename: 'test.vue',
diff --git a/tests/lib/rules/require-render-return.js b/tests/lib/rules/require-render-return.js
@@ -12,9 +12,9 @@ const rule = require('../../../lib/rules/require-render-return')
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 6,
+  ecmaVersion: 2018,
   sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true, jsx: true }
+  ecmaFeatures: { jsx: true }
 }
 
 // ------------------------------------------------------------------------------
diff --git a/tests/lib/rules/require-valid-default-prop.js b/tests/lib/rules/require-valid-default-prop.js
@@ -12,9 +12,9 @@ const rule = require('../../../lib/rules/require-valid-default-prop')
 const RuleTester = require('eslint').RuleTester
 
 const parserOptions = {
-  ecmaVersion: 6,
+  ecmaVersion: 2018,
   sourceType: 'module',
-  ecmaFeatures: { experimentalObjectRestSpread: true, jsx: true }
+  ecmaFeatures: { jsx: true }
 }
 
 function errorMessage (type) {
diff --git a/tests/lib/rules/valid-v-on.js b/tests/lib/rules/valid-v-on.js
@@ -82,6 +82,10 @@ tester.run('valid-v-on', rule, {
     {
       filename: 'test.vue',
       code: '<template><div v-on="$listeners"></div></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div v-on="{a, b, c: d}"></div></template>'
     }
   ],
   invalid: [

Original file line number	Diff line number	Diff line change
`@@ -12,15 +12,8 @@ const rule = require('../../../lib/rules/no-async-in-computed-properties')`
`12`	`12`	`const RuleTester = require('eslint').RuleTester`
`13`	`13`
`14`	`14`	`const parserOptions = {`
`15`		`- ecmaVersion: 6,`
`16`		`- sourceType: 'module',`
`17`		`- ecmaFeatures: { experimentalObjectRestSpread: true }`
`18`		`-}`
`19`		`-`
`20`		`-const parserOptions8 = {`
`21`		`- ecmaVersion: 8,`
`22`		`- sourceType: 'module',`
`23`		`- ecmaFeatures: { experimentalObjectRestSpread: true }`
	`15`	`+ ecmaVersion: 2018,`
	`16`	`+ sourceType: 'module'`
`24`	`17`	`}`
`25`	`18`
`26`	`19`	`// ------------------------------------------------------------------------------`
`@@ -77,7 +70,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {`
`77`	`70`	`}));`
`78`	`71`	`}`
`79`	`72`	`,
`80`		`- parserOptions: parserOptions8`
	`73`	`+ parserOptions`
`81`	`74`	`}`
`82`	`75`	`],`
`83`	`76`
`@@ -93,7 +86,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {`
`93`	`86`	`}`
`94`	`87`	`}`
`95`	`88`	`,
`96`		`- parserOptions: parserOptions8,`
	`89`	`+ parserOptions,`
`97`	`90`	`errors: [{`
`98`	`91`	`message: 'Unexpected async function declaration in "foo" computed property.',`
`99`	`92`	`line: 4`
`@@ -113,7 +106,7 @@ ruleTester.run('no-async-in-computed-properties', rule, {`
`113`	`106`	`}`
`114`	`107`	`}`
`115`	`108`	`,
`116`		`- parserOptions: parserOptions8,`
	`109`	`+ parserOptions,`
`117`	`110`	`errors: [{`
`118`	`111`	`message: 'Unexpected async function declaration in "foo" computed property.',`
`119`	`112`	`line: 4`
Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ ruleTester.run('no-dupe-keys', rule, {`
`79`	`79`	`},`
`80`	`80`	`}`
`81`	`81`	`,
`82`		`- parserOptions: { ecmaVersion: 8, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}`
	`82`	`+ parserOptions: { ecmaVersion: 2018, sourceType: 'module' }`
`83`	`83`	`},`
`84`	`84`
`85`	`85`	`{`
Original file line number	Diff line number	Diff line change
`@@ -11,8 +11,7 @@`
`11`	`11`	`const rule = require('../../../lib/rules/require-default-prop')`
`12`	`12`	`const RuleTester = require('eslint').RuleTester`
`13`	`13`	`const parserOptions = {`
`14`		`- ecmaVersion: 6,`
`15`		`- ecmaFeatures: { experimentalObjectRestSpread: true },`
	`14`	`+ ecmaVersion: 2018,`
`16`	`15`	`sourceType: 'module'`
`17`	`16`	`}`
`18`	`17`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ ruleTester.run('require-prop-types', rule, {`
`31`	`31`	`}`
`32`	`32`	`}`
`33`	`33`	`,
`34`		`- parserOptions: { ecmaVersion: 6, sourceType: 'module', ecmaFeatures: { experimentalObjectRestSpread: true }}`
	`34`	`+ parserOptions: { ecmaVersion: 2018, sourceType: 'module' }`
`35`	`35`	`},`
`36`	`36`	`{`
`37`	`37`	`filename: 'test.vue',`
Original file line number	Diff line number	Diff line change
`@@ -82,6 +82,10 @@ tester.run('valid-v-on', rule, {`
`82`	`82`	`{`
`83`	`83`	`filename: 'test.vue',`
`84`	`84`	`code: '<template><div v-on="$listeners"></div></template>'`
	`85`	`+ },`
	`86`	`+ {`
	`87`	`+ filename: 'test.vue',`
	`88`	`+ code: '<template><div v-on="{a, b, c: d}"></div></template>'`
`85`	`89`	`}`
`86`	`90`	`],`
`87`	`91`	`invalid: [`