Closed
Bug 1612527
Opened 5 years ago
Closed 5 years ago
Telemetry test `test_optout_ping_across_sessions` should use `restart_browser`
Categories
(Toolkit :: Telemetry, defect, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla77
| Tracking | Status | |
|---|---|---|
| firefox77 | --- | fixed |
People
(Reporter: travis_, Assigned: ashrivastava)
Details
Attachments
(1 file)
The test: test_optout_ping_across_sessions in test_deletion_request_ping.py, currently calls self.marionette.quit(in_app=True) when it should be calling the restart_browser instead.
|
||
Comment 1•5 years ago
|
||
It was done that way because restart_browser doesn't contain the behaviour necessary for the test. But when I discovered that I should've augmented the testcase with the functionality, not called directly into marionette. (since the test case is the layer at which we abstract things)
Priority: -- → P3
|
||
Updated•5 years ago
|
Assignee: nobody → ashrivastava
Status: NEW → ASSIGNED
Priority: P3 → P1
Pushed by rpierzina@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e4a49c0a575
Refactor browser calls in deletion-request ping test; r=raphael
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox77:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in
before you can comment on or make changes to this bug.
Description
•