1201195 - [mozrunner] IndexError resolving bindir in application.py

Status: RESOLVED FIXED

(Testing :: Mozbase, defect)

Comment 1

[Andrew Halberstadt [:ahal]]

The root is that application.py in mozrunner is assuming some paths that exist in the context of the B2G.git repo, but that don't exist when running tests from a gaia checkout.

It looks like these paths aren't needed for gaia testing, so we should just make them optional by setting them to None if they don't exist.

Comment 2

[Andrew Halberstadt [:ahal]]

Attachment: MozReview Request: Bug 1201195 - [mozrunner] Fix IndexError on B2G when running without a build (or bindir), r=armenzg

Bug 1201195 - [mozrunner] Fix IndexError on B2G when running without a build (or bindir), r=armenzg

There is an assumption in application.py that there exists a build with a valid bindir. This directory
doesn't exist if no build exist. Before now, this code path hasn't been a problem as it was only used
with built emulators. But now gaia is starting to use it and there might not even be a B2G repo at all!

Comment 3

[Armen [:armenzg]]

Comment on attachment 8656141 [details]
MozReview Request: Bug 1201195 - [mozrunner] Fix IndexError on B2G when running without a build (or bindir), r=armenzg

https://reviewboard.mozilla.org/r/18095/#review16201

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/1dbf8850f7ed

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1dbf8850f7ed