Closed Bug 1259775 Opened 8 years ago Closed 7 years ago

Running web-platform-tests-create adds lots of unrelated gunk to manifest again


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

Not set


(Not tracked)



(Reporter: bzbarsky, Unassigned)




(1 file)

Attached patch test.patchSplinter Review
I just ran 

  mach web-platform-tests-create testing/web-platform/tests/html/webappapis/scripting/processing-model-2/window-onerror-with-cross-frame-event-listeners-1.html

I'm attaching the resulting diff for testing/web-platform/meta/MANIFEST.json.  What happened here, exactly?  Should those entries be in there and were incorrectly missing (and if so why)?  Should they not be there and were incorrectly added?
Flags: needinfo?(james)
So I took a look at one of the added things: the entry for the "2dcontext/building-paths/canvas_complexshapes_arcto_001.htm" reftest.  It's present around line 3000 of the file and again around line 34800, the latter under "local_changes"....  The latter entry is the one the mach command added.

Oh, and just to make this easier to reproduce, I was running on top of inbound revision 4907a0296ff9.
As discussed on irc, the problem is that when you add a reftest the manifest update code copies all the reftest data into locla changes rather than diffing. It seems like someone added such a test and updated the manifest by hand, so now automatic additions are adding gunk. There are then two parts to the solution; update the checked-in manifest to be correct per the current tool behaviour and update the behaviour of the tool to be smarter and copy less data.
Flags: needinfo?(james)
I've had this problem in Bug 1247777 a while ago.

STR could be simplified as follows:

1. add any reftest files
2. run "./mach web-platform-tests --manifest-update"

I've filed for this, but not sure if anyone would take care of that. Maybe file it here would be more apporpriate. Since I'm adding wpt reftest recently, my work around is comment the following line while updating manifest automatically:
I'm encountering this every time I run --manifest-update even when not adding reftest files -- in fact, even when not adding any files at all.
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.