Replace test runner use of devicemanagerADB.py with adb.py

ASSIGNED
Assigned to

Status

Testing
General
ASSIGNED
3 years ago
3 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
While working on bug 1079923, I have found that using the --dm_trans=adb option on the mochitests/runtestsremote.py does not properly execute and retrieve the log files for the test run while using --dm_trans=sut does work.

There are a number of places in the test runner which there is a code branch for sut but none for adb. For example, 

http://dxr.mozilla.org/mozilla-central/source/testing/mochitest/runtestsremote.py#740

http://dxr.mozilla.org/mozilla-central/source/testing/mochitest/runtestsremote.py#750

I haven't discovered the exact missing adb support in the test runners yet, but thought since we have the need to fix adb support in the test runners, that we could take the opportunity to make the change to the adb.py, adb_android.py family in http://dxr.mozilla.org/mozilla-central/source/testing/mozbase/mozdevice/mozdevice/.

This would essentially result in devicemanagerADB being deprecated.

Thoughts?
I second this motion.
I am not opposed, but note that I can run android mochitests via adb (make mochitest-remote) without incident.
(Assignee)

Comment 3

3 years ago
gbrown: that is interesting. I can actually kick off the tests on the phones and see them running, but I don't get the logs back from the device. I'll have to investigate that more.
You need to log in before you can comment on or make changes to this bug.