1437550 - [wpt-sync] PR 9480 - Fix hostnames

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

I believe there was an error in gh-8614. Since it removed the `hosts` property, [the templated `config.json` file which referenced that property](https://github.com/w3c/web-platform-tests/blob/9b23a0cbf6476d3f1d0b25b9bda8e5249725b00b/tools/wptrunner/wptrunner/config.json#L1) can no longer be [expanded](https://github.com/w3c/web-platform-tests/blob/9b23a0cbf6476d3f1d0b25b9bda8e5249725b00b/tools/wptrunner/wptrunner/environment.py#L130-L132). Here's an example of the error I encountered when running `./wpt run chrome` locally:

    Traceback (most recent call last):
      File "./wpt", line 5, in <module>
        wpt.main()
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wpt/wpt.py", line 132, in main
        rv = script(*args, **kwargs)
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wpt/run.py", line 419, in run
        rv = run_single(venv, **kwargs) > 0
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wpt/run.py", line 426, in run_single
        return wptrunner.start(**kwargs)
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wptrunner/wptrunner/wptrunner.py", line 295, in start
        return not run_tests(**kwargs)
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wptrunner/wptrunner/wptrunner.py", line 185, in run_tests
        env_extras) as test_environment:
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wptrunner/wptrunner/environment.py", line 96, in __enter__
        self.config = self.load_config()
      File "/home/mike/projects/bocoup/google-wpt/web-platform-tests/tools/wptrunner/wptrunner/environment.py", line 132, in load_config
        local_config = json.loads(data % self.options)
    KeyError: 'host'

This hasn't interrupted things on CI because [the Firefox configuration object still includes the `hosts`
property](https://github.com/w3c/web-platform-tests/blob/9b23a0cbf6476d3f1d0b25b9bda8e5249725b00b/tools/wptrunner/wptrunner/browsers/firefox.py#L120), and the builds for the effected browsers are configured as "allowed failures". [Those builds have recently been failing for this same reason](https://travis-ci.org/w3c/web-platform-tests/jobs/339769009).

The value in the Firefox configuration (`127.0.0.1`) differs from the value shared by all other browsers (`web-platform.test`). gh-8614 is fairly clear in its goal to reduce variability, so I'm assuming this disparity is not necessary/desirable. Since I can't think of any reason why the host should be different for Firefox, I've authored the patch to normalize on `web-platform.test`). If I'm wrong about that, then we can omit the second commit on this branch and apply only the first commit (which reverts the
original change).

Comment 1

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6d7cc54dce31
[wpt PR 9480] - Fix hostnames, a=testonly

Comment 2

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/6d7cc54dce31