ESLint Plugin: Use Jest related rules only when the package is instal…

…led (#33120) * Add Jest to eslint plugin dependencies * Add test rules only when jest is installed Co-authored-by: Grzegorz Ziolkowski <[email protected]>
WordPress · Sep 9, 2021 · 30744d0 · 30744d0
1 parent d3fa6c3
commit 30744d0
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/packages/eslint-plugin/CHANGELOG.md b/packages/eslint-plugin/CHANGELOG.md
@@ -6,6 +6,10 @@
 
 -   The bundled `eslint-plugin-jsdoc` dependency has been updated from requiring `^34.1.0` to requiring `^36.0.8` ([#34338](https://github.com/WordPress/gutenberg/pull/34338)).
 
+### Bug Fix
+
+-   Use Jest related rules only when the `jest` package is installed ([#33120](https://github.com/WordPress/gutenberg/pull/33120)).
+
 ## 9.1.2 (2021-09-09)
 
 ### Bug Fix

diff --git a/packages/eslint-plugin/configs/recommended-with-formatting.js b/packages/eslint-plugin/configs/recommended-with-formatting.js
@@ -1,4 +1,9 @@
-module.exports = {
+/**
+ * Internal dependencies
+ */
+const { isPackageInstalled } = require( '../utils' );
+
+const config = {
 	parser: 'babel-eslint',
 	extends: [
 		require.resolve( './jsx-a11y.js' ),
@@ -27,7 +32,10 @@ module.exports = {
 		'import/default': 'warn',
 		'import/named': 'warn',
 	},
-	overrides: [
+};
+
+if ( isPackageInstalled( 'jest' ) ) {
+	config.overrides = [
 		{
 			// Unit test files and their helpers only.
 			files: [ '**/@(test|__tests__)/**/*.js', '**/?(*.)test.js' ],
@@ -38,5 +46,7 @@ module.exports = {
 			files: [ '**/specs/**/*.js', '**/?(*.)spec.js' ],
 			extends: [ require.resolve( './test-e2e.js' ) ],
 		},
-	],
-};
+	];
+}
+
+module.exports = config;