Use 'latest' as ecmaVersion (#1465)

fisker · web-flow · commit 9feb181fdd41 · 2021-10-11T19:40:16.000+07:00
diff --git a/configs/base.js b/configs/base.js
@@ -4,7 +4,7 @@ module.exports = {
 		es6: true,
 	},
 	parserOptions: {
-		ecmaVersion: 2021,
+		ecmaVersion: 'latest',
 		sourceType: 'module',
 	},
 	plugins: [
diff --git a/package.json b/package.json
@@ -57,9 +57,9 @@
 	"devDependencies": {
 		"@babel/code-frame": "^7.14.5",
 		"@babel/core": "^7.15.5",
-		"@babel/eslint-parser": "^7.15.4",
+		"@babel/eslint-parser": "^7.15.7",
 		"@lubien/fixture-beta-package": "^1.0.0-beta.1",
-		"@typescript-eslint/parser": "^4.31.0",
+		"@typescript-eslint/parser": "^4.32.0",
 		"ava": "^3.15.0",
 		"chalk": "^4.1.2",
 		"enquirer": "2.3.6",
diff --git a/readme.md b/readme.md
@@ -28,7 +28,7 @@ Configure it in `package.json`.
 			"es6": true
 		},
 		"parserOptions": {
-			"ecmaVersion": 2021,
+			"ecmaVersion": "latest",
 			"sourceType": "module"
 		},
 		"plugins": [
diff --git a/test/consistent-function-scoping.mjs b/test/consistent-function-scoping.mjs
@@ -16,8 +16,6 @@ const createError = (functionNameWithKind, loc) => ({
 test({
 	testerOptions: {
 		parserOptions: {
-			sourceType: 'module',
-			ecmaVersion: 2021,
 			ecmaFeatures: {
 				jsx: true,
 			},
diff --git a/test/integration/config.js b/test/integration/config.js
@@ -4,10 +4,6 @@ module.exports = {
 	root: true,
 	parser: '@babel/eslint-parser',
 	parserOptions: {
-		ecmaVersion: 2021,
-		ecmaFeatures: {
-			jsx: true,
-		},
 		requireConfigFile: false,
 		babelOptions: {
 			babelrc: false,
diff --git a/test/integration/projects.mjs b/test/integration/projects.mjs
@@ -88,8 +88,8 @@ export default [
 	// 		'build/**'
 	// 	]
 	// },
-	'https://github.com/ElemeFE/element',
-	'https://github.com/iview/iview',
+	// 'https://github.com/ElemeFE/element',
+	// 'https://github.com/iview/iview',
 	'https://github.com/sindresorhus/create-dmg',
 	'https://github.com/sindresorhus/cp-file',
 	'https://github.com/sindresorhus/capture-website',
diff --git a/test/no-for-loop.mjs b/test/no-for-loop.mjs
@@ -731,6 +731,7 @@ test({
 test(avoidTestTitleConflict({
 	testerOptions: {
 		parserOptions: {
+			sourceType: 'script',
 			ecmaVersion: 5,
 		},
 	},
diff --git a/test/no-keyword-prefix.mjs b/test/no-keyword-prefix.mjs
@@ -228,12 +228,10 @@ test({
 		},
 		{
 			code: 'function foo({ newBar = \'default value\' }) {};',
-			parserOptions: {ecmaVersion: 6},
 			errors: [errorNew],
 		},
 		{
 			code: 'const newFoo = 0; function foo({ newBar = newFoo}) {}',
-			parserOptions: {ecmaVersion: 6},
 			errors: [errorNew, errorNew],
 		},
 		{
diff --git a/test/prevent-abbreviations.mjs b/test/prevent-abbreviations.mjs
@@ -1271,6 +1271,7 @@ test.typescript(avoidTestTitleConflict(tests, 'typescript'));
 test({
 	testerOptions: {
 		parserOptions: {
+			sourceType: 'script',
 			ecmaVersion: 5,
 		},
 		env: {
diff --git a/test/smoke/eslint-remote-tester.config.js b/test/smoke/eslint-remote-tester.config.js
@@ -29,7 +29,7 @@ module.exports = {
 		},
 		parser: '@typescript-eslint/parser',
 		parserOptions: {
-			ecmaVersion: 2020,
+			ecmaVersion: 'latest',
 			sourceType: 'module',
 			ecmaFeatures: {
 				jsx: true,
diff --git a/test/utils/parsers.mjs b/test/utils/parsers.mjs
@@ -59,10 +59,18 @@ const vue = {
 		return require.resolve(this.name);
 	},
 	mergeParserOptions(options) {
-		return {
+		options = {
 			...defaultParserOptions,
 			...options,
 		};
+
+		// The `espree` version in `vue-eslint-parser` does not understand `'latest'` yet.
+		// https://github.com/vuejs/vue-eslint-parser/pull/113
+		if (options.ecmaVersion === 'latest') {
+			options.ecmaVersion = require('espree').latestEcmaVersion;
+		}
+
+		return options;
 	},
 };
 
diff --git a/test/utils/test.mjs b/test/utils/test.mjs
@@ -59,10 +59,13 @@ class Tester {
 	}
 
 	runTest(tests) {
-		const {beforeAll, testerOptions, valid, invalid} = tests;
+		const {beforeAll, testerOptions = {}, valid, invalid} = tests;
 		const tester = avaRuleTester(test, {
-			parserOptions: defaultParserOptions,
 			...testerOptions,
+			parserOptions: {
+				...defaultParserOptions,
+				...testerOptions.parserOptions,
+			},
 		});
 
 		if (beforeAll) {

Original file line number	Diff line number	Diff line change
`@@ -228,12 +228,10 @@ test({`
`228`	`228`	`},`
`229`	`229`	`{`
`230`	`230`	`code: 'function foo({ newBar = \'default value\' }) {};',`
`231`		`- parserOptions: {ecmaVersion: 6},`
`232`	`231`	`errors: [errorNew],`
`233`	`232`	`},`
`234`	`233`	`{`
`235`	`234`	`code: 'const newFoo = 0; function foo({ newBar = newFoo}) {}',`
`236`		`- parserOptions: {ecmaVersion: 6},`
`237`	`235`	`errors: [errorNew, errorNew],`
`238`	`236`	`},`
`239`	`237`	`{`