Revert "exclude performance test in jests"

This reverts commit 8cc2fcd.

Author: mohanrajpac

jest.config.js

@@ -5,10 +5,7 @@ module.exports = {
 	},
 	modulePathIgnorePatterns: ['<rootDir>/vendor/'],
 	testEnvironment: 'jsdom',
-	testPathIgnorePatterns: [
-		'<rootDir>/tests/e2e/',
-		'<rootDir>/tests/performance/',
-	],
+	testPathIgnorePatterns: ['<rootDir>/tests/e2e/'],
 	collectCoverage: true,
 	coverageReporters: ['text', 'cobertura'],
 };

tests/performance/cli/results.js

@@ -1,39 +1,37 @@
 #!/usr/bin/env node
+
 /**
  * External dependencies
  */
-// eslint-disable-next-line @typescript-eslint/no-var-requires
-const { existsSync, readFileSync, writeFileSync } = require('node:fs');
-// eslint-disable-next-line @typescript-eslint/no-var-requires
-const { join, resolve } = require('node:path');
+const { existsSync, readFileSync, writeFileSync } = require( 'node:fs' );
+const { join, resolve } = require( 'node:path' );
 
-process.env.WP_ARTIFACTS_PATH ??= join(process.cwd(), 'artifacts');
+process.env.WP_ARTIFACTS_PATH ??= join( process.cwd(), 'artifacts' );
 
-const args = process.argv.slice(2);
+const args = process.argv.slice( 2 );
 
-const beforeFile = args[1];
-const afterFile = args[0] || resolve('./artifacts/performance-results.json');
+const beforeFile = args[ 1 ];
+const afterFile =
+	args[ 0 ] || resolve( './artifacts/performance-results.json' );
 
-if (!existsSync(afterFile)) {
-	// eslint-disable-next-line no-console
-	console.error(`File not found: ${afterFile}`);
-	process.exit(1);
+if ( ! existsSync( afterFile ) ) {
+	console.error( `File not found: ${ afterFile }` );
+	process.exit( 1 );
 }
 
-if (beforeFile && !existsSync(beforeFile)) {
-	// eslint-disable-next-line no-console
-	console.error(`File not found: ${beforeFile}`);
-	process.exit(1);
+if ( beforeFile && ! existsSync( beforeFile ) ) {
+	console.error( `File not found: ${ beforeFile }` );
+	process.exit( 1 );
 }
 
 /**
  * @param {unknown} v
  * @return {string} Formatted value.
  */
-function formatTableValue(v) {
-	if (v === true || v === 'true') return '✅';
-	if (!v || v === 'false') return '';
-	return v?.toString() || String(v);
+function formatTableValue( v ) {
+	if ( v === true || v === 'true' ) return '✅';
+	if ( ! v || v === 'false' ) return '';
+	return v?.toString() || String( v );
 }
 
 /**
@@ -64,22 +62,25 @@ function formatTableValue(v) {
  * @param {Array<Object>} rows Table rows.
  * @return {string} Markdown table content.
  */
-function formatAsMarkdownTable(rows) {
+function formatAsMarkdownTable( rows ) {
 	let result = '';
-	const headers = Object.keys(rows[0]);
-	for (const header of headers) {
-		result += `| ${header} `;
+	const headers = Object.keys( rows[ 0 ] );
+	for ( const header of headers ) {
+		result += `| ${ header } `;
 	}
 	result += '|\n';
-	for (const header of headers) {
-		const dashes = '-'.repeat(header.length);
-		result += `| ${dashes} `;
+	for ( const header of headers ) {
+		const dashes = '-'.repeat( header.length );
+		result += `| ${ dashes } `;
 	}
 	result += '|\n';
 
-	for (const row of rows) {
-		for (const [key, value] of Object.entries(row)) {
-			result += `| ${formatTableValue(value).padStart(key.length, ' ')} `;
+	for ( const row of rows ) {
+		for ( const [ key, value ] of Object.entries( row ) ) {
+			result += `| ${ formatTableValue( value ).padStart(
+				key.length,
+				' '
+			) } `;
 		}
 		result += '|\n';
 	}
@@ -93,15 +94,15 @@ function formatAsMarkdownTable(rows) {
  * @param {number[]} array List of numbers.
  * @return {number} Median.
  */
-function median(array) {
-	const mid = Math.floor(array.length / 2);
-	const numbers = [...array].sort((a, b) => a - b);
+function median( array ) {
+	const mid = Math.floor( array.length / 2 );
+	const numbers = [ ...array ].sort( ( a, b ) => a - b );
 	const result =
 		array.length % 2 !== 0
-			? numbers[mid]
-			: (numbers[mid - 1] + numbers[mid]) / 2;
+			? numbers[ mid ]
+			: ( numbers[ mid - 1 ] + numbers[ mid ] ) / 2;
 
-	return Number(result.toFixed(2));
+	return Number( result.toFixed( 2 ) );
 }
 
 /**
@@ -114,39 +115,36 @@ let beforeStats = [];
  */
 let afterStats;
 
-if (beforeFile) {
+if ( beforeFile ) {
 	try {
 		beforeStats = JSON.parse(
-			readFileSync(beforeFile, { encoding: 'utf-8' })
+			readFileSync( beforeFile, { encoding: 'utf-8' } )
 		);
 	} catch {}
 }
 
 try {
-	afterStats = JSON.parse(readFileSync(afterFile, { encoding: 'utf-8' }));
+	afterStats = JSON.parse( readFileSync( afterFile, { encoding: 'utf-8' } ) );
 } catch {
-	// eslint-disable-next-line no-console
-	console.error(`Could not read file: ${afterFile}`);
-	process.exit(1);
+	console.error( `Could not read file: ${ afterFile }` );
+	process.exit( 1 );
 }
 
 let summaryMarkdown = `**Performance Test Results**\n\n`;
 
-if (process.env.GITHUB_SHA) {
-	summaryMarkdown += `Performance test results for ${process.env.GITHUB_SHA} are in 🛎️!\n\n`;
+if ( process.env.GITHUB_SHA ) {
+	summaryMarkdown += `Performance test results for ${ process.env.GITHUB_SHA } are in 🛎️!\n\n`;
 } else {
 	summaryMarkdown += `Performance test results are in 🛎️!\n\n`;
 }
 
-if (beforeFile) {
+if ( beforeFile ) {
 	summaryMarkdown += `Note: the numbers in parentheses show the difference to the previous (baseline) test run.\n\n`;
 }
 
-// eslint-disable-next-line no-console
-console.log('Performance Test Results\n');
+console.log( 'Performance Test Results\n' );
 
-if (beforeFile) {
-	// eslint-disable-next-line no-console
+if ( beforeFile ) {
 	console.log(
 		'Note: the numbers in parentheses show the difference to the previous (baseline) test run.\n'
 	);
@@ -166,32 +164,32 @@ const PERCENTAGE_VARIANCE = 2;
  * @param {string} key   Key.
  * @return {string} Formatted value.
  */
-function formatValue(value, key) {
-	if (key === 'CLS') {
-		return value.toFixed(2);
+function formatValue( value, key ) {
+	if ( key === 'CLS' ) {
+		return value.toFixed( 2 );
 	}
 
-	if (key === 'wpDbQueries') {
-		return value.toFixed(0);
+	if ( key === 'wpDbQueries' ) {
+		return value.toFixed( 0 );
 	}
 
-	if (key === 'wpMemoryUsage') {
-		return `${(value / Math.pow(10, 6)).toFixed(2)} MB`;
+	if ( key === 'wpMemoryUsage' ) {
+		return `${ ( value / Math.pow( 10, 6 ) ).toFixed( 2 ) } MB`;
 	}
 
-	return `${value.toFixed(2)} ms`;
+	return `${ value.toFixed( 2 ) } ms`;
 }
 
-for (const { file, title, results } of afterStats) {
-	const prevStat = beforeStats.find((s) => s.file === file);
+for ( const { file, title, results } of afterStats ) {
+	const prevStat = beforeStats.find( ( s ) => s.file === file );
 
 	/**
 	 * @type {Array<Record<string,string|number|boolean>>}
 	 */
 	const diffResults = [];
 
-	for (const i in results) {
-		const newResult = results[i];
+	for ( const i in results ) {
+		const newResult = results[ i ];
 
 		/**
 		 * @type {Record<string, string>}
@@ -200,54 +198,55 @@ for (const { file, title, results } of afterStats) {
 			Run: i,
 		};
 
-		for (const [key, values] of Object.entries(newResult)) {
+		for ( const [ key, values ] of Object.entries( newResult ) ) {
 			// Only do comparison if the number of results is the same.
 			const prevValues =
 				prevStat?.results.length === results.length
-					? prevStat?.results[i].key
+					? prevStat?.results[ i ].key
 					: null;
 
-			const value = median(values);
-			const prevValue = prevValues ? median(prevValues) : 0;
+			const value = median( values );
+			const prevValue = prevValues ? median( prevValues ) : 0;
 			const delta = value - prevValue;
-			const percentage = (delta / value) * 100;
+			const percentage = ( delta / value ) * 100;
 
 			// Skip if there is not a significant delta or none at all.
 			if (
-				!prevValues ||
-				!percentage ||
-				Math.abs(percentage) <= PERCENTAGE_VARIANCE ||
-				!delta ||
-				Math.abs(delta) <= DELTA_VARIANCE
+				! prevValues ||
+				! percentage ||
+				Math.abs( percentage ) <= PERCENTAGE_VARIANCE ||
+				! delta ||
+				Math.abs( delta ) <= DELTA_VARIANCE
 			) {
-				diffResult[key] = formatValue(
-					/** @type {number} */ (value),
+				diffResult[ key ] = formatValue(
+					/** @type {number} */ ( value ),
 					key
 				);
 				continue;
 			}
 
 			const prefix = delta > 0 ? '+' : '';
 
-			diffResult[key] = `${formatValue(
+			diffResult[ key ] = `${ formatValue(
 				value,
 				key
-			)} (${prefix}${formatValue(delta, key)} / ${prefix}${percentage}%)`;
+			) } (${ prefix }${ formatValue(
+				delta,
+				key
+			) } / ${ prefix }${ percentage }%)`;
 		}
 
-		diffResults.push(diffResult);
+		diffResults.push( diffResult );
 	}
 
-	// eslint-disable-next-line no-console
-	console.log(title);
-	// eslint-disable-next-line no-console
-	console.table(diffResults);
+	console.log( title );
+	console.table( diffResults );
 
-	summaryMarkdown += `**${title}**\n\n`;
-	summaryMarkdown += `${formatAsMarkdownTable(diffResults)}\n`;
+	summaryMarkdown += `**${ title }**\n\n`;
+	summaryMarkdown += `${ formatAsMarkdownTable( diffResults ) }\n`;
 }
 
 writeFileSync(
-	join(process.env.WP_ARTIFACTS_PATH, '/performance-results.md'),
+	join( process.env.WP_ARTIFACTS_PATH, '/performance-results.md' ),
 	summaryMarkdown
 );