Closed Bug 1204288 Opened 5 years ago Closed 5 years ago

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


(Testing :: Autophone, defect)

Not set


(firefox43 affected)

Tracking Status
firefox43 --- affected


(Reporter: bc, Assigned: bc)




(1 file)

Attached patch patch v1Splinter Review
Currently, 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 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, 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 --build-location=tinderbox --repo=try --build-url= --test=autophone-crashtest-1

python --build-location=tinderbox --repo=try --build-url= --test=autophone-robocoptest-autophone

You can see before and after patch runs at

Found this testing the different try options from the deployment in bug 1204092.
Attachment #8660389 - Flags: review?(gbrown)
Comment on attachment 8660389 [details] [diff] [review]
patch v1

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

Good find!
Attachment #8660389 - Flags: review?(gbrown) → review+
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.