Fix broken API tests (#34345)

* Bumped jest to v27

* Used jest in packaged node_modules

* Jest bump

* Removed puppeteer related code

* Updated CHANGELOG

* Added changelog

.github/workflows/scripts/prepare-test-summary.js

@@ -2,10 +2,8 @@
  * Script to generate the test results summary to be posted as a GitHub Job Summary and as a PR comment.
  */
 const {
-	E2E_PLAYWRIGHT,
 	API_SUMMARY_PATH,
 	E2E_PW_SUMMARY_PATH,
-	E2E_PPTR_SUMMARY_PATH,
 	SHA,
 	PR_NUMBER,
 } = process.env;
@@ -64,35 +62,7 @@ const getAPIStatsArr = () => {
  * @returns Array of E2E test result stats.
  */
 const getE2EStatsArr = () => {
-	if ( E2E_PLAYWRIGHT === 'true' ) {
+	return getAllureSummaryStats( E2E_PW_SUMMARY_PATH, 'E2E Tests' );
-		return getAllureSummaryStats( E2E_PW_SUMMARY_PATH, 'E2E Tests' );
-	} else {
-		const summary = require( E2E_PPTR_SUMMARY_PATH );
-		const {
-			numPassedTests: passed,
-			numFailedTests: failed,
-			numTotalTests: total,
-			numPendingTests: skipped,
-			numRuntimeErrorTestSuites: broken,
-			numTodoTests: unknown,
-			startTime,
-			testResults,
-		} = summary;
-		const endTime = testResults[ testResults.length - 1 ].endTime;
-		const duration = endTime - startTime;
-		const durationFormatted = getFormattedDuration( duration );
-
-		return [
-			'E2E Tests',
-			passed.toString(),
-			failed.toString(),
-			broken.toString(),
-			skipped.toString(),
-			unknown.toString(),
-			total.toString(),
-			durationFormatted,
-		];
-	}
 };
 
 /**

packages/js/api-core-tests/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Unreleased
 
+## Changed
+- Bumped jest version to v27
+- Used the jest packaged bundled in this module to run tests
+
 ## Added
 - Shipping Zones API Tests
 - Shipping Methods API Tests

packages/js/api-core-tests/bin/wc-api-tests.sh

@@ -51,7 +51,7 @@ cd "$SCRIPTPATH/$(dirname "$REALPATH")/.."
 # Run scripts
 case $1 in
 'test')
-	jest --group=$2 --runInBand
+	node_modules/.bin/jest --group=$2 --runInBand
 	TESTRESULT=$?
 	report
 	;;

packages/js/api-core-tests/package.json

@@ -24,7 +24,7 @@
 	"dependencies": {
 		"allure-commandline": "^2.17.2",
 		"dotenv": "^10.0.0",
-		"jest": "^25.1.0",
+		"jest": "^27.5.1",
 		"jest-allure": "^0.1.3",
 		"jest-runner-groups": "^2.1.0",
 		"postman-collection": "^4.1.0",

plugins/woocommerce/changelog/fix-broken-api-tests

@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+
+Added phpcs:ignore to silence phpstan

plugins/woocommerce/includes/cli/class-wc-cli-com-command.php

@@ -158,6 +158,7 @@ class WC_CLI_COM_Command {
 			}
 		}
 
+		// phpcs:ignore
 		// @todo add URL to application password section
 		if ( empty( $password ) ) {
 			$password = self::ask( 'Connection password:' );