Bug 258005

Summary: EWS pre-existing failure logic shouldn't exempt tests when removing expectations
Product: WebKit Reporter: Alexey Proskuryakov <ap>
Component: Tools / TestsAssignee: 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
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.