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

Delete excessive test

Browse source
This commit is contained in:
lukinovec 2025-06-09 13:51:41 +02:00
parent 0dce61bd34
commit 1d87e4b00f
1 changed files with 3 additions and 27 deletions
Download patch file Download diff file Expand all files Collapse all files

30
tests/RLS/TableManagerTest.php
View file

@ -676,6 +676,9 @@ test('table manager throws an exception when the only generated paths lead throu
// Add another recursive relationship to demonstrate a more complex case // Add another recursive relationship to demonstrate a more complex case
$table->foreignId('related_post_id')->comment('rls recursive_posts.id'); $table->foreignId('related_post_id')->comment('rls recursive_posts.id');
// Add a foreign key with constraint to the current table (= self-referencing constraint)
$table->foreignId('parent_comment_id')->comment('rls recursive_comments.id');
// Add tenant_id to break the recursion - RecursiveRelationshipException should not be thrown // Add tenant_id to break the recursion - RecursiveRelationshipException should not be thrown
$table->string('tenant_id')->comment('rls')->nullable(); $table->string('tenant_id')->comment('rls')->nullable();
$table->foreign('tenant_id')->references('id')->on('tenants')->onUpdate('cascade')->onDelete('cascade'); $table->foreign('tenant_id')->references('id')->on('tenants')->onUpdate('cascade')->onDelete('cascade');
@ -784,33 +787,6 @@ test('table manager throws an exception when comment constraint is incorrect', f
['rls tenants.nonexistent-column', 'references non-existent column'], ['rls tenants.nonexistent-column', 'references non-existent column'],
]); ]);
test('table manager handles tables with self-referencing foreign keys correctly', function() {
Schema::create('self_referencing_table', function (Blueprint $table) {
$table->id();
$table->unsignedBigInteger('parent_id')->nullable()->comment('rls');
$table->foreign('parent_id')->references('id')->on('self_referencing_table');
$table->string('tenant_id')->comment('rls');
$table->foreign('tenant_id')->references('id')->on('tenants');
});
/** @var TableRLSManager $manager */
$manager = app(TableRLSManager::class);
// Should not throw an exception and should prefer the direct tenant_id path
$shortestPaths = $manager->shortestPaths();
expect($shortestPaths)->toHaveKey('self_referencing_table');
expect($shortestPaths['self_referencing_table'])->toBe([
[
'foreignKey' => 'tenant_id',
'foreignTable' => 'tenants',
'foreignId' => 'id',
],
]);
});
function createPostgresUser(string $username, string $password = 'password', bool $bypassRls = false): array function createPostgresUser(string $username, string $password = 'password', bool $bypassRls = false): array
{ {
try { try {