Add CLI option to cause Marionette not to use temporary userdata.img files

RESOLVED FIXED in mozilla16

Status

Product:
Testing
Component:
Marionette
Type:
defect
Status:
RESOLVED FIXED
Opened:
7 years ago
Updated:
7 years ago

People

(Reporter: jgriffin, Assigned: mihneadb)

Tracking

Version:
unspecified
Target:
mozilla16
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

udpate to latest changes
6.57 KB, patch
jgriffin
: review+
Details | Diff | Splinter Review 
Currently, when Marionette launches the emulator, it copies userdata.img to a temp location, which means that any changes to emulator files do not persist between emulator invocations.

This is good for running tests, but can be suboptimal for debugging.

We should add a Marionette command-line option that causes it not to copy userdata.img, but use the file in-place.
Attachment #641266 - Flags: review?(jgriffin)
Comment on attachment 641266 [details] [diff] [review]
add support for --emulatorImg in Marionette

Review of attachment 641266 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks,looks good!
Attachment #641266 - Flags: review?(jgriffin) → review+
Comment on attachment 641266 [details] [diff] [review]
add support for --emulatorImg in Marionette

This patch doesn't apply cleanly on top of mozilla-central, due to your earlier xUnit patch.  Can you rebase?
Attachment #641266 - Flags: review+ → review-
Attachment #641266 - Attachment is obsolete: true
Attachment #641293 - Flags: review?(jgriffin)
Comment on attachment 641293 [details] [diff] [review]
udpate to latest changes

Thanks, this one applied cleanly!

http://hg.mozilla.org/integration/mozilla-inbound/rev/56141758a0f0
Attachment #641293 - Flags: review?(jgriffin) → review+
https://hg.mozilla.org/mozilla-central/rev/56141758a0f0
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in before you can comment on or make changes to this bug.