Closed Bug 1612527 Opened 1 year ago Closed 1 year ago

Telemetry test `test_optout_ping_across_sessions` should use `restart_browser`

Categories

(Toolkit :: Telemetry, defect, P1)

defect

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.

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
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
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla77
You need to log in before you can comment on or make changes to this bug.