- Issue created by @tstoeckler
- Merge request !12961Wait for cron to complete at the end of functional tests β (Open) created by tstoeckler
- π©πͺGermany tstoeckler Essen, Germany
I just used that because it's the same as what is used in
\Drupal\Core\Cron::run()
so it seemed appropriate. - πΊπΈUnited States smustgrave
Thatβs fair. No objections to the rest of the changes.
- π¬π§United Kingdom catch
I think we might want to make this even more special cased and do a ::moduleExists('automated_cron') first - nothing else should run cron in an end of request task.
There are other things that can happen end of request too, but:
- they should be a lot quicker than a cron run
- I can't think of any way to generically handle those, except for breaking end of request processing for everyone including outside tests. - π©πͺGermany tstoeckler Essen, Germany
That's fair enough, updated the patch accordingly. Updated the patch for Config Overlay, as well, and the passing pipeline proves that this still works: https://git.drupalcode.org/project/config_overlay/-/merge_requests/44