Closed
Bug 1135275
Opened 9 years ago
Closed 9 years ago
Autophone - uninstall previous org.mozilla packages prior to installing build
Categories
(Testing Graveyard :: Autophone, defect)
Testing Graveyard
Autophone
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: bc, Assigned: bc)
Details
Attachments
(1 file)
1.30 KB,
patch
|
mcote
:
review+
|
Details | Diff | Splinter Review |
When running fennec builds for various releases (beta, aurora, release) vs nightly it is possible for previous test runs to leave a build installed if the autophone worker dies or otherwise does not clean up properly after itself. This can be a problem especially for webappstartup tests which will throw a prompt to the user to select which version of fennec should be used to load the web app if there is more than one version of firefox for android installed. This change enumerates all org.mozilla.(fennec|firefox) packages and uninstalls them, then uninstalls the app which is about to be installed.
Attachment #8567360 -
Flags: review?(mcote)
Comment 1•9 years ago
|
||
Comment on attachment 8567360 [details] [diff] [review] bug-xxxxx-remove-org.mozilla-packages.patch v1 Review of attachment 8567360 [details] [diff] [review]: ----------------------------------------------------------------- ::: worker.py @@ +442,5 @@ > + # with the test. > + mozilla_packages = [ > + p.replace('package:', '') for p in > + self.dm.shell_output("pm list package org.mozilla").split() > + if re.match('package:.*(fennec|firefox)', p)] I feel like there's some way to eliminate the replace() call since you can do group matching in re.match(), but it may not be any nicer.
Attachment #8567360 -
Flags: review?(mcote) → review+
Assignee | ||
Comment 2•9 years ago
|
||
https://github.com/mozilla/autophone/commit/1a94922527c3eba8701b4c6de5d081440da75e6d
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•