Skip to content

Commit c504b85

Browse files
committed
Merge branch 'dev/woopmnt-5249-e2e-ensure-version-coverage-for-woocommerce-and-php' into dev/woopmnt-5251-e2e-investigate-and-re-enable-woocommerce-blocks-e2e-tests
2 parents b9e6b4e + 07164d3 commit c504b85

File tree

1 file changed

+0
-20
lines changed

1 file changed

+0
-20
lines changed

tests/e2e/README.md

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,6 @@ The E2E tests use Playwright's built-in retry mechanism:
1111
- **Automatic retries**: Failed tests are automatically retried up to 2 times in CI
1212
- **Configurable**: Retries are enabled in CI (`retries: 2`) and disabled locally (`retries: 0`)
1313

14-
15-
### Improved Timeout Handling
16-
- **Increased timeouts**: UI interaction timeouts increased from 100ms to 10 seconds for better reliability
17-
- **Better error handling**: More robust page loading and element waiting strategies
18-
- **DevTools reliability**: Improved devtools page navigation and interaction
19-
2014
### Dynamic Matrix Generation
2115
- **L-1 Policy**: Tests automatically run against the latest WooCommerce version and the L-1 (previous major) version
2216
- **Dynamic version resolution**: Automatically fetches latest WC, RC, and beta versions from WordPress.org API
@@ -234,10 +228,6 @@ When tests fail in CI, Playwright's built-in retry mechanism automatically handl
234228
2. **Retry logs**: Look for retry attempts in the test output
235229
3. **Final results**: The test run will show the final result after all retry attempts
236230

237-
This approach helps distinguish between:
238-
- **Flaky tests**: Tests that fail occasionally but pass on retry
239-
- **Consistent failures**: Tests that fail all retry attempts (indicating real issues)
240-
241231
## Slack integration
242232

243233
The Slack reporter is a custom reporter that sends e2e test failures to a public Slack channel (search Slack channel ID `CQ0Q6N62D`). The reporter is configured to only send the first failure of a test to Slack. If the retry also fails it will not be sent to prevent spamming the channel.
@@ -285,16 +275,6 @@ await page.getByRole( 'button', { name: /submit/i } ).click();
285275
286276
In some cases, you may need to wait for the page to reach a certain load state before interacting with it. You can use `await page.waitForLoadState( 'domcontentloaded' );` to wait for the page to finish loading.
287277

288-
**What timeout values are used for UI interactions?**
289-
290-
The E2E tests use optimized timeout values for better reliability:
291-
- **Global expect timeout**: 20 seconds (configured in `playwright.config.ts`)
292-
- **UI interaction timeouts**: 10 seconds for critical UI elements (buttons, forms, etc.)
293-
- **Page load timeouts**: 120 seconds for test execution
294-
- **Network idle waits**: Used for dynamic content loading
295-
296-
These timeouts have been increased from the previous 100ms values to provide better stability, especially for slower environments or complex UI interactions.
297-
298278
**What is the best way to target elements in the page?**
299279

300280
Prefer the use of [user-facing attribute or test-id locators](https://playwright.dev/docs/locators#locating-elements) to target elements in the page. This will make the tests more resilient to changes to implementation details, such as class names.

0 commit comments

Comments
 (0)