Bug 2064823 - [efficiency] Convert SettingsSitePermissionsTest.clearAllSitePermissionsExceptionsTest to ui/efficiency
Categories
(Firefox for Android :: UI Tests, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox156 | --- | fixed |
People
(Reporter: jajohnson, Assigned: jajohnson)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
Faithful port of clearAllSitePermissionsExceptionsTest (TestRail 2095124) onto ui/efficiency.
The test creates a real exception by allowing the notifications prompt on the test-eng permissions page, then clears it from Settings > Site settings > Exceptions. The Cancel round-trip is kept because it is the point of the test: legacy dismissed the clear-permissions dialog once and re-opened it, proving Cancel clears nothing, before confirming with OK.
Adds exceptions-screen selectors: the exception row, the clear-on-all-sites button, and the dialog title plus its Cancel/OK buttons, all resolved from string resources rather than literals.
Passes on device with no retry.
Context: part of the ui/efficiency UI-test modernization. This is a faithful port of an existing legacy ui/ smoke test onto the ui/efficiency framework — same coverage and assertions, but far less per-test code via a shared page-object / selector / navigation layer, which bends the UI-test maintenance cost curve. Smoke-conversion campaign.
|
Assignee | |
Updated•23 days ago
|
|
|
Assignee | |
Comment 1•23 days ago
|
||
Correction to comment 0: the TestRail case id there was wrong. clearAllSitePermissionsExceptionsTest is case 246976, not 2095124 (which belongs to a different autoplay test in the same legacy class). The patch itself carries the correct id.
|
|
Assignee | |
Comment 2•23 days ago
|
||
Ports clearAllSitePermissionsExceptionsTest (TestRail 2095124). The test creates a real exception by
allowing the notifications prompt on the test-eng permissions page, then clears it from
Settings > Site settings > Exceptions.
The Cancel round-trip is kept because it is the point of the test: legacy dismissed the
clear-permissions dialog once and re-opened it, proving Cancel clears nothing, before confirming with
OK.
Adds exceptions-screen selectors - the exception row, the clear-on-all-sites button, and the dialog
title with its Cancel/OK buttons - all resolved from string resources rather than literals.
Passes on device with no retry.
|
|
Assignee | |
Comment 3•23 days ago
|
||
Ports clearAllSitePermissionsExceptionsTest (TestRail 2095124). The test creates a real exception by
allowing the notifications prompt on the test-eng permissions page, then clears it from
Settings > Site settings > Exceptions.
The Cancel round-trip is kept because it is the point of the test: legacy dismissed the
clear-permissions dialog once and re-opened it, proving Cancel clears nothing, before confirming with
OK.
Adds exceptions-screen selectors - the exception row, the clear-on-all-sites button, and the dialog
title with its Cancel/OK buttons - all resolved from string resources rather than literals.
Passes on device with no retry.
|
||
Updated•23 days ago
|
|
||
Comment 5•22 days ago
|
||
| bugherder | ||
Description
•