Bug 220421 – LayoutTestFinder should be in charge of finding expectations for each test

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Sam Sneddon [:gsnedders]]

Following on from bug 220025, as mentioned in comment 5:

> To summarise what [myself and jbedard] discussed in person, my longer term plan was to migrate all the expectation-finding code into the LayoutTestFinder as well. Currently we build up TestInputs in the manager, add the reference_files to them in layout_test_runner, and then construct a DriverOutput with all the relevant expectations in single_test_runner. This doesn't really make sense to have split across three places.
>
> At the moment, after [bug 220025], finding expectations (references among them) remain with the Port, but that should be changed in a later patch.

[Radar WebKit Bug Importer]

<rdar://problem/72894403>

[Sam Sneddon [:gsnedders]]

So, there's at least a few complications here:

We're not just wanting to create TestInput in the LayoutTestFinder itself, as these also contain information related to the specific run (e.g., the used timeout, should_dump_jsconsolelog_in_stderr).

Anything related to the specific run should probably just be passed through to the point where we create the port.driver.DriverInput, and most of what's in port.driver.DriverOuput should (perhaps ironically) be in the new TestInput.

Finally, there's a few places where we currently behave like the device_type can depend on the current worker (e.g. `self._port.target_host(self._worker_number).device_type`) and others where we don't (e.g., `self._driver.host.device_type`). I guess we can treat it as the latter everywhere?

I think stage one at this point is to simplify what we store in TestInput and move some of it into DriverInput, so that TestInput moves towards having less complexity in it, to make it easier for us to move where it gets created?

[Sam Sneddon [:gsnedders]]

Created attachment 449501 [details]
WIP patch

[Sam Sneddon [:gsnedders]]

Pull request: https://github.com/WebKit/WebKit/pull/23546

[EWS]

Committed 276129@main (bee9faa82faa): <https://commits.webkit.org/276129@main>

Reviewed commits have been landed. Closing PR #23546 and removing active labels.