Closed Bug 811916 Opened 12 years ago Closed 12 years ago

shellCheckOutput doesn't handle return value of None when calling self.shell

Categories

(Testing :: Mozbase, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jgriffin, Assigned: ahal)

References

Details

Attachments

(1 file)

Blocks: 809437
Blocks: 812149
Assignee: nobody → ahalberstadt
Status: NEW → ASSIGNED
So the fact that we are returning None here: http://mxr.mozilla.org/mozilla-central/source/testing/mozbase/mozdevice/mozdevice/devicemanagerADB.py#153 means that either:

1) There is no return code in the process output
2) There are things being appended after the return code
3) The return code is being corrupted

In all three cases there is something that went wrong. In the best case we need a more robust way of parsing out the return code. In the worst case this is a side effect of a deeper issue in adb or related to how we call adb.
Attached patch Patch 1.0 - FixSplinter Review
I filed bug 813266 to look into why we are hitting that codepath in the first place.

For this bug a quick fix to prevent errors being hidden in the future should be good enough.
Attachment #683263 - Flags: review?(jgriffin)
Attachment #683263 - Flags: review?(jgriffin) → review+
https://github.com/mozilla/mozbase/commit/32538b55548e59b19f045c6df1601942819c46dc

This is minor/rare enough that I won't bother cherry picking into m-c.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
I got a similar error like this, while running a gaia-ui-test and trying to repeat it 50 times. I filed bug 1099985 for it.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: