-
[NA] [QA] fix: realign dataset & test-suite E2E POMs with revamped detail header (#7274)
frostbyte_neo 发布于
2026-06-29 14:07:16 +00:00 - [NA] [QA] fix: realign dataset & test-suite E2E POMs with revamped detail header
PR #7254 (child-page top-section revamp) renamed the detail-page tabs
(Items -> Records / Test cases), turned the header actions into icon-only
tooltip buttons with no accessible name, renamed the primary add to
Record / Test case, and renamed the empty state and add-record dialog.
The dataset and test-suite smoke POMs still targeted the old strings, so
their waitForReady / applyTestSettings / openInPlayground steps timed out
in the post-merge t1 run.- FE: add data-testid to the icon-only header buttons (settings, run-in,
expand) and the primary add button, plus the empty-state add CTA, so
the suite can target them stably instead of by an absent accessible name. - POMs: match the renamed tabs/tabpanels, empty-state text, and add-record
dialog; target the new testids for Test settings and Run-in; drop the
now-dead preferTestid helper and the removed dataset-items-add-button.
Verified locally against a source-served frontend: dataset-crud-smoke
passes; the test-suite selector changes confirmed live via MCP (Test cases
tab, Test settings dialog, Run-in -> Open in Playground).Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
- test(e2e): wrap dataset & test-suite POM methods in test.step
conventions.md mandates each POM method body run inside a returning
test.step() so the Playwright trace and Allure timeline read as named
steps. These three POMs predated the convention (zero usages); bring
them into compliance now that the file is being touched.Behaviour-preserving — methods still return through the step callback;
dataset-crud-smoke verified green locally.Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com
Co-authored-by: Claude Opus 4.8 noreply@anthropic.com
下载附件