Implement Keystone environment deployments
This commit is contained in:
@@ -2,20 +2,21 @@
|
||||
|
||||
use App\Actions\Services\CreateService;
|
||||
use App\Drivers\Driver;
|
||||
use App\Enums\DeployPolicy;
|
||||
use App\Enums\ServiceCategory;
|
||||
use App\Enums\ServiceStatus;
|
||||
use App\Enums\ServiceType;
|
||||
use App\Jobs\Services\DeployService;
|
||||
use App\Models\Network;
|
||||
use App\Models\Organisation;
|
||||
use App\Models\Provider;
|
||||
use App\Models\Server;
|
||||
use App\Models\Service;
|
||||
use App\Models\Organisation;
|
||||
use App\Models\User;
|
||||
use Illuminate\Foundation\Testing\RefreshDatabase;
|
||||
use Inertia\Testing\AssertableInertia;
|
||||
use Illuminate\Support\Facades\Config;
|
||||
use Illuminate\Support\Facades\Bus;
|
||||
use Illuminate\Support\Facades\Config;
|
||||
use Inertia\Testing\AssertableInertia;
|
||||
|
||||
uses(RefreshDatabase::class);
|
||||
|
||||
@@ -24,11 +25,11 @@ function setupTestEnvironment()
|
||||
$user = User::factory()->create();
|
||||
|
||||
$organisation = Organisation::factory()->create([
|
||||
'owner_id' => $user->id
|
||||
'owner_id' => $user->id,
|
||||
]);
|
||||
|
||||
$provider = Provider::factory()->create([
|
||||
'organisation_id' => $organisation->id
|
||||
'organisation_id' => $organisation->id,
|
||||
]);
|
||||
|
||||
$network = Network::create([
|
||||
@@ -61,13 +62,13 @@ test('create service page is accessible', function () {
|
||||
|
||||
$response = $this->get(route('services.create', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]));
|
||||
|
||||
$response->assertStatus(200);
|
||||
$response->assertInertia(
|
||||
fn(AssertableInertia $page) => $page
|
||||
->component('services/Create')
|
||||
fn (AssertableInertia $page) => $page
|
||||
->component('services/Create', false)
|
||||
->has('server')
|
||||
->has('services')
|
||||
);
|
||||
@@ -81,7 +82,7 @@ test('store service with valid data', function () {
|
||||
$mockDefaultCredentials = [
|
||||
'user' => 'test-user',
|
||||
'password' => 'test-password',
|
||||
'db' => 'test-db'
|
||||
'db' => 'test-db',
|
||||
];
|
||||
|
||||
$mockDriver = Mockery::mock(Driver::class);
|
||||
@@ -98,18 +99,18 @@ test('store service with valid data', function () {
|
||||
'name' => 'test-postgres-database',
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => '17',
|
||||
'version' => '18',
|
||||
];
|
||||
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]), $data);
|
||||
|
||||
// Since we're not mocking the entire CreateService action, we should get a proper redirect
|
||||
$response->assertRedirect(route('servers.show', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]));
|
||||
$response->assertSessionHas('success', 'Service created successfully');
|
||||
|
||||
@@ -118,10 +119,28 @@ test('store service with valid data', function () {
|
||||
'server_id' => $setup['server']->id,
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => '17',
|
||||
'driver_name' => 'postgres.17',
|
||||
'version' => '18',
|
||||
'version_track' => '18',
|
||||
'driver_name' => 'postgres.18',
|
||||
'deploy_policy' => DeployPolicy::DEPENDENCY_ONLY->value,
|
||||
'status' => ServiceStatus::NOT_INSTALLED->value,
|
||||
]);
|
||||
$service = Service::query()->where('name', 'test-postgres-database')->firstOrFail();
|
||||
|
||||
expect($service->credentials)
|
||||
->toHaveKey('user')
|
||||
->toHaveKey('password')
|
||||
->toHaveKey('db');
|
||||
|
||||
$this->assertDatabaseHas('service_replicas', [
|
||||
'service_id' => $service->id,
|
||||
'server_id' => $setup['server']->id,
|
||||
'container_name' => "keystone-service-{$service->id}-1",
|
||||
'internal_host' => "keystone-service-{$service->id}",
|
||||
'internal_port' => 5432,
|
||||
'status' => 'pending',
|
||||
'health_status' => 'unknown',
|
||||
]);
|
||||
|
||||
Bus::assertDispatched(DeployService::class);
|
||||
});
|
||||
@@ -140,7 +159,7 @@ test('store service with invalid data', function () {
|
||||
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]), $data);
|
||||
|
||||
$response->assertSessionHasErrors(['name', 'category', 'type', 'version']);
|
||||
@@ -152,29 +171,79 @@ test('store service validates version exists in config', function () {
|
||||
$this->actingAs($setup['user']);
|
||||
|
||||
// Mock the config to simulate the version not existing
|
||||
Config::set('keystone.services.' . ServiceCategory::DATABASE->value . '.' . ServiceType::POSTGRES->value . '.versions', [
|
||||
'16' => [
|
||||
'name' => 'PostgreSQL 16',
|
||||
'description' => 'PostgreSQL 16',
|
||||
'image' => 'postgres:16',
|
||||
]
|
||||
Config::set('keystone.services.'.ServiceCategory::DATABASE->value.'.'.ServiceType::POSTGRES->value.'.versions', [
|
||||
'17' => [
|
||||
'name' => 'PostgreSQL 17',
|
||||
'description' => 'PostgreSQL 17',
|
||||
'image' => 'postgres:17',
|
||||
],
|
||||
]);
|
||||
|
||||
$data = [
|
||||
'name' => 'test-postgres-database',
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => '17', // This version doesn't exist in our mocked config
|
||||
'version' => '18', // This version doesn't exist in our mocked config
|
||||
];
|
||||
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]), $data);
|
||||
|
||||
$response->assertSessionHasErrors(['version']);
|
||||
});
|
||||
|
||||
test('store service prevents duplicate gateway on the same server', function () {
|
||||
$setup = setupTestEnvironment();
|
||||
|
||||
$this->actingAs($setup['user']);
|
||||
|
||||
Service::factory()->for($setup['server'])->create([
|
||||
'organisation_id' => $setup['organisation']->id,
|
||||
'name' => 'gateway',
|
||||
'category' => ServiceCategory::GATEWAY,
|
||||
'type' => ServiceType::CADDY,
|
||||
'version' => '2',
|
||||
'version_track' => '2',
|
||||
'driver_name' => 'caddy.2',
|
||||
]);
|
||||
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id,
|
||||
]), [
|
||||
'name' => 'another-gateway',
|
||||
'category' => ServiceCategory::GATEWAY->value,
|
||||
'type' => ServiceType::CADDY->value,
|
||||
'version' => '2',
|
||||
]);
|
||||
|
||||
$response->assertSessionHasErrors(['category' => 'This server already has a gateway service.']);
|
||||
});
|
||||
|
||||
test('create service action prevents duplicate gateway on the same server', function () {
|
||||
$setup = setupTestEnvironment();
|
||||
|
||||
Service::factory()->for($setup['server'])->create([
|
||||
'organisation_id' => $setup['organisation']->id,
|
||||
'name' => 'gateway',
|
||||
'category' => ServiceCategory::GATEWAY,
|
||||
'type' => ServiceType::CADDY,
|
||||
'version' => '2',
|
||||
'version_track' => '2',
|
||||
'driver_name' => 'caddy.2',
|
||||
]);
|
||||
|
||||
expect(fn () => app(CreateService::class)->execute(
|
||||
server: $setup['server'],
|
||||
name: 'another-gateway',
|
||||
category: ServiceCategory::GATEWAY,
|
||||
type: ServiceType::CADDY,
|
||||
version: '2',
|
||||
))->toThrow(RuntimeException::class, 'This server already has a gateway service.');
|
||||
});
|
||||
|
||||
test('store service with non-existent server returns 404', function () {
|
||||
$setup = setupTestEnvironment();
|
||||
|
||||
@@ -184,12 +253,12 @@ test('store service with non-existent server returns 404', function () {
|
||||
'name' => 'test-postgres-database',
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => '17',
|
||||
'version' => '18',
|
||||
];
|
||||
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => 9999
|
||||
'server' => 9999,
|
||||
]), $data);
|
||||
|
||||
$response->assertStatus(404);
|
||||
@@ -202,7 +271,7 @@ test('create service page with non-existent server returns 404', function () {
|
||||
|
||||
$response = $this->get(route('services.create', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => 9999
|
||||
'server' => 9999,
|
||||
]));
|
||||
|
||||
$response->assertStatus(404);
|
||||
@@ -217,7 +286,7 @@ test('store service is properly created and dispatched', function () {
|
||||
->andReturn([
|
||||
'user' => 'test-user',
|
||||
'password' => 'test-password',
|
||||
'db' => 'test-db'
|
||||
'db' => 'test-db',
|
||||
])
|
||||
->getMock();
|
||||
|
||||
@@ -226,7 +295,7 @@ test('store service is properly created and dispatched', function () {
|
||||
'name' => 'test-postgres-database',
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => '17',
|
||||
'version' => '18',
|
||||
];
|
||||
|
||||
// Mock service class to return our mock driver
|
||||
@@ -243,7 +312,7 @@ test('store service is properly created and dispatched', function () {
|
||||
'category' => ServiceCategory::DATABASE,
|
||||
'type' => ServiceType::POSTGRES,
|
||||
'version' => $testData['version'],
|
||||
'driver_name' => 'postgres.17',
|
||||
'driver_name' => 'postgres.18',
|
||||
'status' => ServiceStatus::NOT_INSTALLED,
|
||||
]);
|
||||
|
||||
@@ -266,27 +335,15 @@ test('store service is properly created and dispatched', function () {
|
||||
// Execute request
|
||||
$response = $this->post(route('services.store', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]), $testData);
|
||||
|
||||
// Assert response
|
||||
$response->assertRedirect(route('servers.show', [
|
||||
'organisation' => $setup['organisation']->id,
|
||||
'server' => $setup['server']->id
|
||||
'server' => $setup['server']->id,
|
||||
]));
|
||||
$response->assertSessionHas('success', 'Service created successfully');
|
||||
|
||||
// Assert database state
|
||||
$this->assertDatabaseHas('services', [
|
||||
'name' => $testData['name'],
|
||||
'server_id' => $setup['server']->id,
|
||||
'category' => ServiceCategory::DATABASE->value,
|
||||
'type' => ServiceType::POSTGRES->value,
|
||||
'version' => $testData['version'],
|
||||
'driver_name' => 'postgres.17',
|
||||
'status' => ServiceStatus::NOT_INSTALLED->value,
|
||||
]);
|
||||
|
||||
// Assert job was dispatched
|
||||
Bus::assertDispatched(DeployService::class);
|
||||
Bus::assertNotDispatched(DeployService::class);
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user