automatic_updates_extensions/tests/src/Build/ModuleUpdateTest.php

@@ -14,7 +14,7 @@ use Symfony\Component\Filesystem\Filesystem as SymfonyFilesystem;
  * @group automatic_updates_extensions
  * @internal
  */
-class ModuleUpdateTest extends UpdateTestBase {
+final class ModuleUpdateTest extends UpdateTestBase {
 
   use FormTestTrait;
 
@@ -24,7 +24,7 @@ class ModuleUpdateTest extends UpdateTestBase {
   protected function createTestProject(string $template): void {
     parent::createTestProject($template);
     $this->setReleaseMetadata([
-      'drupal' => __DIR__ . '/../../../../package_manager/tests/fixtures/release-history/drupal.9.8.1-security.xml',
+      'drupal' => __DIR__ . '/../../../../package_manager/tests/fixtures/release-history/drupal.9.8.2.xml',
       'alpha'  => __DIR__ . '/../../fixtures/release-history/alpha.1.1.0.xml',
       'new_module' => __DIR__ . '/../../fixtures/release-history/new_module.1.1.0.xml',
     ]);
@@ -42,7 +42,7 @@ END;
     $this->writeSettings($code);
     $alpha_repo_path = $this->copyFixtureToTempDirectory(__DIR__ . '/../../../../package_manager/tests/fixtures/build_test_projects/alpha/1.0.0');
     $this->addRepository('alpha', $alpha_repo_path);
-    $this->runComposer('COMPOSER_MIRROR_PATH_REPOS=1 composer require drupal/alpha --update-with-all-dependencies', 'project');
+    $this->runComposer('composer require drupal/alpha --update-with-all-dependencies', 'project');
     $this->assertModuleVersion('alpha', '1.0.0');
     $fs = new SymfonyFilesystem();
     $fs->mirror($this->copyFixtureToTempDirectory(__DIR__ . '/../../fixtures/new_module'), $this->getWorkspaceDirectory() . '/project/web/modules');
@@ -54,6 +54,11 @@ END;
 
     // Change the module's upstream version.
     static::copyFixtureFilesTo(__DIR__ . '/../../../../package_manager/tests/fixtures/build_test_projects/alpha/1.1.0', $alpha_repo_path);
+
+    // Ensure that none of the above changes have caused any status check or
+    // other errors on the status report.
+    $this->checkForUpdates();
+    $this->assertStatusReportChecksSuccessful();
   }
 
   /**
@@ -76,24 +81,24 @@ END;
     $this->assertStringContainsString('The project new_module is not a Drupal project known to Composer and cannot be updated.', $page_text);
     $this->assertStringContainsString('new_module', $page_text);
     // Use the API endpoint to create a stage and update the 'alpha' module to
-    // 1.1.0. We ask the API to return the contents of the module's
-    // composer.json file, so we can assert that they were updated to the
-    // version we expect.
-    // @see \Drupal\automatic_updates_extensions_test_api\ApiController::run()
-    $file_contents = $this->getPackageManagerTestApiResponse(
+    // 1.1.0.
+    $this->makePackageManagerTestApiRequest(
       '/automatic-updates-extensions-test-api',
       [
         'projects' => [
           'alpha' => '1.1.0',
         ],
-        'files_to_return' => [
-          'web/modules/contrib/alpha/composer.json',
-        ],
       ]
     );
 
-    $module_composer_json = json_decode($file_contents['web/modules/contrib/alpha/composer.json']);
-    $this->assertSame('1.1.0', $module_composer_json?->version);
+    $updated_composer_json = $this->getWebRoot() . 'modules/contrib/alpha/composer.json';
+    // Assert the module was updated.
+    $this->assertFileEquals(
+      __DIR__ . '/../../../../package_manager/tests/fixtures/build_test_projects/alpha/1.1.0/composer.json',
+      $updated_composer_json,
+    );
+    $this->assertRequestedChangesWereLogged(['Update drupal/alpha from 1.0.0 to 1.1.0']);
+    $this->assertAppliedChangesWereLogged(['Updated drupal/alpha from 1.0.0 to 1.1.0']);
   }
 
   /**
@@ -127,10 +132,24 @@ END;
     $page->pressButton('Update');
     $this->waitForBatchJob();
     $assert_session->pageTextContains('Ready to update');
+    $page->checkField('backup');
     $page->pressButton('Continue');
     $this->waitForBatchJob();
     $assert_session->pageTextContains('Update complete!');
     $this->assertModuleVersion('alpha', '1.1.0');
+    $this->assertRequestedChangesWereLogged(['Update drupal/alpha from 1.0.0 to 1.1.0']);
+    $this->assertAppliedChangesWereLogged(['Updated drupal/alpha from 1.0.0 to 1.1.0']);
+  }
+
+  /**
+   * {@inheritdoc}
+   */
+  public function copyCodebase(\Iterator $iterator = NULL, $working_dir = NULL): void {
+    parent::copyCodebase($iterator, $working_dir);
+
+    // Ensure that we will install Drupal 9.8.0 (a fake version that should
+    // never exist in real life) initially.
+    $this->setUpstreamCoreVersion('9.8.0');
   }
 
   /**

automatic_updates_extensions/tests/src/Functional/ComposerStagerOperationFailureTest.php

+<?php
+
+declare(strict_types=1);
+
+namespace Drupal\Tests\automatic_updates_extensions\Functional;
+
+use Drupal\package_manager_bypass\LoggingBeginner;
+use Drupal\package_manager_bypass\LoggingCommitter;
+use Drupal\package_manager_bypass\NoOpStager;
+use PhpTuf\ComposerStager\API\Exception\InvalidArgumentException;
+use PhpTuf\ComposerStager\API\Exception\LogicException;
+
+/**
+ * @covers \Drupal\automatic_updates_extensions\Form\UpdaterForm
+ * @group automatic_updates_extensions
+ * @internal
+ */
+class ComposerStagerOperationFailureTest extends UpdaterFormTestBase {
+
+  /**
+   * Tests Composer operation failure is handled properly.
+   *
+   * @param string $exception_class
+   *   The exception class.
+   * @param string $service_class
+   *   The Composer Stager service which should throw an exception.
+   *
+   * @dataProvider providerComposerOperationFailure
+   */
+  public function testComposerOperationFailure(string $exception_class, string $service_class): void {
+    $page = $this->getSession()->getPage();
+    $assert_session = $this->assertSession();
+    $this->setReleaseMetadata(__DIR__ . '/../../../../package_manager/tests/fixtures/release-history/drupal.9.8.2.xml');
+    $this->setReleaseMetadata(__DIR__ . '/../../fixtures/release-history/semver_test.1.1.xml');
+    $this->setProjectInstalledVersion(['semver_test' => '8.1.0']);
+    $this->checkForUpdates();
+    // Navigate to the update form.
+    $this->drupalGet('/admin/reports/updates');
+    $this->clickLink('Update Extensions');
+    $this->assertTableShowsUpdates(
+      'Semver Test',
+      '8.1.0',
+      '8.1.1',
+    );
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test_package_name', '8.1.1');
+    $this->assertUpdatesCount(1);
+    $page->checkField('projects[semver_test]');
+
+    // Make the specified Composer Stager operation class throw an exception.
+    $message = $this->createComposeStagerMessage("Failure from inside $service_class");
+    $exception = new $exception_class($message);
+    call_user_func([$service_class, 'setException'], $exception);
+
+    // Start the update.
+    $page->pressButton('Update');
+    $this->checkForMetaRefresh();
+    // We can't continue the update after an error in the committer.
+    if ($service_class === LoggingCommitter::class) {
+      $this->acceptWarningAndUpdate();
+      $this->clickLink('the error page');
+      $assert_session->statusCodeEquals(200);
+      $assert_session->statusMessageContains('Automatic updates failed to apply, and the site is in an indeterminate state. Consider restoring the code and database from a backup.');
+      return;
+    }
+    $this->clickLink('the error page');
+    $assert_session->statusMessageContains($exception->getMessage());
+
+    // Make the same Composer Stager operation class NOT throw an exception.
+    call_user_func([$service_class, 'setException'], NULL);
+
+    // Stage should be automatically deleted when an error occurs.
+    $assert_session->buttonNotExists('Delete existing update');
+    // This ensures that we can still update after the exception no longer
+    // exists.
+    $page->checkField('projects[semver_test]');
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test_package_name', '8.1.1');
+    $page->pressButton('Update');
+    $this->checkForMetaRefresh();
+    $this->acceptWarningAndUpdate();
+    $assert_session->statusMessageContains('Update complete!');
+  }
+
+  /**
+   * Data provider for testComposerOperationFailure().
+   *
+   * @return string[][]
+   *   The test cases.
+   */
+  public function providerComposerOperationFailure(): array {
+    return [
+      'LogicException from Beginner' => [LogicException::class, LoggingBeginner::class],
+      'LogicException from Stager' => [LogicException::class, NoOpStager::class],
+      'InvalidArgumentException from Stager' => [InvalidArgumentException::class, NoOpStager::class],
+      'LogicException from Committer' => [LogicException::class, LoggingCommitter::class],
+    ];
+  }
+
+}

automatic_updates_extensions/tests/src/Functional/DisplayUpdatesTest.php

@@ -9,7 +9,7 @@ namespace Drupal\Tests\automatic_updates_extensions\Functional;
  * @group automatic_updates_extensions
  * @internal
  */
-class DisplayUpdatesTest extends UpdaterFormTestBase {
+final class DisplayUpdatesTest extends UpdaterFormTestBase {
 
   /**
    * Data provider for testDisplayUpdates().
@@ -67,7 +67,7 @@ class DisplayUpdatesTest extends UpdaterFormTestBase {
     $page->checkField('projects[semver_test]');
     $this->getStageFixtureManipulator()
       ->setVersion('drupal/aaa_update_test', '2.1.0')
-      ->setVersion('drupal/semver_test', '8.1.1');
+      ->setVersion('drupal/semver_test_package_name', '8.1.1');
     $page->pressButton('Update');
     $this->checkForMetaRefresh();
     $this->assertUpdateStagedTimes(1);

automatic_updates_extensions/tests/src/Functional/PreUpdateTest.php

@@ -11,7 +11,7 @@ use Drupal\fixture_manipulator\ActiveFixtureManipulator;
  * @group automatic_updates_extensions
  * @internal
  */
-class PreUpdateTest extends UpdaterFormTestBase {
+final class PreUpdateTest extends UpdaterFormTestBase {
 
   /**
    * Tests the form when modules requiring an update not installed via composer.
@@ -48,7 +48,7 @@ class PreUpdateTest extends UpdaterFormTestBase {
 
     // Both of the modules not installed through composer.
     (new ActiveFixtureManipulator())
-      ->removePackage('drupal/semver_test')
+      ->removePackage('drupal/semver_test_package_name')
       ->commitChanges();
     $this->getSession()->reload();
     $assert->pageTextContains('Updates were found, but they must be performed manually. See the list of available updates for more information.');

automatic_updates_extensions/tests/src/Functional/StatusCheckerRunAfterUpdateTest.php

@@ -13,7 +13,7 @@ use Drupal\package_manager\ValidationResult;
  * @group automatic_updates_extensions
  * @internal
  */
-class StatusCheckerRunAfterUpdateTest extends UpdaterFormTestBase {
+final class StatusCheckerRunAfterUpdateTest extends UpdaterFormTestBase {
 
   /**
    * Data provider for testStatusCheckerRunAfterUpdate().
@@ -48,6 +48,7 @@ class StatusCheckerRunAfterUpdateTest extends UpdaterFormTestBase {
     $assert_session->pageTextNotContains(static::$warningsExplanation);
 
     $this->assertTableShowsUpdates('Semver Test', '8.1.0', '8.1.1');
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test_package_name', '8.1.1');
     $this->assertUpdatesCount(1);
     $page->checkField('projects[semver_test]');
     $page->pressButton('Update');
@@ -62,10 +63,9 @@ class StatusCheckerRunAfterUpdateTest extends UpdaterFormTestBase {
       // We must do this after the update has started, because the pending
       // updates validator will prevent an update from starting.
       $this->container->get('state')->set('automatic_updates_test.new_update', TRUE);
-      $page->pressButton('Continue');
-      $this->checkForMetaRefresh();
+      $this->acceptWarningAndUpdate();
       $assert_session->pageTextContainsOnce('An error has occurred.');
-      $assert_session->pageTextContainsOnce('Please continue to the error page');
+      $assert_session->pageTextContainsOnce('Continue to the error page');
       $page->clickLink('the error page');
       $assert_session->pageTextContains('Some modules have database schema updates to install. You should run the database update script immediately.');
       $assert_session->linkExists('database update script');
@@ -79,23 +79,18 @@ class StatusCheckerRunAfterUpdateTest extends UpdaterFormTestBase {
       $page->clickLink('Apply pending updates');
       $this->checkForMetaRefresh();
       $assert_session->pageTextContains('Updates were attempted.');
-      // PendingUpdatesValidator prevented the update to complete, so the status
-      // checks weren't run.
-      $this->drupalGet('/admin');
-      $assert_session->pageTextContains('Your site has not recently run an update readiness check. Rerun readiness checks now.');
     }
     else {
-      $page->pressButton('Continue');
-      $this->checkForMetaRefresh();
+      $this->acceptWarningAndUpdate();
       $assert_session->addressEquals('/admin/reports/updates');
       $assert_session->pageTextContainsOnce('Update complete!');
-      // Status checks should display errors on admin page.
-      $this->drupalGet('/admin');
-      // Confirm that the status checks were run and the new error is displayed.
-      $assert_session->statusMessageContains('Error before continue.', 'error');
-      $assert_session->statusMessageContains(static::$errorsExplanation, 'error');
-      $assert_session->pageTextNotContains('Your site has not recently run an update readiness check. Rerun readiness checks now.');
     }
+    // Status checks should display errors on admin page.
+    $this->drupalGet('/admin');
+    // Confirm that the status checks were run and the new error is displayed.
+    $assert_session->statusMessageContains('Error before continue.', 'error');
+    $assert_session->statusMessageContains('Your site does not pass some readiness checks for automatic updates. It cannot be automatically updated until further action is performed.', 'error');
+    $assert_session->pageTextNotContains('Your site has not recently run an update readiness check. Rerun readiness checks now.');
   }
 
 }

automatic_updates_extensions/tests/src/Functional/SuccessfulUpdateTest.php

@@ -5,6 +5,7 @@ declare(strict_types=1);
 namespace Drupal\Tests\automatic_updates_extensions\Functional;
 
 use Drupal\package_manager\Event\PreApplyEvent;
+use Drupal\package_manager\LegacyVersionUtility;
 use Drupal\package_manager_test_validation\StagedDatabaseUpdateValidator;
 
 /**
@@ -12,7 +13,7 @@ use Drupal\package_manager_test_validation\StagedDatabaseUpdateValidator;
  * @group automatic_updates_extensions
  * @internal
  */
-class SuccessfulUpdateTest extends UpdaterFormTestBase {
+final class SuccessfulUpdateTest extends UpdaterFormTestBase {
 
   /**
    * Data provider for testSuccessfulUpdate().
@@ -23,13 +24,13 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
   public function providerSuccessfulUpdate(): array {
     return [
       'maintenance mode on, semver module' => [
-        TRUE, 'semver_test', 'Semver Test', '8.1.0', '8.1.1',
+        TRUE, 'semver_test', 'drupal/semver_test_package_name', 'Semver Test', '8.1.0', '8.1.1',
       ],
       'maintenance mode off, legacy module' => [
-        FALSE, 'aaa_update_test', 'AAA Update test', '8.x-2.0', '8.x-2.1',
+        FALSE, 'aaa_update_test', 'drupal/aaa_update_test', 'AAA Update test', '8.x-2.0', '8.x-2.1',
       ],
       'maintenance mode off, legacy theme' => [
-        FALSE, 'automatic_updates_extensions_test_theme', 'Automatic Updates Extensions Test Theme', '8.x-2.0', '8.x-2.1',
+        FALSE, 'automatic_updates_extensions_test_theme', 'drupal/automatic_updates_extensions_test_theme', 'Automatic Updates Extensions Test Theme', '8.x-2.0', '8.x-2.1',
       ],
     ];
   }
@@ -41,6 +42,8 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
    *   Whether maintenance should be on at the beginning of the update.
    * @param string $project_name
    *   The project name.
+   * @param string $package_name
+   *   The package name.
    * @param string $project_title
    *   The project title.
    * @param string $installed_version
@@ -50,7 +53,7 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
    *
    * @dataProvider providerSuccessfulUpdate
    */
-  public function testSuccessfulUpdate(bool $maintenance_mode_on, string $project_name, string $project_title, string $installed_version, string $target_version): void {
+  public function testSuccessfulUpdate(bool $maintenance_mode_on, string $project_name, string $package_name, string $project_title, string $installed_version, string $target_version): void {
     $this->container->get('theme_installer')->install(['automatic_updates_theme_with_updates']);
     // By default, the Update module only checks for updates of installed
     // modules and themes. The two modules we're testing here (semver_test and
@@ -60,6 +63,7 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
     $path_to_fixtures_folder = $project_name === 'aaa_update_test' ? '/../../../../package_manager/tests' : '/../..';
     $this->setReleaseMetadata(__DIR__ . $path_to_fixtures_folder . '/fixtures/release-history/' . $project_name . '.1.1.xml');
     $this->setProjectInstalledVersion([$project_name => $installed_version]);
+    $this->getStageFixtureManipulator()->setVersion($package_name, LegacyVersionUtility::convertToSemanticVersion($target_version));
     $this->checkForUpdates();
     $state = $this->container->get('state');
     $state->set('system.maintenance_mode', $maintenance_mode_on);
@@ -82,7 +86,7 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
 
     // Submit without selecting a project.
     $page->pressButton('Update');
-    $assert_session->pageTextContains('Please select one or more projects.');
+    $assert_session->pageTextContains('Select one or more projects.');
 
     // Submit with a project selected.
     $page->checkField('projects[' . $project_name . ']');
@@ -96,10 +100,9 @@ class SuccessfulUpdateTest extends UpdaterFormTestBase {
     // Ensure that a list of pending database updates is visible, along with a
     // short explanation, in the warning messages.
     $warning_messages = $assert_session->elementExists('xpath', '//div[@data-drupal-messages]//div[@aria-label="Warning message"]');
-    $this->assertStringContainsString('Possible database updates have been detected in the following extensions.<ul><li>System</li><li>Automatic Updates Theme With Updates</li></ul>', $warning_messages->getHtml());
+    $this->assertStringContainsString('Database updates have been detected in the following extensions.<ul><li>System</li><li>Automatic Updates Theme With Updates</li></ul>', $warning_messages->getHtml());
 
-    $page->pressButton('Continue');
-    $this->checkForMetaRefresh();
+    $this->acceptWarningAndUpdate();
     $assert_session->addressEquals('/admin/reports/updates');
     // Confirm that the site was in maintenance before the update was applied.
     // @see \Drupal\package_manager_test_validation\EventSubscriber\TestSubscriber::handleEvent()

automatic_updates_extensions/tests/src/Functional/UnsuccessfulUpdateTest.php

@@ -14,7 +14,7 @@ use Drupal\package_manager_test_validation\EventSubscriber\TestSubscriber;
  * @group automatic_updates_extensions
  * @internal
  */
-class UnsuccessfulUpdateTest extends UpdaterFormTestBase {
+final class UnsuccessfulUpdateTest extends UpdaterFormTestBase {
 
   /**
    * Test the form for warning messages.
@@ -31,12 +31,14 @@ class UnsuccessfulUpdateTest extends UpdaterFormTestBase {
     $this->drupalGet('/admin/reports/updates');
     $this->clickLink('Update Extensions');
     $this->assertTableShowsUpdates('Semver Test', '8.1.0', '8.1.1');
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test_package_name', '8.1.1');
     $this->assertUpdatesCount(1);
     $this->checkForMetaRefresh();
     $assert->pageTextNotContains(static::$errorsExplanation);
     $assert->elementExists('css', '#edit-projects-semver-test')->check();
     $assert->checkboxChecked('edit-projects-semver-test');
-    $assert->pageTextContains(static::$warningsExplanation);
+    // No explanation is given for warnings.
+    $assert->pageTextNotContains(static::$warningsExplanation);
     $assert->buttonExists('Update');
 
     // Add warnings from StatusCheckEvent.
@@ -51,9 +53,9 @@ class UnsuccessfulUpdateTest extends UpdaterFormTestBase {
     $this->getSession()->getPage()->pressButton('Update');
     $this->checkForMetaRefresh();
     $assert->buttonExists('Continue');
-    $assert->pageTextContains($summary_status_check_event);
+    $assert->pageTextContains((string) $summary_status_check_event);
     foreach ($messages_status_check_event as $message) {
-      $assert->pageTextContains($message);
+      $assert->pageTextContains((string) $message);
     }
   }
 

automatic_updates_extensions/tests/src/Functional/UpdateErrorTest.php

@@ -15,7 +15,7 @@ use Drupal\package_manager_test_validation\EventSubscriber\TestSubscriber;
  * @group automatic_updates_extensions
  * @internal
  */
-class UpdateErrorTest extends UpdaterFormTestBase {
+final class UpdateErrorTest extends UpdaterFormTestBase {
 
   /**
    * Tests that an exception is thrown if a previous apply failed.
@@ -32,6 +32,7 @@ class UpdateErrorTest extends UpdaterFormTestBase {
     $assert_session->pageTextNotContains(static::$warningsExplanation);
 
     $this->assertTableShowsUpdates('Semver Test', '8.1.0', '8.1.1');
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test_package_name', '8.1.1');
     $this->assertUpdatesCount(1);
     $page->checkField('projects[semver_test]');
     $page->pressButton('Update');
@@ -39,8 +40,7 @@ class UpdateErrorTest extends UpdaterFormTestBase {
     $this->assertUpdateStagedTimes(1);
     $assert_session->pageTextNotContains('The following dependencies will also be updated:');
     LoggingCommitter::setException(new \Exception('failed at committer'));
-    $page->pressButton('Continue');
-    $this->checkForMetaRefresh();
+    $this->acceptWarningAndUpdate();
     $failure_message = 'Automatic updates failed to apply, and the site is in an indeterminate state. Consider restoring the code and database from a backup.';
     $assert_session->pageTextContainsOnce('An error has occurred.');
     $assert_session->pageTextContains($failure_message);
@@ -78,7 +78,7 @@ class UpdateErrorTest extends UpdaterFormTestBase {
     $error = ValidationResult::createError([$message]);
     TestSubscriber1::setTestResult([$error], StatusCheckEvent::class);
     $this->getSession()->reload();
-    $assert->pageTextContains($message);
+    $assert->pageTextContains((string) $message);
     $assert->pageTextContains(static::$errorsExplanation);
     $assert->pageTextNotContains(static::$warningsExplanation);
     $assert->buttonNotExists('Update');
@@ -111,9 +111,9 @@ class UpdateErrorTest extends UpdaterFormTestBase {
     $this->checkForMetaRefresh();
     $assert->pageTextContains(static::$errorsExplanation);
     $assert->pageTextNotContains(static::$warningsExplanation);
-    $assert->pageTextContains($summary);
+    $assert->pageTextContains((string) $summary);
     foreach ($messages as $message) {
-      $assert->pageTextContains($message);
+      $assert->pageTextContains((string) $message);
     }
     $assert->buttonNotExists('Continue');
   }

automatic_updates_extensions/tests/src/Functional/UpdaterFormTestBase.php

@@ -13,11 +13,18 @@ use Drupal\Tests\automatic_updates\Functional\UpdaterFormTestBase as UpdaterForm
  * Base class for functional tests of updater form.
  *
  * @internal
+ *   This class is an internal part of the module's testing infrastructure and
+ *   should not be used by external code.
  */
 abstract class UpdaterFormTestBase extends UpdaterFormFunctionalTestBase {
 
   use FormTestTrait;
 
+  /**
+   * {@inheritdoc}
+   */
+  protected static $errorsExplanation = 'Your site cannot be automatically updated until further action is performed.';
+
   /**
    * The path of the test project's active directory.
    *
@@ -43,11 +50,19 @@ abstract class UpdaterFormTestBase extends UpdaterFormFunctionalTestBase {
 
     $this->activeDir = $this->container->get(PathLocator::class)->getProjectRoot();
     (new ActiveFixtureManipulator())
-      ->addPackage([
-        'name' => 'drupal/semver_test',
-        'version' => '8.1.0',
-        'type' => 'drupal-module',
-      ])
+      // Add a package where the Composer package name,
+      // drupal/semver_test_package_name, does not match the project name,
+      // semver_test.
+      ->addPackage(
+        [
+          'name' => 'drupal/semver_test_package_name',
+          'version' => '8.1.0',
+          'type' => 'drupal-module',
+        ],
+        extra_files: [
+          'semver_test.info.yml' => '{name: "Semver Test", project: "semver_test", type: "module"}',
+        ]
+      )
       ->addPackage([
         'name' => 'drupal/aaa_update_test',
         'version' => '2.0.0',
@@ -124,4 +139,16 @@ abstract class UpdaterFormTestBase extends UpdaterFormFunctionalTestBase {
     $this->checkForMetaRefresh();
   }
 
+  /**
+   * Continue the update after checking the backup warning checkbox.
+   */
+  protected function acceptWarningAndUpdate(): void {
+    $page = $this->getSession()->getPage();
+    $page->pressButton('Continue');
+    $this->assertSession()->statusMessageContains('Warning: Updating contributed modules or themes may leave your site inoperable or looking wrong. field is required.', 'error');
+    $page->checkField('backup');
+    $page->pressButton('Continue');
+    $this->checkForMetaRefresh();
+  }
+
 }

automatic_updates_extensions/tests/src/Kernel/AutomaticUpdatesExtensionsKernelTestBase.php

@@ -4,15 +4,19 @@ declare(strict_types = 1);
 
 namespace Drupal\Tests\automatic_updates_extensions\Kernel;
 
+use Drupal\automatic_updates_extensions\ExtensionUpdateStage;
 use Drupal\fixture_manipulator\ActiveFixtureManipulator;
 use Drupal\package_manager\Event\PreOperationStageEvent;
 use Drupal\package_manager\Exception\StageEventException;
+use Drupal\package_manager\Validator\ComposerValidator;
 use Drupal\Tests\automatic_updates\Kernel\AutomaticUpdatesKernelTestBase;
 
 /**
  * Base class for kernel tests of the Automatic Updates Extensions module.
  *
  * @internal
+ *   This class is an internal part of the module's testing infrastructure and
+ *   should not be used by external code.
  */
 abstract class AutomaticUpdatesExtensionsKernelTestBase extends AutomaticUpdatesKernelTestBase {
 
@@ -33,7 +37,7 @@ abstract class AutomaticUpdatesExtensionsKernelTestBase extends AutomaticUpdates
     // server. This should be okay in most situations because, apart from the
     // validator, only Composer Stager needs run Composer, and
     // package_manager_bypass is disabling those operations.
-    $this->disableValidators[] = 'package_manager.validator.composer';
+    $this->disableValidators[] = ComposerValidator::class;
     parent::setUp();
     // Install additional packages that will be needed in tests.
     (new ActiveFixtureManipulator())
@@ -54,7 +58,7 @@ abstract class AutomaticUpdatesExtensionsKernelTestBase extends AutomaticUpdates
       ], TRUE)
       ->addPackage([
         "name" => "drupal/semver_test",
-        "version" => "1.0.0",
+        "version" => "8.1.0",
         "type" => "drupal-module",
       ])
       ->addPackage([
@@ -63,7 +67,7 @@ abstract class AutomaticUpdatesExtensionsKernelTestBase extends AutomaticUpdates
         "type" => "drupal-module",
       ])
       ->addPackage([
-        "name" => "drupal/aaa_package_manager_test",
+        "name" => "drupal/package_manager_test_update",
         "version" => "1.0.0",
         "type" => "drupal-module",
       ])->commitChanges();
@@ -81,7 +85,7 @@ abstract class AutomaticUpdatesExtensionsKernelTestBase extends AutomaticUpdates
    *   be passed if $expected_results is not empty.
    */
   protected function assertUpdateResults(array $project_versions, array $expected_results, string $event_class = NULL): void {
-    $stage = $this->container->get('automatic_updates_extensions.update_stage');
+    $stage = $this->container->get(ExtensionUpdateStage::class);
 
     try {
       $stage->begin($project_versions);

automatic_updates_extensions/tests/src/Kernel/ExtensionUpdateStageTest.php

@@ -5,14 +5,16 @@ declare(strict_types = 1);
 namespace Drupal\Tests\automatic_updates_extensions\Kernel;
 
 use Drupal\automatic_updates_extensions\ExtensionUpdateStage;
+use Drupal\automatic_updates_extensions\Validator\UpdateReleaseValidator;
 use Drupal\Tests\user\Traits\UserCreationTrait;
+use PhpTuf\ComposerStager\API\Core\StagerInterface;
 
 /**
  * @coversDefaultClass \Drupal\automatic_updates_extensions\ExtensionUpdateStage
  * @group automatic_updates_extensions
  * @internal
  */
-class ExtensionUpdateStageTest extends AutomaticUpdatesExtensionsKernelTestBase {
+final class ExtensionUpdateStageTest extends AutomaticUpdatesExtensionsKernelTestBase {
 
   use UserCreationTrait;
 
@@ -33,7 +35,7 @@ class ExtensionUpdateStageTest extends AutomaticUpdatesExtensionsKernelTestBase
     // This test doesn't need to validate that the test projects used are in the
     // codebase. Therefore, we need to disable the following validators that
     // require real Drupal projects.
-    $this->disableValidators[] = 'automatic_updates_extensions.validator.target_release';
+    $this->disableValidators[] = UpdateReleaseValidator::class;
     parent::setUp();
     $this->installEntitySchema('user');
 
@@ -98,13 +100,14 @@ class ExtensionUpdateStageTest extends AutomaticUpdatesExtensionsKernelTestBase
       [
         'update',
         '--with-all-dependencies',
+        '--optimize-autoloader',
         'drupal/my_module:9.8.1',
         'drupal/my_dev_module:1.2.0-alpha1@alpha',
       ],
     ];
     $stage->stage();
 
-    $actual_arguments = $this->container->get('package_manager.stager')
+    $actual_arguments = $this->container->get(StagerInterface::class)
       ->getInvocationArguments();
 
     $this->assertCount(count($expected_arguments), $actual_arguments);

automatic_updates_extensions/tests/src/Kernel/Validator/ForbidCoreChangesValidatorTest.php

+<?php
+
+namespace Drupal\Tests\automatic_updates_extensions\Kernel\Validator;
+
+use Drupal\automatic_updates_extensions\ExtensionUpdateStage;
+use Drupal\package_manager\Exception\StageEventException;
+use Drupal\package_manager\ValidationResult;
+use Drupal\Tests\automatic_updates_extensions\Kernel\AutomaticUpdatesExtensionsKernelTestBase;
+
+/**
+ * @coversDefaultClass \Drupal\automatic_updates_extensions\Validator\ForbidCoreChangesValidator
+ * @group automatic_updates_extensions
+ * @internal
+ */
+class ForbidCoreChangesValidatorTest extends AutomaticUpdatesExtensionsKernelTestBase {
+
+  /**
+   * Tests error messages if requested updates were not staged.
+   *
+   * @param array $staged_versions
+   *   An array of the staged versions where the keys are the package names and
+   *   the values are the package versions or NULL if the package should be
+   *   removed in the stage.
+   * @param string[][] $new_packages
+   *   An array of the new packages to add to the stage.
+   * @param \Drupal\package_manager\ValidationResult[] $expected_results
+   *   The expected validation results.
+   *
+   * @dataProvider providerTestErrorMessage
+   */
+  public function testErrorMessages(array $staged_versions, array $new_packages, array $expected_results): void {
+    $this->setReleaseMetadata([
+      'semver_test' => __DIR__ . '/../../../fixtures/release-history/semver_test.1.1.xml',
+      'drupal' => __DIR__ . '/../../../../../package_manager/tests/fixtures/release-history/drupal.9.8.2.xml',
+    ]);
+    $this->getStageFixtureManipulator()->addPackage([
+      'name' => 'drupal/non-core',
+      'version' => '1.0.0',
+      'type' => 'package',
+    ]);
+
+    foreach ($staged_versions as $package => $version) {
+      if ($version === NULL) {
+        $this->getStageFixtureManipulator()->removePackage($package);
+        continue;
+      }
+      $this->getStageFixtureManipulator()->setVersion($package, $version);
+    }
+    foreach ($new_packages as $package) {
+      $this->getStageFixtureManipulator()->addPackage($package);
+    }
+    $this->getStageFixtureManipulator()->setVersion('drupal/semver_test', '8.1.1');
+
+    $stage = $this->container->get(ExtensionUpdateStage::class);
+    $stage->begin([
+      'semver_test' => '8.1.1',
+    ]);
+    $stage->stage();
+    $this->assertStatusCheckResults($expected_results, $stage);
+    try {
+      $stage->apply();
+      $this->fail('Expecting an exception.');
+    }
+    catch (StageEventException $exception) {
+      $this->assertExpectedResultsFromException($expected_results, $exception);
+    }
+  }
+
+  /**
+   * Data provider for testErrorMessage().
+   *
+   * @return mixed[]
+   *   The test cases.
+   */
+  public function providerTestErrorMessage(): array {
+    $summary = t('Updating Drupal Core while updating extensions is currently not supported. Use <a href="/admin/reports/updates/update">this form</a> to update Drupal core. The following changes were made to the Drupal core packages:');
+    return [
+      'drupal/core updated, non-core updated' => [
+        [
+          'drupal/core' => '9.8.1',
+          'drupal/non-core' => '1.0.1',
+        ],
+        [],
+        [ValidationResult::createError([t("'drupal/core' version changed from 9.8.0 to 9.8.1.")], $summary)],
+      ],
+      'drupal/core-recommended and drupal/core updated, non-core package installed' => [
+        [
+          'drupal/core-recommended' => '9.8.1',
+          'drupal/core' => '9.8.1',
+        ],
+        [
+          [
+            'name' => 'other-org/other-package',
+            'type' => 'package',
+          ],
+        ],
+        [
+          ValidationResult::createError(
+            [
+              t("'drupal/core-recommended' version changed from 9.8.0 to 9.8.1."),
+              t("'drupal/core' version changed from 9.8.0 to 9.8.1."),
+            ],
+            $summary
+          ),
+        ],
+      ],
+      'drupal/core-recommended removed, drupal/core updated, drupal/core-composer-scaffold installed, non-core package removed' => [
+        [
+          'drupal/core-recommended' => NULL,
+          'drupal/core' => '9.8.1',
+          'drupal/non-core' => NULL,
+        ],
+        [
+          [
+            'name' => 'drupal/core-composer-scaffold',
+            'type' => 'package',
+          ],
+        ],
+        [
+          ValidationResult::createError(
+            [
+              t("'drupal/core-composer-scaffold' installed."),
+              t("'drupal/core-recommended' removed."),
+              t("'drupal/core' version changed from 9.8.0 to 9.8.1."),
+            ],
+            $summary
+          ),
+        ],
+      ],
+    ];
+  }
+
+}

automatic_updates_extensions/tests/src/Kernel/Validator/RequestedUpdateValidatorTest.php

+<?php
+
+declare(strict_types=1);
+
+namespace Drupal\Tests\automatic_updates_extensions\Kernel\Validator;
+
+use Drupal\automatic_updates_extensions\ExtensionUpdateStage;
+use Drupal\fixture_manipulator\ActiveFixtureManipulator;
+use Drupal\package_manager\Exception\StageEventException;
+use Drupal\package_manager\ValidationResult;
+use Drupal\Tests\automatic_updates_extensions\Kernel\AutomaticUpdatesExtensionsKernelTestBase;
+
+/**
+ * @coversDefaultClass \Drupal\automatic_updates_extensions\Validator\RequestedUpdateValidator
+ * @group automatic_updates_extensions
+ * @internal
+ */
+class RequestedUpdateValidatorTest extends AutomaticUpdatesExtensionsKernelTestBase {
+
+  /**
+   * Tests error messages if requested updates were not staged.
+   *
+   * @param array $staged_versions
+   *   An array of the staged versions where the keys are the package names and
+   *   the values are the package versions.
+   * @param array $expected_results
+   *   The expected validation results.
+   *
+   * @dataProvider providerTestErrorMessage
+   */
+  public function testErrorMessage(array $staged_versions, array $expected_results): void {
+    if ($staged_versions) {
+      // If we are going to stage updates to Drupal packages also update a
+      // non-Drupal. The validator should ignore the non-Drupal packages.
+      (new ActiveFixtureManipulator())
+        ->addPackage([
+          "name" => 'vendor/non-drupal-package',
+          "version" => "1.0.0",
+          "type" => "drupal-module",
+        ])
+        ->commitChanges();
+      $this->getStageFixtureManipulator()->setVersion('vendor/non-drupal-package', '1.0.1');
+      foreach ($staged_versions as $package => $version) {
+        $this->getStageFixtureManipulator()->setVersion($package, $version);
+      }
+    }
+
+    $this->setReleaseMetadata([
+      'semver_test' => __DIR__ . '/../../../fixtures/release-history/semver_test.1.1.xml',
+      'drupal' => __DIR__ . '/../../../../../package_manager/tests/fixtures/release-history/drupal.9.8.2.xml',
+      'aaa_update_test' => __DIR__ . "/../../../../../package_manager/tests/fixtures/release-history/aaa_update_test.1.1.xml",
+    ]);
+    // Set the project version to '8.0.1' so that there 2 versions of above this
+    // that will be in the list of supported releases, 8.1.0 and 8.1.1.
+    (new ActiveFixtureManipulator())
+      ->setVersion('drupal/semver_test', '8.0.1')
+      ->commitChanges();
+    // @todo Replace with use of the trait from the Update module in https://drupal.org/i/3348234.
+    $module_info = ['version' => '8.0.1', 'project' => 'semver_test'];
+    $this->config('update_test.settings')
+      ->set("system_info.semver_test", $module_info)
+      ->save();
+
+    $stage = $this->container->get(ExtensionUpdateStage::class);
+    $stage->begin([
+      'semver_test' => '8.1.1',
+      'aaa_update_test' => '8.x-1.1',
+    ]);
+    $stage->stage();
+    $this->assertStatusCheckResults($expected_results, $stage);
+    try {
+      $stage->apply();
+      $this->fail('Expecting an exception.');
+    }
+    catch (StageEventException $exception) {
+      $this->assertExpectedResultsFromException($expected_results, $exception);
+    }
+  }
+
+  /**
+   * Data provider for testErrorMessage().
+   *
+   * @return mixed[]
+   *   The test cases.
+   */
+  public function providerTestErrorMessage() {
+    return [
+      'no updates' => [
+        [],
+        [
+          ValidationResult::createError([t('No updates detected in the staging area.')]),
+        ],
+      ],
+      '1 project not updated' => [
+        [
+          'drupal/aaa_update_test' => '1.1.0',
+        ],
+        [
+          ValidationResult::createError([t("The requested update to 'drupal/semver_test' to version '8.1.1' was not performed.")]),
+        ],
+      ],
+      'project updated to wrong version' => [
+        [
+          'drupal/semver_test' => '8.1.0',
+          'drupal/aaa_update_test' => '1.1.0',
+        ],
+        [
+          ValidationResult::createError([t("The requested update to 'drupal/semver_test' to version '8.1.1' does not match the actual staged update to '8.1.0'.")]),
+        ],
+      ],
+    ];
+  }
+
+}

automatic_updates_extensions/tests/src/Kernel/Validator/UpdateReleaseValidatorTest.php

@@ -14,7 +14,7 @@ use Drupal\Tests\automatic_updates_extensions\Kernel\AutomaticUpdatesExtensionsK
  * @group automatic_updates_extensions
  * @internal
  */
-class UpdateReleaseValidatorTest extends AutomaticUpdatesExtensionsKernelTestBase {
+final class UpdateReleaseValidatorTest extends AutomaticUpdatesExtensionsKernelTestBase {
 
   /**
    * {@inheritdoc}
@@ -79,6 +79,9 @@ class UpdateReleaseValidatorTest extends AutomaticUpdatesExtensionsKernelTestBas
       ];
     }
     else {
+      // Ensure the correct version of the package is staged because the update
+      // is expected to succeed.
+      $this->getStageFixtureManipulator()->setVersion("drupal/$project", LegacyVersionUtility::convertToSemanticVersion($target_version));
       $expected_results = [];
     }
 

automatic_updates_extensions/tests/src/Traits/FormTestTrait.php

 <?php
 
-declare(strict_types = 1);
+declare(strict_types=1);
 
 namespace Drupal\Tests\automatic_updates_extensions\Traits;
 
@@ -11,6 +11,8 @@ use Drupal\Tests\BrowserTestBase;
  * Common methods for testing the update form.
  *
  * @internal
+ *   This class is an internal part of the module's testing infrastructure and
+ *   should not be used by external code.
  */
 trait FormTestTrait {
 
@@ -32,7 +34,10 @@ trait FormTestTrait {
     $row_selector = ".update-recommended tr:nth-of-type($row)";
     $assert->elementTextContains('css', $row_selector . ' td:nth-of-type(2)', $expected_project_title);
     $assert->elementTextContains('css', $row_selector . ' td:nth-of-type(3)', $expected_installed_version);
-    $assert->elementTextContains('css', $row_selector . ' td:nth-of-type(4)', $expected_target_version);
+    $target_selector = $row_selector . ' td:nth-of-type(4)';
+    $cell = $assert->elementExists('css', $target_selector);
+    $link_url = $assert->elementExists('named', ['link', 'Release notes'], $cell)->getAttribute('href');
+    $this->assertStringContainsString(str_replace('.', '-', $expected_target_version) . '-release', $link_url);
   }
 
   /**

composer.json

@@ -14,12 +14,13 @@
     },
     "require": {
         "ext-json": "*",
-        "drupal/core": "^9.7 || ^10",
-        "php-tuf/composer-stager": "2.0-alpha1",
-        "composer-runtime-api": "^2.1"
+        "php-tuf/composer-stager": "2.0-beta3",
+        "composer-runtime-api": "^2.1",
+        "symfony/console": "^6.2"
     },
     "require-dev": {
-        "colinodell/psr-testlogger": "^1.2"
+        "colinodell/psr-testlogger": "^1.2",
+        "symfony/config": "^6.2"
     },
     "scripts": {
         "phpcbf": "scripts/phpcbf.sh",
@@ -28,19 +29,20 @@
             "Composer\\Config::disableProcessTimeout",
             "scripts/phpunit.sh"
         ],
-        "core-convert": "Drupal\\automatic_updates\\Development\\Converter::doConvert",
-        "core-commit": "Drupal\\automatic_updates\\Development\\Converter::makeCommit"
+        "core-convert": "Drupal\\automatic_updates\\Development\\ConverterCommand",
+        "core-commit": "Drupal\\automatic_updates\\Development\\ConverterCommand::makeCommit"
     },
     "scripts-descriptions": {
         "phpcbf": "Automatically fixes standards violations where possible.",
         "phpcs": "Checks code for standards compliance.",
         "test": "Runs PHPUnit tests.",
-        "core-convert": "Converts this module to a core merge request. Excepts 2 arguments. 1) The core clone directory. 2) The core merge request branch.",
+        "core-convert": "Converts this module to a core merge request. For arguments and options see \\Drupal\\automatic_updates\\Development\\ConverterCommand::configure().",
         "core-commit": "Converts this module to a core merge request. Excepts 1 arguments. The core clone directory."
     },
     "autoload": {
         "psr-4": {
             "Drupal\\automatic_updates\\Development\\": "scripts/src"
         }
-    }
+    },
+    "bin": ["auto-update"]
 }

config/install/automatic_updates.settings.yml

-cron: disable
+unattended:
+  method: web
+  level: disable
 allow_core_minor_updates: false
 status_check_mail: errors_only

config/schema/automatic_updates.schema.yml

@@ -2,9 +2,22 @@ automatic_updates.settings:
   type: config_object
   label: 'Automatic Updates settings'
   mapping:
-    cron:
-      type: string
-      label: 'Enable automatic updates during cron'
+    unattended:
+      type: mapping
+      label: 'Settings for unattended update'
+      mapping:
+        method:
+          type: string
+          label: 'Method of running unattended updates'
+          constraints:
+            NotNull: []
+            Choice: [console, web]
+        level:
+          type: string
+          label: 'Which level of unattended updates to perform'
+          constraints:
+            NotNull: []
+            Choice: [disable, security, patch]
     cron_port:
       type: integer
       label: 'Port to use for finalization sub-request'
@@ -13,4 +26,4 @@ automatic_updates.settings:
       label: 'Allow minor level Drupal core updates'
     status_check_mail:
       type: string
-      label: 'Whether to send status check failure e-mail notifications during cron'
+      label: 'Whether to send status check failure email notifications during cron'

dictionary.txt

@@ -10,4 +10,7 @@ unwritable
 filedate
 unshallow
 hhvm
-tmpdir
+proc_open
+bootable
+Syncer
+gitlabci

drupalci.yml

-# https://www.drupal.org/drupalorg/docs/drupal-ci/customizing-drupalci-testing
-# Copied from https://git.drupalcode.org/project/ckeditor5/-/blob/c68be57fbc2c29969e9726ec932af3f0e80c3dd9/drupalci.yml
-# cspell:disable
-build:
-  assessment:
-    # validate_codebase:
-    # automatic_updates code quality checking matches that of Drupal core: it is checked by container_command.commit_checks.
-    testing:
-      # Run code quality checks.
-      container_command.commit-checks:
-        commands:
-          # @todo Replace in favor of commit-code-check.sh once https://www.drupal.org/project/drupal/issues/3314100 lands.
-          - modules/contrib/automatic_updates/scripts/commit-code-check.sh --drupalci
-        halt-on-fail: true
-      # run_tests task is executed several times in order of performance speeds.
-      # halt-on-fail can be set on the run_tests tasks in order to fail fast.
-      # suppress-deprecations is false in order to be alerted to usages of
-      # deprecated code.
-      run_tests.phpunit:
-        types: 'PHPUnit-Unit'
-        testgroups: '--all'
-        suppress-deprecations: false
-        halt-on-fail: false
-      run_tests.kernel:
-        types: 'PHPUnit-Kernel'
-        testgroups: '--all'
-        suppress-deprecations: false
-        halt-on-fail: false
-      run_tests.build:
-        # Limit concurrency due to disk space concerns.
-        concurrency: 15
-        types: 'PHPUnit-Build'
-        testgroups: '--all'
-        suppress-deprecations: false
-        halt-on-fail: false
-      run_tests.functional:
-        types: 'PHPUnit-Functional'
-        testgroups: '--all'
-        suppress-deprecations: false
-        halt-on-fail: false
-      # Functional JavaScript tests require a concurrency of 1 because there is
-      # only one instance of PhantomJS on the testbot machine.
-      #run_tests.javascript:
-      #  concurrency: 1
-      #  types: 'PHPUnit-FunctionalJavascript'
-      #  testgroups: '--all'
-      #  suppress-deprecations: false
-      #  halt-on-fail: false