From c2616b5498c0d21f46e8839418dd24bf75eeb373 Mon Sep 17 00:00:00 2001 From: Kevin Batdorf Date: Mon, 3 Aug 2026 18:34:45 +0700 Subject: [PATCH 1/2] Retry failed tests once in CI to absorb Playground flakes WP Playground intermittently 500s or crashes mid-test in CI across the sibling block repos, sending false-alarm failure emails while the rest of the matrix passes. One CI retry absorbs those one-offs; a real regression still fails after the retry. Local runs keep retries: 0 to fail fast while iterating. Update the testing rules, which previously banned retries outright, to match. Co-Authored-By: Claude Fable 5 --- .claude/rules/testing.md | 2 +- playwright.config.ts | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/.claude/rules/testing.md b/.claude/rules/testing.md index 03d646e..f18ae02 100644 --- a/.claude/rules/testing.md +++ b/.claude/rules/testing.md @@ -19,7 +19,7 @@ path: tests/** ## Things That Break - **Never use `page.goBack()`.** WP Playground crashes. Split into separate tests instead. -- **No `retries` in playwright config.** Retries mask real failures. Use `retries: 0`. +- **Retries are CI-only.** `retries: process.env.CI ? 1 : 0`. One CI retry absorbs WP Playground flakes (random 500s, crashes); local runs stay at 0 so real failures surface. - **Tests share state within a spec.** Settings persist between tests in the same spec file. Explicitly reset anything a previous test might have changed. - **Isolate heavy tests.** Complex tests (WASM compilation, multi-step workflows) should get their own directory with a `blueprint.json` so they run in a fresh Playground instance. diff --git a/playwright.config.ts b/playwright.config.ts index 85ae128..e06dbc2 100644 --- a/playwright.config.ts +++ b/playwright.config.ts @@ -44,7 +44,7 @@ const portMap = Object.fromEntries( export default defineConfig({ forbidOnly: !!process.env.CI, - retries: 0, + retries: process.env.CI ? 1 : 0, workers: 1, reporter: 'html', use: { From 7f992d4f3d2050541c03a9b27c2f00b27b8d5094 Mon Sep 17 00:00:00 2001 From: Kevin Batdorf Date: Mon, 3 Aug 2026 20:26:44 +0700 Subject: [PATCH 2/2] Mark as tested up to WordPress 7.1 Co-Authored-By: Claude Fable 5 --- readme.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/readme.txt b/readme.txt index e3c855e..8fab9b4 100644 --- a/readme.txt +++ b/readme.txt @@ -1,7 +1,7 @@ === Pattern CSS - The CSS Editor For Blocks === Contributors: kbat82 Tags: block, css, styles, inline, editor -Tested up to: 7.0 +Tested up to: 7.1 Stable tag: 1.5.6 License: GPL-2.0-or-later License URI: https://www.gnu.org/licenses/gpl-2.0.html