Closed
Bug 1446564
Opened 7 years ago
Closed 7 years ago
Consider changes to root handling in adb.py and test harnesses
Categories
(Testing :: Mozbase, enhancement, P2)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: gbrown, Assigned: gbrown)
References
Details
Some test harnesses need to execute some adb commands as root, but that's not explicit in the code and we haven't verified test harnesses on many physical devices.
See https://bugzilla.mozilla.org/show_bug.cgi?id=1440714#c2.
|
Assignee | |
Updated•7 years ago
|
Priority: -- → P2
|
|
Assignee | |
Comment 1•7 years ago
|
||
:bc - You landed some root-related patches in bug 1440714 and are running more unit tests on device now. Is there still more to do here?
Flags: needinfo?(bob)
|
||
Comment 2•7 years ago
|
||
Let me think about it a bit.
|
|
||
Comment 3•7 years ago
|
||
I think we are in pretty good shape for now, but we have baked root into the frameworks which as you've seen has already caused some problems with people running the tests on their own devices. I think the long term view would be to eliminate the need for rooted devices but that is not something on the near term horizon as far as I can tell and would require help from the android platform folks. I think we can close this out for now as it is not really actionable and consider our future moves elsewhere. If we run into issues where we need to add root=True locally we can deal with that as it appears.
Flags: needinfo?(bob)
|
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → gbrown
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•