Bug 258005

Summary:	EWS pre-existing failure logic shouldn't exempt tests when removing expectations
Product:	WebKit	Reporter:	Alexey Proskuryakov <ap>
Component:	Tools / Tests	Assignee:	Nobody <webkit-unassigned>
Status:	RESOLVED DUPLICATE
Severity:	Normal	CC:	aakash_jain, jbedard, ryanhaddad
Priority:	P2
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=257832

Alexey Proskuryakov

Reported 2023-06-12 22:22:17 PDT

https://github.com/WebKit/WebKit/pull/14694 was intended to fix imported/w3c/web-platform-tests/html/rendering/the-details-element/summary-text-decoration.html, and removed it from TestExpectations. The fix didn't work, so the test started failing in post commit. But EWS didn't tell us about it, because it has incorrectly applied the pre-existing failure exception. https://ews-build.webkit.org/#/builders/39/builds/7479 Ignored pre-existing failure: imported/w3c/web-platform-tests/css/css-fonts/cjk-kerning.html, imported/w3c/web-platform-tests/css/css-ui/text-overflow-028.html, imported/w3c/web-platform-tests/html/rendering/the-details-element/summary-text-decoration.html, imported/w3c/web-platform-tests/html/semantics/disabled-elements/disabled-event-dispatch.tentative.html

Attachments
Add attachment proposed patch, testcase, etc.

Ryan Haddad

Comment 1 2023-06-13 06:43:07 PDT

*** This bug has been marked as a duplicate of bug 255035 ***

Note You need to log in before you can comment on or make changes to this bug.