Enable integration test on more files (#628)

Author: fisker

test/integration/projects.js

@@ -1,7 +1,7 @@
 'use strict';
 
-const typescriptArguments = ['--parser', '@typescript-eslint/parser', '--ext', '.ts'];
-const vueArguments = ['--parser', 'vue-eslint-parser', '--ext', '.vue'];
+const typescriptArguments = ['--parser', '@typescript-eslint/parser', '--ext', '.ts,.js'];
+const vueArguments = ['--parser', 'vue-eslint-parser', '--ext', '.vue,.js'];
 
 module.exports = [
 	'https://github.com/avajs/ava',
@@ -49,42 +49,35 @@ module.exports = [
 		extraArguments: typescriptArguments
 	},
 	// TODO: Add this project when #561 got fixed
-	// {
-	// 	repository: 'https://github.com/eslint/eslint',
-	// 	path: 'lib'
-	// },
+	// 'https://github.com/eslint/eslint',
+	'https://github.com/prettier/prettier',
+	'https://github.com/facebook/react',
 	{
-		repository: 'https://github.com/prettier/prettier',
-		path: 'src'
-	},
-	{
-		repository: 'https://github.com/facebook/react',
-		path: 'packages'
+		repository: 'https://github.com/angular/angular',
+		extraArguments: [
+			...typescriptArguments,
+
+			'--ignore-pattern',
+			'aio/content/examples/animations/src/app/open-close.component.3.ts',
+
+			'--ignore-pattern',
+			'aio/tools/transforms/templates/data-module.template.js'
+		]
 	},
 	{
-		repository: 'https://github.com/angular/angular',
-		path: 'packages',
+		repository: 'https://github.com/microsoft/typescript',
 		extraArguments: typescriptArguments
 	},
 	{
-		repository: 'https://github.com/microsoft/typescript',
-		path: 'src',
+		repository: 'https://github.com/microsoft/vscode',
 		extraArguments: typescriptArguments
 	},
-	// TODO: Add this project when `@typescript-eslint/parser` support `Type-Only Imports and Export`
-	// {
-	// 	repository: 'https://github.com/microsoft/vscode',
-	// 	path: 'src/vs',
-	// 	extraArguments: typescriptArguments
-	// },
 	{
 		repository: 'https://github.com/ElemeFE/element',
-		path: 'packages',
 		extraArguments: vueArguments
 	},
 	{
 		repository: 'https://github.com/iview/iview',
-		path: 'src',
 		extraArguments: vueArguments
 	},
 	'https://github.com/sindresorhus/create-dmg',
@@ -95,12 +88,26 @@ module.exports = [
 	'https://github.com/gatsbyjs/gatsby',
 	{
 		repository: 'https://github.com/puppeteer/puppeteer',
-		path: 'lib'
+		extraArguments: [
+			// Parser error on `await page.evaluate(() => delete Node);`
+			// https://github.com/puppeteer/puppeteer/blob/0b1a9ceee2f05f534f0d50079ece172d627a93c7/test/jshandle.spec.js#L151
+			'--ignore-pattern',
+			'test/jshandle.spec.js',
+
+			// `package` keyword
+			// https://github.com/puppeteer/puppeteer/blob/0b1a9ceee2f05f534f0d50079ece172d627a93c7/utils/apply_next_version.js#L17
+			'--ignore-pattern',
+			'utils/apply_next_version.js'
+		]
 	},
 	{
 		repository: 'https://github.com/zeit/next.js',
-		path: 'packages',
-		extraArguments: typescriptArguments
+		extraArguments: [
+			...typescriptArguments,
+
+			'--ignore-pattern',
+			'examples/**'
+		]
 	},
 	'https://github.com/chakra-ui/chakra-ui',
 	'https://github.com/ReactTraining/react-router',

test/integration/test.js

@@ -23,15 +23,13 @@ const makeEslintTask = (project, destination) => {
 		'eslint',
 		'--fix-dry-run',
 		'--no-eslintrc',
-		// https://github.com/microsoft/TypeScript/blob/a1c8608f681488fda3f0b6ffd89195a81f80f0b0/.eslintignore#L6
-		project.name === 'typescript' ? '' : '--no-ignore',
 		'--format',
 		'json',
 		'--config',
 		path.join(__dirname, 'config.js'),
-		project.path ? path.join(destination, project.path) : destination,
+		project.path || '.',
 		...project.extraArguments
-	].filter(Boolean);
+	];
 
 	return enrichErrors(project.name, arguments_, async () => {
 		let stdout;
@@ -64,7 +62,11 @@ const makeEslintTask = (project, destination) => {
 			for (const message of file.messages) {
 				if (message.fatal) {
 					const error = new Error(message.message);
-					error.eslintFile = file;
+					error.eslintJob = {
+						destination,
+						project,
+						file
+					};
 					error.eslintMessage = message;
 					throw error;
 				}
@@ -142,7 +144,11 @@ list.run()
 				}
 
 				if (error2.eslintMessage) {
-					console.error(chalk.gray(error2.eslintFile.filePath), chalk.gray(JSON.stringify(error2.eslintMessage, null, 2)));
+					const {file, project, destination} = error2.eslintJob;
+					const {line} = error2.eslintMessage;
+
+					console.error(chalk.gray(`${project.repository}/tree/master/${path.relative(destination, file.filePath)}#L${line}`));
+					console.error(chalk.gray(JSON.stringify(error2.eslintMessage, null, 2)));
 				}
 			}
 		} else {