Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 60edc560c7b85a4b258d6ed501459fef5eddcbaa
https://github.com/WebKit/WebKit/commit/60edc560c7b85a4b258d6ed501459fef5eddcbaa
Author: Aakash Jain <[email protected]>
Date: 2026-07-21 (Tue, 21 Jul 2026)
Changed paths:
M Tools/CISupport/ews-build/steps.py
M Tools/CISupport/ews-build/steps_unittest.py
Log Message:
-----------
[EWS] Don't blame PR author for pre-existing flaky API test failures
https://bugs.webkit.org/show_bug.cgi?id=318816
rdar://problem/181634454
Reviewed by Ryan Haddad.
A pre-existing flaky API test can fail with the change and then pass on the
clean-tree run
(because it is flaky), which made AnalyzeAPITestsResults report it as a new
failure and blame
the PR author. Use the results-db-filtered failure lists, which already have
pre-existing
failures removed, so such a test is never counted as new. This relies on those
lists being
complete, so we no longer stop the results-db lookup at the first
non-pre-existing failure.
* Tools/CISupport/ews-build/steps.py:
(RunAPITests.filter_api_test_failures_using_results_db):
(AnalyzeAPITestsResults.run):
* Tools/CISupport/ews-build/steps_unittest.py:
(TestAnalyzeAPITestsResults):
(TestFilterAPITestFailuresUsingResultsDB):
Canonical link: https://commits.webkit.org/317657@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
