1338070 - I can't seem to be able to create a *.any.js file that ends up with substitutions in the usual something.sub.html way

Status: NEW

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

Description

[Boris Zbarsky [:bzbarsky]]

I tried creating a test that uses the any.js infrastructure but gets substitutions via the normal .sub.something stuff.

If I try calling the new test html/dom/self-origin.any.sub.js, I get an error during manifest update:

  File "testing/web-platform/tests/tools/manifest/sourcefile.py", line 467, in manifest_items
    TestharnessTest(self, replace_end(self.url, ".any.js", ".any.html")),
  File "testing/web-platform/tests/tools/manifest/sourcefile.py", line 29, in replace_end
    assert s.endswith(old)

If I try calling it html/dom/self-origin.any.sub.js, the manifest is updated properly and we thing we have html/dom/self-origin.sub.any.html and html/dom/self-origin.sub.any.worker.html tests.

However, if I run the html/dom/self-origin.sub.any.html test, it loads /html/dom/self-origin.sub.any.js which doesn't get substitutions performed on it.  not surprising, since afaict our harness only does substitutions on things ending with .sub.someext, and this filename doesn't.

Comment 1

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

I guess it's not terribly surprising that the code gets more brittle as we combine more features into the file name :/

Something like this could work after the next wpt sync: https://github.com/w3c/web-platform-tests/pull/4708/files#diff-fc42d85aa48305641e0c7dcb9d40fee5R1

Still, it's probably worth checking how much work it'd be to make this work.

Comment 2

[James Graham [:jgraham]]

I think maybe we can move the .sub. handling out of the file handler and put it later in the pipeline, just before the content is sent over the wire. I'll investigate.

Comment 3

[James Graham [:jgraham]]

I have a PR up to make .sub.any.js work: https://github.com/w3c/wpt-tools/pull/183