1370203 - |mach wpt| doesn't provide a helpful error when the hosts entries are missing

Status: RESOLVED FIXED

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

Comment 1

[James Graham [:jgraham]]

Attachment: Bug 1370203 - Change wptrun to raise an exception instead of exiting directly,

This makes integration with other frontends a little easier.

Review commit: https://reviewboard.mozilla.org/r/145780/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/145780/

Comment 2

[James Graham [:jgraham]]

Attachment: Bug 1370203 - Check hosts are configured with |mach wpt|,

For Chrome and Edge we don't have any way to set the DNS configuration
to include web-platform.test, so we need to error if this isn't already set.

Review commit: https://reviewboard.mozilla.org/r/145782/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/145782/

Comment 3

[James Graham [:jgraham]]

Attachment: Bug 1370203 - Don't require setting the wpt hosts configuration for servo,

Servo reads the hosts from a file created at runtime, so this
configuration isn't required.

Review commit: https://reviewboard.mozilla.org/r/145784/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/145784/

Comment 4

[Andreas Farre [:farre]]

Comment on attachment 8874414 [details]
Bug 1370203 - Change wptrun to raise an exception instead of exiting directly,

https://reviewboard.mozilla.org/r/145780/#review149698

Comment 5

[Andreas Farre [:farre]]

Comment on attachment 8874415 [details]
Bug 1370203 - Check hosts are configured with |mach wpt|,

https://reviewboard.mozilla.org/r/145782/#review149700

Comment 6

[Andreas Farre [:farre]]

Comment on attachment 8874416 [details]
Bug 1370203 - Don't require setting the wpt hosts configuration for servo,

https://reviewboard.mozilla.org/r/145784/#review149702

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0ec65d58e0ae
Change wptrun to raise an exception instead of exiting directly, r=farre
https://hg.mozilla.org/integration/autoland/rev/ae7d51d0a455
Check hosts are configured with |mach wpt|, r=farre
https://hg.mozilla.org/integration/autoland/rev/60d3fb7b6810
Don't require setting the wpt hosts configuration for servo, r=farre

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0ec65d58e0ae
https://hg.mozilla.org/mozilla-central/rev/ae7d51d0a455
https://hg.mozilla.org/mozilla-central/rev/60d3fb7b6810