1204288 - Autophone - do not short circuit test downloads based on existence of tests directory

Status: RESOLVED FIXED

(Testing Graveyard :: Autophone, defect)

Description

[Bob Clary [:bc] (inactive)]

Attachment: patch v1

Currently, builds.py checks if the tests subdirectory exists in the build directory to determine if the tests should be downloaded or not. If it does not exist, then all of the test packages which are required for the current job are downloaded and unpacked in the build's tests subdirectory. This can be a problem if a build is downloaded via trigger_runs.py for use with a specific test package (e.g. reftests), then later is downloaded again for a different test package (e.g. mochitests). In this scenario, builds.py will see that the tests directory exists and will not attempt to download the mochitests test package.

The following patch removes the check for the tests directoy and instead checks for the existence of each individual test package while also respecting the force parameter though it is not used anywhere at the moment. I tested this via try with

python trigger_runs.py --build-location=tinderbox --repo=try --build-url=http://ftp-ssl.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-657a16829b91/try-android-api-9/ --test=autophone-crashtest-1

python trigger_runs.py --build-location=tinderbox --repo=try --build-url=http://ftp-ssl.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-657a16829b91/try-android-api-9/ --test=autophone-robocoptest-autophone

You can see before and after patch runs at https://treeherder.allizom.org/#/jobs?repo=try&revision=657a16829b91

Found this testing the different try options from the deployment in bug 1204092.

Comment 1

[Geoff Brown [:gbrown]]

Comment on attachment 8660389 [details] [diff] [review]
patch v1

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

Good find!

Comment 2

[Bob Clary [:bc] (inactive)]

https://github.com/mozilla/autophone/commit/bd00ca865ec1445be56f69981bc8647fbb556ef9