Closed Bug 1446564 Opened 6 years ago Closed 6 years ago

Consider changes to root handling in adb.py and test harnesses

Categories

(Testing :: Mozbase, enhancement, P2)

Product:
Testing
Component:
Mozbase
Version:
Version 3
Type:
enhancement

Tracking

(Not tracked)

Status:
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.
Priority: -- → P2
: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)
Let me think about it a bit.
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: nobody → gbrown
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.