docs: point Playwright test examples to support/fixtures/base - #32774
docs: point Playwright test examples to support/fixtures/base#32774detail-app[bot] wants to merge 1 commit into
Conversation
|
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
✅ Playwright Results — workflow succeededValidated commit ✅ 557 passed · ❌ 0 failed · 🟡 0 flaky · ⏭️ 0 skipped · 🧰 0 lifecycle flaky PerformanceBlocking targets: ✅ met · Optimization targets: 🟡 in progress Shard-job maxima below are not the full workflow wall time; the linked run includes build, fixture, planning, and reporting. 🕒 Full workflow signal wall (to summary) 51m 26s ⏱️ Max setup 4m 35s · max shard execution 23m 8s · max shard-job elapsed before upload 26m 36s · reporting 5s 🌐 234.72 requests/attempt · 2.82 app boots/UI scenario · 22.41% common-shard skew Optimization targets still in progress:
How to debug locally# Download playwright-test-results-<shard> artifact and unzip
npx playwright show-trace path/to/trace.zip # view trace |
Describe your changes:
Corrected three living Playwright docs whose copy-paste example imports
still pulled
test/expectfrom@playwright/test, the conventionbefore
support/fixtures/base.tsbecame the suite's single entry point.The examples now import from the new base fixture module (with each
file's existing relative path), matching the 224 specs migrated in the
source change.
A contributor following the old example would write a spec that
bypasses
installServerLoadReducersand the boot-endpoint read-throughcache, silently opting out of the server-load reductions the rest of
the suite relies on — no lint rule enforces the new import source, so
the docs are the only guide.
Introduced by commit 90e9a78 (@pmbrull, #32594)
Type of change:
High-level design:
N/A — small change.
Tests:
Not applicable — documentation only.
UI screen recording / screenshots:
Not applicable.
Checklist:
Doc Drift PRs can be configured here.