diff --git a/core/modules/migrate/tests/src/Kernel/Plugin/MigrationProvidersExistTest.php b/core/modules/migrate/tests/src/Kernel/Plugin/MigrationProvidersExistTest.php index 517299e..5357407 100644 --- a/core/modules/migrate/tests/src/Kernel/Plugin/MigrationProvidersExistTest.php +++ b/core/modules/migrate/tests/src/Kernel/Plugin/MigrationProvidersExistTest.php @@ -178,20 +178,22 @@ public function testFieldProvidersExist() { * @dataProvider fieldPluginDefinitionsProvider */ public function testFieldProviderMissingRequiredProperty(array $definitions, $missing_property) { - $mock_discovery = $this->getMockBuilder(MigrateFieldPluginManager::class) + $discovery = $this->getMockBuilder(MigrateFieldPluginManager::class) ->disableOriginalConstructor() ->setMethods(['getDefinitions']) ->getMock(); - $mock_discovery->method('getDefinitions') + $discovery->method('getDefinitions') ->willReturn($definitions); - $mock_plugin_manager = $this->getMockBuilder(MigrateFieldPluginManager::class) + + $plugin_manager = $this->getMockBuilder(MigrateFieldPluginManager::class) ->disableOriginalConstructor() ->setMethods(['getDiscovery']) ->getMock(); - $mock_plugin_manager->method('getDiscovery') - ->willReturn($mock_discovery); + $plugin_manager->method('getDiscovery') + ->willReturn($discovery); + $this->setExpectedException(InvalidPluginDefinitionException::class, "The missing_{$missing_property} plugin should define the $missing_property property."); - $mock_plugin_manager->getDefinitions(); + $plugin_manager->getDefinitions(); } /**