Repository navigation
Tests: Cover erase personal data request flow in admin E2E - #15668
shail-mehta wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Both assertions check row visibility rather than the exact requester email link, so they can pass even if the Requester column no longer displays the address.
1 open issue
What changed in this PR
Adds admin end-to-end coverage for creating an Erase Personal Data request, supporting ticket #52895 without running the destructive erasure.
Changes:
- Creates a temporary subscriber and submits a request by username with confirmation email disabled.
- Checks the success notice and requester row before and after searching.
- Adds teardown to delete the request and temporary user.
| File | Description |
|---|---|
tests/e2e/specs/erase-personal-data.test.js |
Adds request-creation coverage with user setup, list checks, and cleanup. |
Review effort: Balanced
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|

Trac ticket: #52895
Adds an admin e2e test for creating an Erase Personal Data request from ticket #52895.
What
erase-personal-data.phpWhy
Ticket #52895 / comment feedback: create a test user, add an erasure request, and verify they appear under Requester—without executing the destructive erase.
How
New Playwright spec:
tests/e2e/specs/erase-personal-data.test.js, following existing admin e2e patterns.Testing instructions
npm run env:startnpm run test:e2e -- tests/e2e/specs/erase-personal-data.test.jsUse of AI Tools