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
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
*** This bug has been marked as a duplicate of bug 255035 ***