jgraham / arewewptyet Goto Github PK

For WPT tests where the test itself has URL params (?a=b&c=d) in the URL, the wpt.fyi/results page seems to require the question-mark, ampersand, and equals sign to be url-encoded, since wpt.fyi appends its own URL-params at the end of the URL (?label=experimental etc.) which are distinct from the test URL.

We're not handling that properly right now, which means we produce broken wpt.fyi links for such tests.

For example, this CSV file...
https://jgraham.github.io/arewewptyet/interop-2023/interop-2023-webcompat.csv
...has e.g. the following entry:

"/editing/other/insertparagraph-in-inline-editing-host.tentative.html?white-space=pre&display=table-cell&method=enter",18,0,0,""

...which we end up linking like so in our internal spreadsheet:
https://wpt.fyi/results//editing/other/insertparagraph-in-inline-editing-host.tentative.html?white-space=pre&display=table-cell&method=enter

...and that's a broken link. The correct link (taken from the wpt.fyi dashboard itself, e.g. from the directory view for this test) is:
https://wpt.fyi/results/editing/other/insertparagraph-in-inline-editing-host.tentative.html%3Fwhite-space%3Dpre%26display%3Dtable-cell%26method%3Denter

...which is almost what we've got, but with url-encoded question-mark / ampersand / equals (and without the duplicate / after results, though that part doesn't seem to matter).