Remove /wordpress from test/linting ignore paths (#20270)

* Remove /wordpress from ignore paths * Remove /wordpress from gitignore * Remove note about wordpress folders being ignored * Update package-lock.json from master * Fix JS linting errors
WordPress · Jan 3, 2021 · 9dd9d28 · 9dd9d28
1 parent 2bce694
commit 9dd9d28
Show file tree

Hide file tree

Showing 11 changed files with 12 additions and 23 deletions.
diff --git a/.eslintignore b/.eslintignore
@@ -7,5 +7,4 @@ packages/block-serialization-spec-parser/parser.js
 packages/e2e-tests/plugins
 packages/react-native-editor/bundle
 vendor
-wordpress
 !.*.js
diff --git a/.gitignore b/.gitignore
@@ -11,7 +11,6 @@ gutenberg.zip
 *.log
 phpcs.xml
 yarn.lock
-/wordpress
 /artifacts
 
 .cache

diff --git a/packages/dependency-extraction-webpack-plugin/test/fixtures/wordpress/webpack.config.js b/packages/dependency-extraction-webpack-plugin/test/fixtures/wordpress/webpack.config.js
@@ -1,3 +1,6 @@
+/**
+ * Internal dependencies
+ */
 const DependencyExtractionWebpackPlugin = require( '../../..' );
 
 module.exports = {

diff --git a/packages/e2e-tests/jest.config.js b/packages/e2e-tests/jest.config.js
@@ -10,7 +10,6 @@ module.exports = {
 	],
 	testPathIgnorePatterns: [
 		'/node_modules/',
-		'<rootDir>/wordpress/',
 		'e2e-tests/specs/performance/',
 	],
 };
diff --git a/packages/jest-preset-default/jest-preset.js b/packages/jest-preset-default/jest-preset.js
@@ -21,11 +21,7 @@ module.exports = {
 		'**/test/*.[jt]s',
 		'**/?(*.)test.[jt]s',
 	],
-	testPathIgnorePatterns: [
-		'/node_modules/',
-		'<rootDir>/vendor/',
-		'<rootDir>/wordpress/',
-	],
+	testPathIgnorePatterns: [ '/node_modules/', '<rootDir>/vendor/' ],
 	timers: 'fake',
 	transform: {
 		'^.+\\.[jt]sx?$': require.resolve( 'babel-jest' ),

diff --git a/packages/scripts/README.md b/packages/scripts/README.md
@@ -150,7 +150,7 @@ This is how you execute the script with presented setup:
 
 When you run commands similar to the `npm run format:js:src` example above, you can provide a file, a directory, or `glob` syntax or any combination of them.
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 ### `lint-js`
 
@@ -174,7 +174,7 @@ This is how you execute the script with presented setup:
 
 When you run commands similar to the `npm run lint:js:src` example above, you can provide a file, a directory, or `glob` syntax or any combination of them. See [more examples](https://eslint.org/docs/user-guide/command-line-interface).
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 #### Advanced information
 
@@ -202,7 +202,7 @@ This is how you execute those scripts using the presented setup:
 
 When you run commands similar to the `npm run lint:pkg-json:src` example above, you can provide one or multiple directories to scan as well. See [more examples](https://github.com/tclindner/npm-package-json-lint/blob/HEAD/README.md#examples).
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 #### Advanced information
 
@@ -226,7 +226,7 @@ This is how you execute the script with presented setup:
 
 -   `npm run lint:md:docs` - lints markdown files in the entire project’s directories.
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 #### Advanced information
 
@@ -250,7 +250,7 @@ This is how you execute the script with presented setup:
 
 -   `npm run lint:md:js` - lints markdown files in the entire project’s directories.
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 #### Advanced information
 
@@ -278,7 +278,7 @@ This is how you execute the script with presented setup:
 
 When you run commands similar to the `npm run lint:css:src` example above, be sure to include the quotation marks around file globs. This ensures that you can use the powers of [globby](https://github.com/sindresorhus/globby) (like the `**` globstar) regardless of your shell. See [more examples](https://github.com/stylelint/stylelint/blob/HEAD/docs/user-guide/cli.md#examples).
 
-By default, files located in `build`, `node_modules`, `vendor` and `wordpress` folders are ignored.
+By default, files located in `build`, `node_modules`, and `vendor` folders are ignored.
 
 #### Advanced information
 

diff --git a/packages/scripts/config/.eslintignore b/packages/scripts/config/.eslintignore
@@ -1,4 +1,3 @@
 build
 node_modules
 vendor
-wordpress
diff --git a/packages/scripts/config/.stylelintignore b/packages/scripts/config/.stylelintignore
@@ -2,4 +2,3 @@
 
 build
 vendor
-wordpress
diff --git a/packages/scripts/config/jest-e2e.config.js b/packages/scripts/config/jest-e2e.config.js
@@ -11,7 +11,7 @@ const { hasBabelConfig } = require( '../utils' );
 const jestE2EConfig = {
 	preset: 'jest-puppeteer',
 	testMatch: [ '**/specs/**/*.[jt]s', '**/?(*.)spec.[jt]s' ],
-	testPathIgnorePatterns: [ '/node_modules/', '/wordpress/' ],
+	testPathIgnorePatterns: [ '/node_modules/' ],
 	reporters:
 		'TRAVIS' in process.env && 'CI' in process.env
 			? [

diff --git a/test/native/jest.config.js b/test/native/jest.config.js
@@ -34,11 +34,7 @@ module.exports = {
 		'**/test/*.native.[jt]s?(x)',
 		'<rootDir>/packages/react-native-*/**/?(*.)+(spec|test).[jt]s?(x)',
 	],
-	testPathIgnorePatterns: [
-		'/node_modules/',
-		'<rootDir>/wordpress/',
-		'/__device-tests__/',
-	],
+	testPathIgnorePatterns: [ '/node_modules/', '/__device-tests__/' ],
 	testURL: 'http://localhost/',
 	// Add the `Libraries/Utilities` subfolder to the module directories, otherwise haste/jest doesn't find Platform.js on Travis,
 	// and add it first so https://github.com/facebook/react-native/blob/v0.60.0/Libraries/react-native/react-native-implementation.js#L324-L326 doesn't pick up the Platform npm module.

diff --git a/test/unit/jest.config.js b/test/unit/jest.config.js
@@ -25,7 +25,6 @@ module.exports = {
 		'/.git/',
 		'/node_modules/',
 		'/packages/e2e-tests',
-		'<rootDir>/wordpress/',
 		'<rootDir>/.*/build/',
 		'<rootDir>/.*/build-module/',
 		'<rootDir>/.+.native.js$',