Verified Commit af93e630 authored by Jess's avatar Jess
Browse files

Issue #3015369 by quietone, mikelutz, xjm: Fix... 

Issue #3015369 by quietone, mikelutz, xjm: Fix MigrateTestBase::executeMigrations() to execute migrations in dependency order

(cherry picked from commit 31911458)
parent 56cfeaf5

core/modules/migrate/tests/src/Kernel/MigrateTestBase.php

+2 −6
Original line number Diff line number Diff line
@@ -186,12 +186,8 @@ protected function executeMigration($migration) { 
   */

  protected function executeMigrations(array $ids) {

    $manager = $this->container->get('plugin.manager.migration');

    array_walk($ids, function ($id) use ($manager) {

      // This is possibly a base plugin ID and we want to run all derivatives.

      $instances = $manager->createInstances($id);

      $this->assertNotEmpty($instances, sprintf("No migrations created for id '%s'.", $id));

    $instances = $manager->createInstances($ids);

    array_walk($instances, [$this, 'executeMigration']);

    });

  }



  /**

core/modules/node/tests/src/Kernel/Migrate/d7/MigrateNodeTest.php

+1 −1
Original line number Diff line number Diff line
@@ -60,8 +60,8 @@ protected function setUp(): void { 
      'd7_language_content_settings',

      'd7_comment_field',

      'd7_comment_field_instance',

      'd7_node',

      'd7_node_translation',

      'd7_node',

      'd7_entity_translation_settings',

      'd7_taxonomy_term_entity_translation',

      'd7_node_entity_translation',