archtechx/tenancy
1
0
Fork 0
mirror of https://github.com/archtechx/tenancy.git synced 2025-12-12 22:54:03 +00:00
Code Activity

Fix typo, improve comment

Browse source
This commit is contained in:
lukinovec 2025-01-14 16:05:28 +01:00
parent 1f41a8a863
commit cf8546fe84
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
tests/RLS/TableManagerTest.php
View file

@ -600,8 +600,9 @@ test('user without BYPASSRLS can only query owned tables if forceRls is true', f
tenancy()->end(); tenancy()->end();
if ($forceRls) { if ($forceRls) {
// RLS is forced, so by default, not even the table owner should not be able to query the table protected by the RLS policy // RLS is forced, so by default, not even the table owner should be able to query the table protected by the RLS policy.
// "unrecognized configuration parameter" = the my.current_tenant session variable isn't set -- the RLS policy is working // The RLS policy is not being bypassed, 'unrecognized configuration parameter' means
// that the my.current_tenant session variable isn't set.
expect(fn () => Order::first())->toThrow(QueryException::class, 'unrecognized configuration parameter'); expect(fn () => Order::first())->toThrow(QueryException::class, 'unrecognized configuration parameter');
} else { } else {
// RLS is not forced, so the table owner should be able to query the table, bypassing the RLS policy // RLS is not forced, so the table owner should be able to query the table, bypassing the RLS policy