Python client should delete marionette session in cleanup()

RESOLVED FIXED in Firefox OS v1.3

Status

Testing
Marionette
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: wlach, Assigned: wlach)

Tracking

unspecified
mozilla29
x86_64
Linux
Points:
---

Firefox Tracking Flags

(b2g-v1.3 fixed)

Details

Attachments

(1 attachment)

Per discussion in bug 960063:

https://bugzilla.mozilla.org/show_bug.cgi?id=960063#c8
https://bugzilla.mozilla.org/show_bug.cgi?id=960063#c9

Let's delete the session if it exists during cleanup.
Created attachment 8362722 [details] [diff] [review]
961874.patch

Simple patch. Will run through try.
Attachment #8362722 - Flags: review?(mdas)
Try run: https://tbpl.mozilla.org/?tree=Try&rev=35f3f349e091

(seems to be some sort of weird thing going on with pushlog, so that link looks broken to me atm... if it still is later tonight, I will repush)
Attachment #8362722 - Flags: review?(mdas) → review+
Ok, the above try push didn't have the right params, but I did another one that worked and shows that we're in good shape. 

https://tbpl.mozilla.org/?tree=Try&rev=395dda7e4b17
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/4488b21a7d59
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4488b21a7d59
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
status-b2g-v1.3: --- → fixed
You need to log in before you can comment on or make changes to this bug.