1034406 - [mozdevice] Do not allow ADBCommand to be instantiated.

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Description

[Bob Clary [:bc] (inactive)]

Attachment: incorporate-bug-1007220.patch

As part of the review process for bug 1007220 when adb.py was incorporated into Autophone, there were concerns about confusion with the use of ADBCommand which was not intended to be directly instantiated. As a result, this patch makes ADBCommand raise a NotImplementedError if instantiated, and adds additional comments to clarify the usage.

A couple of other changes were incorporated:

1. fix to the ADBHost.kill_server in mozdevice.rst where cmds was incorrectly listed as an 
   argument.
2. fix to possible bug in ADBDevice.shell to return the proper exitcode in case the adb process exited with a non-zero exitcode.

wlach, this is the same code as in bug 1007220. If we want to try abstract base classes, can we deal with that in a separate bug?

Comment 1

[William Lachance (:wlach)]

Comment on attachment 8450721 [details] [diff] [review]
incorporate-bug-1007220.patch

Review of attachment 8450721 [details] [diff] [review]:
-----------------------------------------------------------------

Yup, that's fine with me.

Comment 2

[Bob Clary [:bc] (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b2535c273dc4

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/b2535c273dc4