Issue #3375679: CronFrequencyValidator should not check if updates will be run via console

src/Validator/CronFrequencyValidator.php

@@ -82,9 +82,11 @@ final class CronFrequencyValidator implements EventSubscriberInterface {
     if (!$event->stage instanceof CronUpdateStage) {
       return;
     }
-    // If automatic updates are disabled during cron, there's nothing we need
-    // to validate.
-    if ($event->stage->getMode() === CronUpdateStage::DISABLED) {
+    // If automatic updates are disabled during cron or updates will be run via
+    // the console command, there's nothing we need to validate.
+    $method = $this->configFactory->get('automatic_updates.settings')
+      ->get('unattended.method');
+    if ($event->stage->getMode() === CronUpdateStage::DISABLED || $method !== 'web') {
       return;
     }
     // If cron is running right now, cron is clearly being run recently enough!

tests/src/Kernel/StatusCheck/CronFrequencyValidatorTest.php

@@ -35,18 +35,33 @@ class CronFrequencyValidatorTest extends AutomaticUpdatesKernelTestBase {
   }
 
   /**
-   * Tests that nothing is validated if updates are disabled during cron.
+   * Tests that nothing is validated if not needed.
    */
-  public function testNoValidationIfCronDisabled(): void {
-    $this->config('automatic_updates.settings')
-      ->set('unattended.level', CronUpdateStage::DISABLED)
-      ->save();
+  public function testNoValidation(): void {
     $state = $this->container->get('state');
     $state->delete('system.cron_last');
     $state->delete('install_time');
+
+    // If the method is 'web' but cron updates are disabled no validation is
+    // needed.
+    $this->config('automatic_updates.settings')
+      ->set('unattended.level', CronUpdateStage::DISABLED)
+      ->set('unattended.method', 'web')
+      ->save();
     $this->assertCheckerResultsFromManager([], TRUE);
+
+    // If cron updates are enabled but the method is 'console' no validation is
+    // needed.
+    $this->config('automatic_updates.settings')
+      ->set('unattended.level', CronUpdateStage::ALL)
+      ->set('unattended.method', 'console')
+      ->save();
+    $this->assertCheckerResultsFromManager([], TRUE);
+
+    // If cron updates are enabled and the method is 'web' validation is needed.
     $this->config('automatic_updates.settings')
       ->set('unattended.level', CronUpdateStage::ALL)
+      ->set('unattended.method', 'web')
       ->save();
     $error = ValidationResult::createError([
       t('Cron has not run recently. For more information, see the online handbook entry for <a href="https://www.drupal.org/cron">configuring cron jobs</a> to run at least every 3 hours.'),