archtechx/tenancy
1
0
Fork 0
mirror of https://github.com/archtechx/tenancy.git synced 2026-02-05 05:44:04 +00:00
Code Activity

Fix job & test

Browse source
This commit is contained in:
lukinovec 2023-04-25 12:33:29 +02:00
parent 4eab27e1f0
commit ae25e25715
2 changed files with 7 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/Jobs/CreatePostgresUserForTenant.php
View file

@ -38,7 +38,7 @@ class CreatePostgresUserForTenant implements ShouldQueue
$password = $this->tenant->database()->getPassword() ?? 'password';
// Create the user only if it doesn't already exist
if (count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0) {
if (! count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0) {
DB::statement("CREATE USER \"$name\" LOGIN PASSWORD '$password';");
}
}

12
tests/PostgresTest.php
View file

@ -15,13 +15,13 @@ test('postgres user can get created using the job', function() {
$tenant = Tenant::create();
$name = $tenant->getTenantKey();
$tenantHasPostgresUser = count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0;
$tenantHasPostgresUser = fn () => count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0;
expect($tenantHasPostgresUser)->toBeFalse();
expect($tenantHasPostgresUser())->toBeFalse();
CreatePostgresUserForTenant::dispatchSync($tenant);
expect($tenantHasPostgresUser)->toBeTrue();
expect($tenantHasPostgresUser())->toBeTrue();
});
@ -30,13 +30,13 @@ test('postgres user can get deleted using the job', function() {
$name = $tenant->getTenantKey();
CreatePostgresUserForTenant::dispatchSync($tenant);
$tenantHasPostgresUser = count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0;
$tenantHasPostgresUser = fn () => count(DB::select("SELECT usename FROM pg_user WHERE usename = '$name';")) > 0;
expect($tenantHasPostgresUser)->toBeTrue();
expect($tenantHasPostgresUser())->toBeTrue();
DeleteTenantsPostgresUser::dispatchSync($tenant);
expect($tenantHasPostgresUser)->toBeFalse();
expect($tenantHasPostgresUser())->toBeFalse();
});
test('correct rls policies get created using the command', function() {