Change the format of missing requirements message for migrate source …

…plugins (drush-ops#5051).
Matroskeen · Jan 29, 2022 · 2ef70f0 · 2ef70f0
1 parent 4700fec
commit 2ef70f0
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Drupal/Commands/core/MigrateRunnerCommands.php b/src/Drupal/Commands/core/MigrateRunnerCommands.php
@@ -788,7 +788,7 @@ protected function getMigrationList(?string $migrationIds, ?string $tags): array
                     $sourcePlugin->checkRequirements();
                 }
             } catch (RequirementsException $exception) {
-                $this->logger()->debug("Migration '$migrationId' is skipped as its source plugin has missed requirements: {$exception->getRequirementsString()}");
+                $this->logger()->debug("Migration '$migrationId' is skipped as its source plugin has missed requirements: {$exception->getMessage()}");
                 unset($migrations[$migrationId]);
             }
         }

diff --git a/tests/fixtures/modules/woot/src/Plugin/migrate/source/TestSourceWithMissedRequirements.php b/tests/fixtures/modules/woot/src/Plugin/migrate/source/TestSourceWithMissedRequirements.php
@@ -30,7 +30,7 @@ public function getIds(): array
 
     public function checkRequirements(): array
     {
-        throw new RequirementsException('message', [
+        throw new RequirementsException('Requirements message.', [
           'type1' => ['a', 'b', 'c'],
           'type2' => ['x', 'y', 'z'],
         ]);

diff --git a/tests/integration/MigrateRunnerTest.php b/tests/integration/MigrateRunnerTest.php
@@ -43,7 +43,7 @@ public function testMigrateStatus(): void
 
         // Debug message registered for 'test_migration_source_issues'.
         // @see \Drush\Drupal\Commands\core\MigrateRunnerCommands::getMigrationList()
-        $this->assertStringContainsString("[debug] Migration 'test_migration_source_issues' is skipped as its source plugin has missed requirements: type1: a. type1: b. type1: c. type2: x. type2: y. type2: z.", $this->getErrorOutputRaw());
+        $this->assertStringContainsString("[debug] Migration 'test_migration_source_issues' is skipped as its source plugin has missed requirements: Requirements message.", $this->getErrorOutputRaw());
 
         // With arguments.
         $this->drush(