- ๐ฉ๐ชGermany donquixote
This might be true.
But I actually had an example where visiting /cron had a different effect than calling $cron->run() from the service.Surely there is a module somewhere that is doing things wrong, e.g. not reset a cache variable etc.
I have not figured it out yet.
But in a functional test we want to be realistic.
So we want at least to keep the option to run cron as a separate request. - ๐ฎ๐ณIndia onkararun
Arun.k โ made their first commit to this issueโs fork.
- Status changed to Needs review
7 months ago 5:13am 12 July 2024 - ๐ฎ๐ณIndia Vishal Choudhary Dharmshala
@arun.k I have Tested Your branch 3274813-deprecate-cronruntrait
The deprecated code was removed from this File
like
Filename:
Db_LogTest.php
CronRunTrait.phpRemove code:
Before:
$cron_count = $this->runCron();After:
$cron_count = \Drupal::service('cron')->run();I Think We Will need to move RTBC
Thanks For Contribution.
- Status changed to RTBC
7 months ago 9:55am 12 July 2024 - Status changed to Needs work
7 months ago 10:51am 12 July 2024 - ๐ฌ๐งUnited Kingdom longwave UK
CronRunTrait has the cronRun method, here we have only changed the runCron method which is different. There is no open MR and so the tests haven't even run yet.