From a6be918569ffe91421e63f29bde71765ca2f2d75 Mon Sep 17 00:00:00 2001 From: brendt Date: Tue, 27 Jan 2026 11:45:50 +0100 Subject: [PATCH 1/3] wip --- .../auth/src/Installer/AuthenticationInstaller.php | 9 +++------ packages/database/src/Testing/DatabaseTester.php | 1 + .../Integration/Auth/Installer/OAuthInstallerTest.php | 10 ++++++++++ 3 files changed, 14 insertions(+), 6 deletions(-) diff --git a/packages/auth/src/Installer/AuthenticationInstaller.php b/packages/auth/src/Installer/AuthenticationInstaller.php index d80395421..29ec896e0 100644 --- a/packages/auth/src/Installer/AuthenticationInstaller.php +++ b/packages/auth/src/Installer/AuthenticationInstaller.php @@ -5,17 +5,16 @@ namespace Tempest\Auth\Installer; use Tempest\Console\Console; +use Tempest\Console\ConsoleCommand; use Tempest\Console\Input\ConsoleArgumentBag; use Tempest\Container\Container; use Tempest\Core\Installer; use Tempest\Core\PublishesFiles; use Tempest\Database\Migrations\MigrationManager; -use function Tempest\root_path; use function Tempest\src_path; -use function Tempest\Support\Namespace\to_fqcn; -if (class_exists(\Tempest\Console\ConsoleCommand::class)) { +if (class_exists(ConsoleCommand::class)) { final class AuthenticationInstaller implements Installer { use PublishesFiles; @@ -36,9 +35,7 @@ public function install(): void $this->publishImports(); if ($migration && $this->shouldMigrate()) { - $this->migrationManager->executeUp( - migration: $this->container->get(to_fqcn($migration, root: root_path())), - ); + $this->migrationManager->up(); } if ($this->shouldInstallOAuth()) { diff --git a/packages/database/src/Testing/DatabaseTester.php b/packages/database/src/Testing/DatabaseTester.php index 93d80430f..41430006a 100644 --- a/packages/database/src/Testing/DatabaseTester.php +++ b/packages/database/src/Testing/DatabaseTester.php @@ -4,6 +4,7 @@ use PHPUnit\Framework\Assert; use Tempest\Container\Container; +use Tempest\Database\Exceptions\QueryWasInvalid; use Tempest\Database\Migrations\MigrationManager; use function Tempest\Database\query; diff --git a/tests/Integration/Auth/Installer/OAuthInstallerTest.php b/tests/Integration/Auth/Installer/OAuthInstallerTest.php index 22e754b60..2303faecb 100644 --- a/tests/Integration/Auth/Installer/OAuthInstallerTest.php +++ b/tests/Integration/Auth/Installer/OAuthInstallerTest.php @@ -33,6 +33,16 @@ protected function tearDown(): void parent::tearDown(); } + + #[Test] + public function install_oauth_provider_with_migrations(): void + { + $this->console + ->call('install auth --oauth --force --migrate') + ->input(0) + ->assertSuccess(); + } + #[Test] #[DataProvider('oauthProvider')] public function install_oauth_provider( From 0eef6a3c2094a1ac0e24618df010160bce6bebfc Mon Sep 17 00:00:00 2001 From: brendt Date: Tue, 27 Jan 2026 11:46:38 +0100 Subject: [PATCH 2/3] wip --- packages/database/src/Testing/DatabaseTester.php | 1 - tests/Integration/Auth/Installer/OAuthInstallerTest.php | 1 - 2 files changed, 2 deletions(-) diff --git a/packages/database/src/Testing/DatabaseTester.php b/packages/database/src/Testing/DatabaseTester.php index 41430006a..93d80430f 100644 --- a/packages/database/src/Testing/DatabaseTester.php +++ b/packages/database/src/Testing/DatabaseTester.php @@ -4,7 +4,6 @@ use PHPUnit\Framework\Assert; use Tempest\Container\Container; -use Tempest\Database\Exceptions\QueryWasInvalid; use Tempest\Database\Migrations\MigrationManager; use function Tempest\Database\query; diff --git a/tests/Integration/Auth/Installer/OAuthInstallerTest.php b/tests/Integration/Auth/Installer/OAuthInstallerTest.php index 2303faecb..bae124f74 100644 --- a/tests/Integration/Auth/Installer/OAuthInstallerTest.php +++ b/tests/Integration/Auth/Installer/OAuthInstallerTest.php @@ -33,7 +33,6 @@ protected function tearDown(): void parent::tearDown(); } - #[Test] public function install_oauth_provider_with_migrations(): void { From e5f48af53dbed97e69d5e97adc7e6618ef1add2b Mon Sep 17 00:00:00 2001 From: brendt Date: Tue, 27 Jan 2026 11:49:05 +0100 Subject: [PATCH 3/3] wip --- .../Installer/AuthenticationInstallerTest.php | 40 +++++++++++++++++++ .../Auth/Installer/OAuthInstallerTest.php | 9 ----- 2 files changed, 40 insertions(+), 9 deletions(-) create mode 100644 tests/Integration/Auth/Installer/AuthenticationInstallerTest.php diff --git a/tests/Integration/Auth/Installer/AuthenticationInstallerTest.php b/tests/Integration/Auth/Installer/AuthenticationInstallerTest.php new file mode 100644 index 000000000..dae280110 --- /dev/null +++ b/tests/Integration/Auth/Installer/AuthenticationInstallerTest.php @@ -0,0 +1,40 @@ +installer + ->configure( + __DIR__ . '/install', + new Psr4Namespace('App\\', __DIR__ . '/install/App'), + ) + ->setRoot(__DIR__ . '/install') + ->put('.env.example', '') + ->put('.env', ''); + } + + protected function tearDown(): void + { + $this->installer->clean(); + + parent::tearDown(); + } + + #[Test] + public function install_oauth_provider_with_migrations(): void + { + $this->console + ->call('install auth --force --migrate') + ->input(0) + ->assertSuccess(); + } +} diff --git a/tests/Integration/Auth/Installer/OAuthInstallerTest.php b/tests/Integration/Auth/Installer/OAuthInstallerTest.php index bae124f74..22e754b60 100644 --- a/tests/Integration/Auth/Installer/OAuthInstallerTest.php +++ b/tests/Integration/Auth/Installer/OAuthInstallerTest.php @@ -33,15 +33,6 @@ protected function tearDown(): void parent::tearDown(); } - #[Test] - public function install_oauth_provider_with_migrations(): void - { - $this->console - ->call('install auth --oauth --force --migrate') - ->input(0) - ->assertSuccess(); - } - #[Test] #[DataProvider('oauthProvider')] public function install_oauth_provider(