dmADB setupDeviceRoot() expects mkDir to return T/F

RESOLVED FIXED

Status

defect
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: ahal, Assigned: ahal)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

7 years ago
Which is no longer the case as of: https://github.com/mozilla/mozbase/commit/f2cfb5e1ee6dc50449d222da1246ca1e39bfac14

Unfortunately b2g mochitests set self.deviceRoot which trips this line and raises an exception: https://github.com/mozilla/mozbase/blob/master/mozdevice/mozdevice/devicemanagerADB.py#L502
Assignee

Comment 1

7 years ago
Attachment #670092 - Flags: review?(wlachance)
Comment on attachment 670092 [details] [diff] [review]
Patch 1.0 - fix call to mkDir

>+                try:
>+                    self.mkDir(self.deviceRoot)
>+                except:
>+                    print "Unable to create device root %s" % self.deviceRoot
>+                    raise

Except DMError plz. :) Otherwise LGTM
Attachment #670092 - Flags: review?(wlachance) → review+
Assignee

Comment 3

7 years ago
https://github.com/mozilla/mozbase/commit/819610ba4dd37753dfcd930f01b8135838b2abc5
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.