Closed Bug 1503335 Opened Last year Closed 3 months ago

assertion count failing in tests under html/semantics/embedded-content/the-area-element/ after changing number of chunks for wpt

Categories

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

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: marco, Assigned: marco)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Attached patch Patch (obsolete) — Splinter Review
Attachment #9021263 - Flags: review?(ryanvm)
Comment on attachment 9021263 [details] [diff] [review]
Patch

r+ to get past the current tree closure, but marking it leave-open so we can have a better investigation into what's going on. A change in chunking shouldn't cause a test to start exploding assertions when it didn't previously.
Attachment #9021263 - Flags: review?(ryanvm) → review+
Pushed by ccoroiu@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7e06b3de1158
Disable area-coords.html test on Windows debug as its assertion count is too high. r=RyanVM
After disabling area-coords.html, another test starts failing:
> TEST-UNEXPECTED-FAIL | /html/semantics/embedded-content/the-area-element/area-download-click.html | assertion count 102 is more than expected 0 assertions
Summary: html/semantics/embedded-content/the-area-element/area-coords.html assertion count failing → assertion count failing in tests under html/semantics/embedded-content/the-area-element/ after changing number of chunks for wpt
Any idea what's wrong here? Maybe we should disable all the tests under html/semantics/embedded-content/the-area-element/?
Flags: needinfo?(james)
I don't know what the original problem was so it's hard to debug, but disabling the tests definitely seems wrong; if we can't actually fix the issue we can mark all these tests as having allowed asserts (I have no idea why a chunking change would make a difference here either).
Flags: needinfo?(james)
Assignee: nobody → mcastelluccio
Attachment #9021263 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #9022714 - Flags: review?(james)
Attachment #9022714 - Flags: review?(james) → review+
Pushed by mcastelluccio@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/964dfd7fce11
Allow a maximum of 102 assertion failures in area-coords.html. r=jgraham

The leave-open keyword is there and there is no activity for 6 months.
:marco, maybe it's time to close this bug?

Flags: needinfo?(mcastelluccio)
Flags: needinfo?(mcastelluccio)

The leave-open keyword is there and there is no activity for 6 months.
:marco, maybe it's time to close this bug?

Flags: needinfo?(mcastelluccio)
Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Flags: needinfo?(mcastelluccio)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.