Test type detection code and document does not match

[WeizhongX]

In the document (https://web-platform-tests.org/writing-tests/file-names.html) it says:
[1] (for test type)These flags must be the last element in the filename before the extension e.g. foo-manual.html will indicate a manual test, but foo-manual-other.html will not.
[2] (for test feature) These flags are preceded by a . in the filename, and must themselves precede any test type flag, but are otherwise unordered.

So according to this, for a manual tentative test, we should write ".tentative-manual.html", but we have plenty of tests named in the reverse order, e.g. "permissions-policy/experimental-features/vertical-scroll-main-frame-manual.tentative.html", "payment-method-basic-card/empty-data-manual.https.html".

In the code [3], it basically only fetch the type flag from the first component after "-".

wpt/tools/manifest/sourcefile.py

Lines 207 to 208 in 8814952

	if "-" in name:
	type_flag = name.rsplit("-", 1)[1].split(".")[0]

[WeizhongX]

Despite what the document says, looks like the common pattern ppl are using is "-(.)*.html", maybe because it will be more eye pleasing to put "-" always before ".", e.g. "vertical-scroll-main-frame-manual.tentative.html" is better than "vertical-scroll-main-frame.tentative-manual.html".

Should we update the document to accept this common practice?

[WeizhongX]

@jgraham WDYT?

[gsnedders]

This changed in cb4fb70; w3c/wpt-tools#123 / w3c/wpt-tools#124.

IMO, given it's been the behaviour for well over seven years, we should just change the docs.

[WeizhongX]

Thanks, uploaded #45696