From cf8546fe84eed27f4f50fdbe970e393d49ffb3c0 Mon Sep 17 00:00:00 2001 From: lukinovec Date: Tue, 14 Jan 2025 16:05:28 +0100 Subject: [PATCH] Fix typo, improve comment --- tests/RLS/TableManagerTest.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/RLS/TableManagerTest.php b/tests/RLS/TableManagerTest.php index 723ae3da..c4bf6658 100644 --- a/tests/RLS/TableManagerTest.php +++ b/tests/RLS/TableManagerTest.php @@ -600,8 +600,9 @@ test('user without BYPASSRLS can only query owned tables if forceRls is true', f tenancy()->end(); 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 - // "unrecognized configuration parameter" = the my.current_tenant session variable isn't set -- the RLS policy is working + // RLS is forced, so by default, not even the table owner should be able to query the table protected by the RLS policy. + // 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'); } else { // RLS is not forced, so the table owner should be able to query the table, bypassing the RLS policy