1897403 - moztest.resolve doesn't handle wpt tests with a query string containign a `/`

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect, P1)

Description

[James Graham [:jgraham]]

If you have a test with an id like /foo/bar/baz?foobar=sp/am" thenmoztestis putting that in a group likefoo/bar/baz?foobar=sprather thanfoo/bar/bazbecause we're trying to parse test ids as os pathnames rather than URLs. This later breaks the metadata update which is trying to convert the group name into a filesystem path for thedir.ini` file.

Comment 1

[James Graham [:jgraham]]

Attachment: Bug 1897403 - Handle wpt test id as a URL when computing group name,

wpt test ids are URLs, and can have query components. This means that
using os.path.dirname isn't a correct way to extract different parts
of the URL

In particular a test id like /foo/bar/baz?foobar=sp/am can lead to an
invalid group name like foo/bar/baz?foobar=sp, which breaks the
invariant that we can turn the group name into a filesystem path that
may contain a dir.ini file with metadata for that group.

The fix here is to correctly treat the test id as a URL using the same
logic as in wptrunner.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/778b00802571
Handle wpt test id as a URL when computing group name, r=Sasha

Comment 3

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/778b00802571