I did not get rid of the redundant code in SingleExecutor::check() because after all the TODOs have been fixed and the executors are aligned and use the shared code it should have been implicitly fixed.

I will also fix up the SingleExecutor before I start applying the fixes to the other executors.

The tests fail because they are run in parallel and use the same file names. I will try to come up with something that provides incremental files names with a unique base.

The tests fail because they are run in parallel and use the same file names. I will try to come up with something that provides incremental files names with a unique base.

They only consistently fail on MacOS with CMake. If my assumption was true they should fail randomly for any CMake build. So there might be another issue at bay here.

Turns out the tests were interfering with each other so I add a cost to one of them so they don't run at the same time.

I first thought I might have hit UB since I was modifying an object after it was const referenced but I never hit that code. I fixed it anyways.

It seems we have broken the support for qml files

So I would suggest that we fix it first.. see #5001