Closed Bug 1583012 Opened 2 months ago Closed 27 days ago

web-platform-tests sometimes nests the contents of the whole file in the metadata multiple times

Categories

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

Version 3
defect

Tracking

(firefox71 fixed)

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: birtles, Assigned: dheiberg)

Details

Attachments

(3 files)

e.g.

https://searchfox.org/mozilla-central/rev/962c605a9c72dcec644c05bb80c1ffe790a28037/testing/web-platform/meta/css/css-transforms/animation/translate-interpolation.html.ini#106

(contains a nested [translate-interpolation.html] section under an existing [translate-interpolation.html] section)

produced by the merge from bug 1574607.

There are more examples in:

https://hg.mozilla.org/mozilla-central/rev/37cba102bc14

In some cases the nested block appears multiple times in the same file.

Note that I dropped a bunch of these in bug 1578125 but there may be more still left in the tree.

I think this is fallout from a bug in the mergedriver that the sync bot was using for ini files. The bug is fixed, and the driver is currently disabled for unrelated reasons. But we should clean up any leftovers in the tree; they aren't harmful but are confusing.

Note that this isn't coming from mach wpt-update so that's perfectly safe to use.

One thing to do here would be to just make mach wpt-update --full check for children of subtests and remove them when we are doing the other orphan cleanup. That would prevent people adding this kind of noise in the future.

dheiberg: Do you fancy modifying https://searchfox.org/mozilla-central/source/testing/web-platform/tests/tools/wptrunner/wptrunner/metadata.py#630-645 so that we check seen_subtests for children and add them to the list to remove if any?

Flags: needinfo?(dheiberg)

jgraham: sure, I'll have a look at that

Flags: needinfo?(dheiberg)

The priority flag is not set for this bug.
:jgraham, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(james)

dheiberg: Did you have an incomplete patch for this somewhere? I can try and ressurect it along with some other changes I need to make.

Flags: needinfo?(james) → needinfo?(dheiberg)
Attached patch patch.diffSplinter Review

jgraham: this is what I was trying but it did not seem to work. Or I was running it incorrectly?

Assignee: nobody → dheiberg
Flags: needinfo?(dheiberg) → needinfo?(james)
Flags: needinfo?(james)
Priority: -- → P2
Pushed by dheiberg@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2b068107c651
Ensure removing a node from a manifest removes the underlying tree node, r=dheiberg
https://hg.mozilla.org/integration/autoland/rev/06b945715ba3
Remove subsubtest junk when doing a full metadata update, r=dheiberg
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/19730 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 27 days ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.