Verified Commit 90c7d4c3 authored by Lee Rowlands's avatar Lee Rowlands
Browse files

Issue #3154398 by neclimdul, quietone, alexpott: Migrations don't have an accessor for requirements

parent 1671837e

core/modules/migrate/src/Plugin/Migration.php

+7 −0
Original line number Diff line number Diff line
@@ -417,6 +417,13 @@ public function getIdMap() { 
    return $this->idMapPlugin;

  }



  /**

   * {@inheritDoc}

   */

  public function getRequirements(): array {

    return $this->requirements;

  }



  /**

   * {@inheritdoc}

   */

core/modules/migrate/src/Plugin/MigrationInterface.php

+7 −0
Original line number Diff line number Diff line
@@ -103,6 +103,13 @@ public function id(); 
   */

  public function label();



  /**

   * Get a list of required plugin IDs.

   *

   * @returns string[]

   */

  public function getRequirements(): array;



  /**

   * Returns the initialized source plugin.

   *

core/modules/migrate/tests/src/Unit/MigrationTest.php

+13 −0
Original line number Diff line number Diff line
@@ -113,6 +113,19 @@ public function testRequirementsForMigrations() { 
    $migration->checkRequirements();

  }



  /**

   * Tests getting requirement list.

   *

   * @covers ::getRequirements

   */

  public function testGetMigrations() {

    $migration = new TestMigration();



    $requirements = ['test_a', 'test_b', 'test_c', 'test_d'];

    $migration->setRequirements($requirements);

    $this->assertArrayEquals($requirements, $migration->getRequirements());

  }



  /**

   * Tests valid migration dependencies configuration returns expected values.

   *