Skip to content
Snippets Groups Projects
Verified Commit a0442b5f authored by Dave Long's avatar Dave Long
Browse files

Issue #3466524 by catch: ModuleConfigureRouteTest is slow

(cherry picked from commit d3b144d3)
parent 136e3dcb
No related branches found
No related tags found
10 merge requests!10602Issue #3438769 by vinmayiswamy, antonnavi, michelle, amateescu: Sub workspace does not clear,!10301Issue #3469309 by mstrelan, smustgrave, moshe weitzman: Use one-time login...,!10187Issue #3487488 by dakwamine: ExtensionMimeTypeGuesser::guessMimeType must support file names with "0" (zero) like foo.0.zip,!9929Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9787Resolve issue 3479427 - bootstrap barrio issue under Windows,!9742Issue #3463908 by catch, quietone: Split OptionsFieldUiTest into two,!9526Issue #3458177 by mondrake, catch, quietone, godotislate, longwave, larowlan,...,!6502Draft: Resolve #2938524 "Plach testing issue",!38582585169-10.1.x,!3226Issue #2987537: Custom menu link entity type should not declare "bundle" entity key
Checking pipeline status
......@@ -47,17 +47,24 @@ protected function setUp(): void {
/**
* Tests if the module configure routes exists.
*
* @dataProvider coreModuleListDataProvider
*/
public function testModuleConfigureRoutes(string $module_name): void {
public function testModuleConfigureRoutes(): void {
foreach (static::coreModuleListDataProvider() as $module_name => $info) {
$this->doTestModuleConfigureRoutes($module_name);
}
}
/**
* Checks the configure route for a single module.
*/
protected function doTestModuleConfigureRoutes(string $module_name): void {
$module_info = $this->moduleInfo[$module_name]->info;
if (!isset($module_info['configure'])) {
$this->markTestSkipped("$module_name has no configure route");
return;
}
$module_lifecycle = $module_info[ExtensionLifecycle::LIFECYCLE_IDENTIFIER];
if (isset($module_lifecycle) && $module_lifecycle === ExtensionLifecycle::DEPRECATED) {
$this->markTestSkipped("$module_name is $module_lifecycle");
return;
}
$this->container->get('module_installer')->install([$module_name]);
$this->assertModuleConfigureRoutesExist($module_name, $module_info);
......@@ -70,17 +77,24 @@ public function testModuleConfigureRoutes(string $module_name): void {
* deprecated module doesn't trigger a deprecation notice.
*
* @group legacy
*
* @dataProvider coreModuleListDataProvider
*/
public function testDeprecatedModuleConfigureRoutes(string $module_name): void {
public function testDeprecatedModuleConfigureRoutes(): void {
foreach (static::coreModuleListDataProvider() as $module_name => $info) {
$this->doTestDeprecatedModuleConfigureRoutes($module_name);
}
}
/**
* Check the configure route for a single module.
*/
protected function doTestDeprecatedModuleConfigureRoutes(string $module_name): void {
$module_info = $this->moduleInfo[$module_name]->info;
if (!isset($module_info['configure'])) {
$this->markTestSkipped("$module_name has no configure route");
return;
}
$module_lifecycle = $module_info[ExtensionLifecycle::LIFECYCLE_IDENTIFIER];
if (isset($module_lifecycle) && $module_lifecycle !== ExtensionLifecycle::DEPRECATED) {
$this->markTestSkipped("$module_name is not $module_lifecycle");
return;
}
$this->container->get('module_installer')->install([$module_name]);
$this->assertModuleConfigureRoutesExist($module_name, $module_info);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment