Closed Bug 1454535 Opened 7 years ago Closed 7 years ago

[wpt-sync] Sync PR 10499 - [wptrunner] Install Ahem in Sauce Labs Windows VM

Categories

(Testing :: web-platform-tests, enhancement, P4)

Product:
Testing
Component:
web-platform-tests
Type:
enhancement

Tracking

(firefox61 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox61 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 10499 into mozilla-central (this bug is closed when the sync is complete). PR: https://github.com/w3c/web-platform-tests/pull/10499 Details from upstream follow. Mike Pennisi <mike@mikepennisi.com> wrote: > [wptrunner] Install Ahem in Sauce Labs Windows VM > > In gh-10491, @gsnedders [suggested](https://github.com/w3c/web-platform-tests/pull/10491#discussion_r181912073): > > > Can we download files over the tunnel? Does the tunnel already exist at the point the prerun script runs? Because if it does, then we could just download http://web-platform.test:8000/fonts/Ahem.ttf, which would be the right version for the revision being run? > > That sounded like a great idea ([although we would have to construct the URL based on the relevant `env_config` values](https://github.com/w3c/web-platform-tests/commit/0fdf04d1a851e7717d03512533a9f9548c29a8aa)), and I almost included it here. However, the "prerun" script is currently uploaded with the same file name for every test execution. That would cause confusing problems if two test execution tasks used different configurations but the same Sauce Labs account. That might seem unlikely, but it may actually occur on a regular basis in http://builds.wpt.fyi fairly soon. > > I have some thoughts on how we can address that problem, but I recommend we move forward with this simple solution for now because it can only improve result accuracy.
Result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/83b6ccd8b4de [wpt PR 10499] - [wptrunner] Install Ahem in Sauce Labs Windows VM, a=testonly
https://hg.mozilla.org/mozilla-central/rev/83b6ccd8b4de
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.