Commit 7b00acc3126 for woocommerce
commit 7b00acc3126d3cfd61e860e4c1c74cfb382b8ae8
Author: Adrian Moldovan <3854374+adimoldovan@users.noreply.github.com>
Date: Fri Oct 30 10:04:00 2026 +0200
Remove unused Allure test reporting (#70390)
diff --git a/.github/workflows/scripts/install-allure.sh b/.github/workflows/scripts/install-allure.sh
deleted file mode 100755
index 33f83140910..00000000000
--- a/.github/workflows/scripts/install-allure.sh
+++ /dev/null
@@ -1,29 +0,0 @@
-#!/bin/bash
-
-set -eo pipefail
-
-# Test Java installation
-java -version
-
-if [[ -z "$DESTINATION_PATH" ]]; then
- echo "::error::DESTINATION_PATH must be set"
- exit 1
-fi
-
-ALLURE_VERSION=2.27.0
-ALLURE_DOWNLOAD_URL=https://github.com/allure-framework/allure2/releases/download/$ALLURE_VERSION/allure-$ALLURE_VERSION.zip
-
-echo "Installing Allure $ALLURE_VERSION in $DESTINATION_PATH"
-wget --no-verbose -O allure.zip $ALLURE_DOWNLOAD_URL \
- && unzip allure.zip -d "$DESTINATION_PATH" \
- && rm -rf allure.zip \
-
-ALLURE_PATH=$(realpath "$DESTINATION_PATH"/allure-$ALLURE_VERSION/bin)
-
-# Test Allure installation
-echo "$ALLURE_PATH"
-export PATH="$ALLURE_PATH:$PATH"
-allure --version
-
-# Add Allure in Github PATH to make it available to all subsequent actions in the current job
-echo "$ALLURE_PATH" >> "$GITHUB_PATH"
diff --git a/.gitignore b/.gitignore
index 69625500e62..db930ba6a66 100644
--- a/.gitignore
+++ b/.gitignore
@@ -91,10 +91,6 @@ phpcs-report.xml
#PHP Unit
.phpunit.result.cache
-# Allure directories
-allure-report
-allure-results
-
# Code analyzer output and env
changes.json
.env
diff --git a/plugins/woocommerce/changelog/remove-allure-playwright b/plugins/woocommerce/changelog/remove-allure-playwright
new file mode 100644
index 00000000000..21be1d4b68f
--- /dev/null
+++ b/plugins/woocommerce/changelog/remove-allure-playwright
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+Comment: Remove unused Allure test reporting dependencies; no production change.
+
diff --git a/plugins/woocommerce/client/blocks/CLAUDE.md b/plugins/woocommerce/client/blocks/CLAUDE.md
index 009223eb142..5971cf9fa8f 100644
--- a/plugins/woocommerce/client/blocks/CLAUDE.md
+++ b/plugins/woocommerce/client/blocks/CLAUDE.md
@@ -220,7 +220,7 @@ Webpack writes directly to `plugins/woocommerce/assets/client/blocks/` so PHP en
- Projects: `blocks-parallel` (default) and `blocks-serial` (specs in `blocksSerialSpecs`, database reset after each test). Write new specs parallel-safe, or use a test lock; add them to `blocksSerialSpecs` only as a last resort. See `../../tests/e2e/README.md#parallel-locked-and-serial-specs`.
- Test themes: `block-theme`, `classic-theme`, `block-theme-with-templates`
- Setup script: `../../tests/e2e/bin/blocks/test-env-setup.sh`
-- Uses MSW for API mocking, Allure for reporting
+- Uses MSW for API mocking
### PHP Tests
diff --git a/plugins/woocommerce/client/blocks/changelog/remove-allure-playwright b/plugins/woocommerce/client/blocks/changelog/remove-allure-playwright
new file mode 100644
index 00000000000..21be1d4b68f
--- /dev/null
+++ b/plugins/woocommerce/client/blocks/changelog/remove-allure-playwright
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+Comment: Remove unused Allure test reporting dependencies; no production change.
+
diff --git a/plugins/woocommerce/client/blocks/package.json b/plugins/woocommerce/client/blocks/package.json
index 1ca4a150909..3fed8de8741 100644
--- a/plugins/woocommerce/client/blocks/package.json
+++ b/plugins/woocommerce/client/blocks/package.json
@@ -158,7 +158,6 @@
"@wordpress/stylelint-config": "catalog:tooling",
"@wordpress/theme": "catalog:wp-bundled",
"ajv-cli": "3.3.x",
- "allure-playwright": "^2.9.2",
"autoprefixer": "10.4.14",
"babel-jest": "29.5.x",
"babel-loader": "9.2.x",
diff --git a/plugins/woocommerce/package.json b/plugins/woocommerce/package.json
index 1616e512aa4..40f23661f6e 100644
--- a/plugins/woocommerce/package.json
+++ b/plugins/woocommerce/package.json
@@ -1532,8 +1532,6 @@
"@wordpress/browserslist-config": "next",
"@wordpress/e2e-test-utils-playwright": "catalog:wp-min",
"@wordpress/env": "catalog:tooling",
- "allure-commandline": "^2.32.2",
- "allure-playwright": "^3.1.0",
"autoprefixer": "9.8.6",
"axios": "^0.33.0",
"config": "3.3.7",
diff --git a/plugins/woocommerce/tests/e2e/README.md b/plugins/woocommerce/tests/e2e/README.md
index c2c4f849388..bdb54acfc82 100644
--- a/plugins/woocommerce/tests/e2e/README.md
+++ b/plugins/woocommerce/tests/e2e/README.md
@@ -267,11 +267,12 @@ In CI, the full suite runs as part of the existing "Core e2e tests" job — no s
## Test reports
-The tests would generate three kinds of reports after the run:
+The tests generate these reports after the run:
-- A Playwright HTML report.
- A Playwright JSON report.
-- Allure results.
+- A [CTRF](https://ctrf.io/) JSON report.
+- A Playwright HTML report, on local runs only.
+- A JUnit report, in CI only.
By default, they are saved inside the `test-results` folder.
@@ -289,45 +290,6 @@ pnpm exec playwright show-report tests/e2e/test-results/playwright-report
For more details about the Playwright HTML report, see
their [HTML Reporter](https://playwright.dev/docs/test-reporters#html-reporter) documentation.
-### Viewing the Allure report
-
-This assumes that you're already familiar with reports generated by
-the [Allure Framework](https://github.com/allure-framework), particularly:
-
-- What the `allure-results` and `allure-report` folders are, and how they're different from each other.
-- Allure commands like `allure generate` and `allure open`.
-
-Use the `allure generate` command to generate an HTML report from the `allure-results` directory created at the end of
-the test run. Then, use the `allure open` command to open it on your browser. For example, assuming that:
-
-- You're at the root of the WooCommerce monorepo
-- You want to generate the `allure-report` folder in `plugins/woocommerce/tests/e2e/test-results`
-
-Then you would need to use the following commands:
-
-```bash
-pnpm exec allure generate --clean plugins/woocommerce/tests/e2e/test-results/allure-results --output plugins/woocommerce/tests/e2e/test-results/allure-report
-pnpm exec allure open plugins/woocommerce/tests/e2e/test-results/allure-report
-```
-
-A browser window should open the Allure report.
-
-If you're on [WSL](https://learn.microsoft.com/en-us/windows/wsl/about) however, you might get this message right after
-running the `allure open` command:
-
-```bash
-Starting web server...
-2022-12-09 18:52:01.323:INFO::main: Logging initialized @286ms to org.eclipse.jetty.util.log.StdErrLog
-Can not open browser because this capability is not supported on your platform. You can use the link below to open the report manually.
-Server started at <http://127.0.1.1:38917/>. Press <Ctrl+C> to exit
-```
-
-In this case, take note of the port number (38917 in the example above) and then use it to navigate to localhost. Taking
-the example above, you should be able to view the Allure report on localhost:38917 in your browser.
-
-To know more about the `allure-playwright` integration, see
-their [GitHub documentation](https://github.com/allure-framework/allure-js/tree/master/packages/allure-playwright).
-
## Debugging tests
For Playwright debugging, follow [Playwright's documentation](https://playwright.dev/docs/debug).
diff --git a/plugins/woocommerce/tests/e2e/playwright.config.ts b/plugins/woocommerce/tests/e2e/playwright.config.ts
index 8d3e773c9c5..4ac7a3e02b1 100644
--- a/plugins/woocommerce/tests/e2e/playwright.config.ts
+++ b/plugins/woocommerce/tests/e2e/playwright.config.ts
@@ -38,14 +38,6 @@ export const CONSUMER_KEY = { name: '', key: '', secret: '' };
const reporter = [
[ 'list' ],
- [
- 'allure-playwright',
- {
- resultsDir: `${ TESTS_ROOT_PATH }/test-results/allure-results`,
- detail: true,
- suiteTitle: true,
- },
- ],
[
'json',
{
@@ -63,10 +55,6 @@ const reporter = [
repositoryName: process.env.GITHUB_REPOSITORY || '',
},
],
- [
- `${ TESTS_ROOT_PATH }/reporters/environment-reporter.ts`,
- { outputFolder: `${ TESTS_ROOT_PATH }/test-results/allure-results` },
- ],
];
if ( process.env.CI ) {
diff --git a/plugins/woocommerce/tests/e2e/reporters/environment-reporter.ts b/plugins/woocommerce/tests/e2e/reporters/environment-reporter.ts
deleted file mode 100644
index 2bfd9abd3a6..00000000000
--- a/plugins/woocommerce/tests/e2e/reporters/environment-reporter.ts
+++ /dev/null
@@ -1,73 +0,0 @@
-/**
- * External dependencies
- */
-import { request } from '@playwright/test';
-import fs from 'fs';
-import path from 'path';
-
-/**
- * Internal dependencies
- */
-import { admin } from '../test-data/data';
-
-class EnvironmentReporter {
- reportOptions: { outputFolder?: string };
-
- constructor( options: { outputFolder?: string } ) {
- this.reportOptions = options;
- }
-
- async onEnd() {
- console.log( '::debug::Getting environment details' );
- const { outputFolder } = this.reportOptions;
-
- if ( ! outputFolder ) {
- console.error(
- 'Error getting environment info: no output folder specified!'
- );
- return;
- }
-
- const { BASE_URL, CI } = process.env;
- let environmentData = '';
-
- if ( CI ) {
- environmentData += `CI=${ CI }`;
- }
-
- try {
- const wpApi = await request.newContext( {
- baseURL: BASE_URL,
- extraHTTPHeaders: {
- Authorization: `Basic ${ Buffer.from(
- `${ admin.username }:${ admin.password }`
- ).toString( 'base64' ) }`,
- },
- } );
-
- const info = await wpApi.get( `./wp-json/e2e-environment/info` );
-
- if ( info.ok() ) {
- const data = await info.json();
- for ( const [ key, value ] of Object.entries( data ) ) {
- // We need to format the values to be compatible with the Java properties file format
- environmentData += `\n${ key
- .replace( / /g, '\\u0020' )
- .replace( /:/g, '-' ) }=${ value }`;
- }
- }
- } catch ( err ) {
- console.error( `Error getting environment info: ${ err }` );
- }
-
- const filePath = path.resolve( outputFolder, 'environment.properties' );
-
- try {
- fs.writeFileSync( filePath, environmentData );
- } catch ( err ) {
- console.error( `Error writing environment.properties: ${ err }` );
- }
- }
-}
-
-export default EnvironmentReporter;
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index ece613bfcec..c8e26df358c 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -2588,12 +2588,6 @@ importers:
'@wordpress/env':
specifier: catalog:tooling
version: 11.17.0(@types/node@24.12.2)
- allure-commandline:
- specifier: ^2.32.2
- version: 2.38.1
- allure-playwright:
- specifier: ^3.1.0
- version: 3.7.1(@playwright/test@1.63.0)
autoprefixer:
specifier: 9.8.6
version: 9.8.6
@@ -3711,9 +3705,6 @@ importers:
ajv-cli:
specifier: 3.3.x
version: 3.3.0
- allure-playwright:
- specifier: ^2.9.2
- version: 2.15.1
autoprefixer:
specifier: 10.4.14
version: 10.4.14(postcss@8.5.26)
@@ -11060,29 +11051,6 @@ packages:
ajv@8.18.0:
resolution: {integrity: sha512-PlXPeEWMXMZ7sPYOHqmDyCJzcfNrUr3fGNKtezX14ykXOEIvyK81d+qydx89KY5O71FKMPaQ2vBfBFI5NHR63A==}
- allure-commandline@2.38.1:
- resolution: {integrity: sha512-qkoEz8/Y+K6XRlkJFmPEsT4bJFa2nHmYMkp6aEg6HaM4uewpJRI0jJsQhVWl8xZ6k6VtVr3vWnsAMPL5LPthvA==}
- hasBin: true
-
- allure-js-commons@2.15.1:
- resolution: {integrity: sha512-5V/VINplbu0APnfSZOkYpKOzucO36Q2EtTD1kqjWjl7n6tj7Hh+IHCZsH3Vpk/LXRDfj9RuXugBBvwYKV5YMJw==}
-
- allure-js-commons@3.7.1:
- resolution: {integrity: sha512-S8tFymEzgWpp3ITGFu01z2+6KYdiNZC3sX8FbJaNgLoLWBHDBNQ0lq4lTQgkVjV6IN4TIUrnYfa2YuytWpv8WA==}
- peerDependencies:
- allure-playwright: 3.7.1
- peerDependenciesMeta:
- allure-playwright:
- optional: true
-
- allure-playwright@2.15.1:
- resolution: {integrity: sha512-P1Uu1j/ptDHdYp3V5ZAeBZyt33+L+OQu0otUIEl/zkOcv0KRycHqlHwC0GEJmpgnLKvVP7s+K37LcLoSUUj3Cg==}
-
- allure-playwright@3.7.1:
- resolution: {integrity: sha512-uEuVz/VJWWEWvMV9iwsxSXJ3Y1RejNG8CUM13HyvMiQDfLI8xHCx7JWNSfGrBVIWVfo6+jsySpZ4Clu1EKXgYw==}
- peerDependencies:
- '@playwright/test': '>=1.53.0'
-
ansi-align@2.0.0:
resolution: {integrity: sha512-TdlOggdA/zURfMYa7ABC66j+oqfMew58KpJMbUlH3bcZP1b+cBHIHDDn5uH9INsxrHBPjsqM0tDB4jPTF/vgJA==}
@@ -17290,10 +17258,6 @@ packages:
prop-types@15.8.1:
resolution: {integrity: sha512-oj87CgZICdulUohogVAR7AjlC0327U4el4L6eAvOqCeudMDVU0NThNaV+b9Df4dXgSP1gXMTnPdhfe/2qDH5cg==}
- properties@1.2.1:
- resolution: {integrity: sha512-qYNxyMj1JeW54i/EWEFsM1cVwxJbtgPp8+0Wg9XjNaK6VE/c4oRi6PNu5p7w1mNXEIQIjV5Wwn8v8Gz82/QzdQ==}
- engines: {node: '>=0.10'}
-
proto-list@1.2.4:
resolution: {integrity: sha512-vtK/94akxsTMhe0/cbfpR+syPuszcuwhqVjJq26CuNDgFGj682oRBXOP5MJpv2r7JtE8MsiepGIqvvOTBwn2vA==}
@@ -32688,29 +32652,6 @@ snapshots:
json-schema-traverse: 1.0.0
require-from-string: 2.0.2
- allure-commandline@2.38.1: {}
-
- allure-js-commons@2.15.1:
- dependencies:
- md5: 2.3.0
- properties: 1.2.1
- strip-ansi: 5.2.0
-
- allure-js-commons@3.7.1(allure-playwright@3.7.1(@playwright/test@1.63.0)):
- dependencies:
- md5: 2.3.0
- optionalDependencies:
- allure-playwright: 3.7.1(@playwright/test@1.63.0)
-
- allure-playwright@2.15.1:
- dependencies:
- allure-js-commons: 2.15.1
-
- allure-playwright@3.7.1(@playwright/test@1.63.0):
- dependencies:
- '@playwright/test': 1.63.0
- allure-js-commons: 3.7.1(allure-playwright@3.7.1(@playwright/test@1.63.0))
-
ansi-align@2.0.0:
dependencies:
string-width: 2.1.1
@@ -40032,8 +39973,6 @@ snapshots:
object-assign: 4.1.1
react-is: 16.13.1
- properties@1.2.1: {}
-
proto-list@1.2.4: {}
proxy-addr@2.0.7: