Closed Bug 646462 Opened 13 years ago Closed 13 years ago

Mozmill 1.5.x doesn't remove temporary add-ons folders

Categories

(Testing :: Mozbase, defect)

defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: whimboo, Assigned: whimboo)

Details

(Whiteboard: [mozmill-1.5.3+])

Attachments

(1 file)

If a Mozmill test has to be run with an add-on you can use the --addon command line option to get this add-on installed. For determining the ID Mozmill unpacks the XPI to a temporary location but fails in deleting it in the 1.5.x branch.

Now that we promote our testruns with add-ons we have to make sure to not clutter the users system with those temporary files.
Attached patch Patch v1Splinter Review
Patch with nearly no impact to Mozmill but a big jump to keep the users system clean.
Attachment #523001 - Flags: review?(jhammel)
Just to note, this is a backport from Mozmill 2.
Comment on attachment 523001 [details] [diff] [review]
Patch v1

wfm
Attachment #523001 - Flags: review?(jhammel) → review+
Whiteboard: [mozmill-1.5.x?] → [mozmill-1.5.x+]
Landed on hotfix-1.5 as:
https://github.com/mozautomation/mozmill/commit/bb5de4912698b3d46c424068088bd642440d237e
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-1.5.x+] → [mozmill-1.5.3+]
Looks good with 1.5.3.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: