1561196 - Wdspec test for file protocols has too many slashes

Status: RESOLVED FIXED

(Testing :: geckodriver, defect, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Reported via: https://github.com/web-platform-tests/wpt/issues/17385

The following tests use a file:/// prefix instead of file://, which resolves as file:////var/log instead of file:///var/log for a POSIX path like /var/log.

Affected tests:
/tests/navigate_to/navigate.py
/tests/get_current_url/get.py

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=62f6d2dfb679f2d70d3e89e0226bbe855657707a

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Attachment: Bug 1561196 - [wdspec] Fix number of slashes for file:// url tests. r=#webdriver

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b58ed9cc347c
[wdspec] Fix number of slashes for file:// url tests. r=webdriver-reviewers,ato

Comment 4

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

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/17492 for changes under testing/web-platform/tests

Comment 5

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

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 6

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/b58ed9cc347c

Comment 7

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

Upstream PR merged

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Is this a test-only uplift we should take on 68?

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #8)

Is this a test-only uplift we should take on 68?

It's a minor test-only fix which you could take for uplift if you want. It's not mandatory.