diff --git a/phpunit.xml b/phpunit.xml index 1d7b8a2..b0c0b77 100644 --- a/phpunit.xml +++ b/phpunit.xml @@ -45,7 +45,8 @@ tests/Architecture/AsideNavigationContractTest.php - tests/Architecture/AuthHelpLinksContractTest.php + tests/Architecture/AuthHelpLinksPageContractTest.php + tests/Architecture/AuthHelpLinksPartialContractTest.php tests/Architecture/AuthLoginAccessibilityContractTest.php tests/Architecture/AuthzUiActionContractTest.php tests/Architecture/AuthzUiLayoutContractTest.php diff --git a/tests/Architecture/AuthHelpLinksContractFiles.php b/tests/Architecture/AuthHelpLinksContractFiles.php new file mode 100644 index 0000000..fa9b7a0 --- /dev/null +++ b/tests/Architecture/AuthHelpLinksContractFiles.php @@ -0,0 +1,20 @@ + + */ + public static function secondaryAuthPages(): array + { + return [ + 'pages/auth/register(login).phtml', + 'pages/auth/forgot(login).phtml', + 'pages/auth/verify(login).phtml', + 'pages/auth/reset(login).phtml', + 'pages/auth/verify-email(login).phtml', + ]; + } +} diff --git a/tests/Architecture/AuthHelpLinksContractTest.php b/tests/Architecture/AuthHelpLinksContractTest.php deleted file mode 100644 index 6f19fe1..0000000 --- a/tests/Architecture/AuthHelpLinksContractTest.php +++ /dev/null @@ -1,57 +0,0 @@ -readProjectFile('templates/partials/auth-help-links.phtml'); - - $this->assertStringContainsString('$authHelpContext', $content); - $this->assertStringContainsString("'show_forgot'", $content); - $this->assertStringContainsString("'show_register'", $content); - $this->assertStringContainsString("'show_back_to_login'", $content); - $this->assertStringContainsString("'show_support'", $content); - } - - public function testRegisterLinkRemainsGuardedByRegistrationFlag(): void - { - $content = $this->readProjectFile('templates/partials/auth-help-links.phtml'); - - $this->assertStringContainsString('allowRegistration()', $content); - $this->assertStringContainsString("lurl('register')", $content); - } - - public function testLoginTemplateDefinesStageSpecificLinkMatrix(): void - { - $content = $this->readProjectFile('pages/auth/login(login).phtml'); - - $this->assertStringContainsString("'show_register' => true", $content); - $this->assertStringContainsString("'show_support' => true", $content); - $this->assertStringContainsString('\'show_forgot\' => !empty($selectedTenantMethods[\'local\'])', $content); - $this->assertStringContainsString('$stage === \'choose_tenant\'', $content); - } - - public function testSecondaryAuthPagesProvideBackToLoginAndSupportLinks(): void - { - $files = [ - 'pages/auth/register(login).phtml', - 'pages/auth/forgot(login).phtml', - 'pages/auth/verify(login).phtml', - 'pages/auth/reset(login).phtml', - 'pages/auth/verify-email(login).phtml', - ]; - - foreach ($files as $file) { - $content = $this->readProjectFile($file); - $this->assertStringContainsString("'show_back_to_login' => true", $content, $file); - $this->assertStringContainsString("'show_support' => true", $content, $file); - $this->assertStringContainsString("templatePath('partials/auth-help-links.phtml')", $content, $file); - } - } -} diff --git a/tests/Architecture/AuthHelpLinksPageContractTest.php b/tests/Architecture/AuthHelpLinksPageContractTest.php new file mode 100644 index 0000000..3050234 --- /dev/null +++ b/tests/Architecture/AuthHelpLinksPageContractTest.php @@ -0,0 +1,30 @@ +readProjectFile('pages/auth/login(login).phtml'); + + $this->assertStringContainsString("'show_register' => true", $content); + $this->assertStringContainsString("'show_support' => true", $content); + $this->assertStringContainsString('\'show_forgot\' => !empty($selectedTenantMethods[\'local\'])', $content); + $this->assertStringContainsString('$stage === \'choose_tenant\'', $content); + } + + public function testSecondaryAuthPagesProvideBackToLoginAndSupportLinks(): void + { + foreach (AuthHelpLinksContractFiles::secondaryAuthPages() as $file) { + $content = $this->readProjectFile($file); + $this->assertStringContainsString("'show_back_to_login' => true", $content, $file); + $this->assertStringContainsString("'show_support' => true", $content, $file); + $this->assertStringContainsString("templatePath('partials/auth-help-links.phtml')", $content, $file); + } + } +} diff --git a/tests/Architecture/AuthHelpLinksPartialContractTest.php b/tests/Architecture/AuthHelpLinksPartialContractTest.php new file mode 100644 index 0000000..92fec48 --- /dev/null +++ b/tests/Architecture/AuthHelpLinksPartialContractTest.php @@ -0,0 +1,29 @@ +readProjectFile('templates/partials/auth-help-links.phtml'); + + $this->assertStringContainsString('$authHelpContext', $content); + $this->assertStringContainsString("'show_forgot'", $content); + $this->assertStringContainsString("'show_register'", $content); + $this->assertStringContainsString("'show_back_to_login'", $content); + $this->assertStringContainsString("'show_support'", $content); + } + + public function testRegisterLinkRemainsGuardedByRegistrationFlag(): void + { + $content = $this->readProjectFile('templates/partials/auth-help-links.phtml'); + + $this->assertStringContainsString('allowRegistration()', $content); + $this->assertStringContainsString("lurl('register')", $content); + } +} diff --git a/tests/README.md b/tests/README.md index 263996e..602478b 100644 --- a/tests/README.md +++ b/tests/README.md @@ -35,4 +35,4 @@ ohne dass jede Aenderung immer die komplette Suite im Kopf behalten muss. - `tests/Architecture/ContainerFactoryContractTest.php` - `tests/Architecture/SecurityLoggingContractTest.php` - `tests/Architecture/StatusTaxonomyContractTest.php` -- `tests/Architecture/AuthHelpLinksContractTest.php` +- `tests/Architecture/AuthLoginAccessibilityContractTest.php`