Add the lookup_migrations option to the menu_link_parent process plugin
Closed
requested to merge issue/drupal-3498963:3498963-add-lookup-migrations-configuration into 11.x
1 unresolved thread
Closes #3498963
Merge request reports
Activity
added 240 commits
-
d24dfe45...bb1f2a6d - 234 commits from branch
project:11.x
- 38506689 - Add the lookup_migrations option to the menu_link_parent process plugin
- fa54e8f8 - Simplify validation of the new option
- 6f50b549 - Add test coverage for the lookup_migrations option
- 0f7e48a4 - Use $lookupMigrations for the class property, not $lookup_migrations
- 96be5e54 - Add return types for data providers
- b3b024a3 - Also update the property name in the test
Toggle commit list-
d24dfe45...bb1f2a6d - 234 commits from branch
92 public function testConstructorException(array $configuration, bool $is_valid): void { 93 if (!$is_valid) { 94 $this->expectException('TypeError'); 95 $this->expectExceptionMessage('Cannot assign string to property ' . MenuLinkParent::class . '::$lookupMigrations of type array'); 96 } 97 $plugin = new MenuLinkParent($configuration, 'map', [], $this->migrateLookup->reveal(), $this->menuLinkManager->reveal(), $this->menuLinkStorage->reveal(), $this->migration->reveal()); 98 if ($is_valid) { 99 $this->assertInstanceOf(MenuLinkParent::class, $plugin); 100 } 101 } 102 103 /** 104 * Provides data for testConstructorException(). 105 */ 106 public static function providerConstructorException(): array { 107 // The parent ID does not for the following tests. changed this line in version 6 of the diff
Please register or sign in to reply