Bug 229577

Summary: [EWS] find-modified-layout-tests step should include .svg tests
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: aakash_jain, ap, jbedard, simon.fraser, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   

Description Ryan Haddad 2021-08-26 10:58:02 PDT
[EWS] find-modified-layout-tests step should include .svg tests
Comment 1 Simon Fraser (smfr) 2021-08-26 10:58:59 PDT
I think the right logic is to re-run any tests whose:
1. test changed
2. expectation changed
3. change in TestExpectations (removed/added/changed lines)
Comment 2 Alexey Proskuryakov 2021-08-26 17:00:36 PDT
It would also be relevant to run tests whose subresources changed, but that's hard.
Comment 3 Alexey Proskuryakov 2021-08-26 17:00:56 PDT
But let's just track .svg in this bug.
Comment 4 Wenson Hsieh 2021-08-26 18:16:47 PDT
(In reply to Alexey Proskuryakov from comment #2)
> It would also be relevant to run tests whose subresources changed, but
> that's hard.

Would this include script? (I imagine that stress-testing all layout tests that import "resources/js-test.js" when adjusting that script could be a bit excessive, for instance).
Comment 5 Radar WebKit Bug Importer 2021-09-02 10:58:17 PDT
<rdar://problem/82683414>