Add expectations for Safari in for infrastructure/ tests #13638
Conversation
In preparation for running these tests in CI. Verified locally with Safari Technology Preview 67: ``` export no_proxy='*' ./wpt run --metadata infrastructure/metadata/ --channel=preview safari infrastructure/ ./wpt run --metadata infrastructure/metadata/ --channel=preview safari_webdriver infrastructure/ ``` With Safari 12 there are still failures, but we'll not use that in CI.
|
Note that https://bugs.webkit.org/show_bug.cgi?id=190775 is newly filed for this PR. |
|
It would be nice to have this working with Safari 12 too, but I expect it's not really worthwhile; what are the extra failures there? |
|
I didn't check too closely, some service worker tests I think. If it's possible to use the browser version as part of the condition I could set it up, but is it? Then it's depend on my PR to get the version... |
|
No merge_pr_13638 tag was created for this because https://travis-ci.org/web-platform-tests/wpt/jobs/444636306 failed, which is because the GitHub API doesn't realize that commit 40e9266 is for this PR. Given the timing I assume it's because of https://blog.github.com/2018-10-21-october21-incident-report/ so I've manually pushed the tag. |
In preparation for running these tests in CI.
Verified locally with Safari Technology Preview 67:
With Safari 12 there are still failures, but we'll not use that in CI.