ci: fix unit tests

Author: evansims

src/Authorization/OpenFgaGate.php

@@ -13,7 +13,6 @@
 use OpenFGA\Exceptions\ClientThrowable;
 use OpenFGA\Laravel\Contracts\{AuthorizableUser, AuthorizationObject, AuthorizationType, ManagerInterface, OpenFgaGateInterface};
 use Override;
-use UnitEnum;
 
 use function is_array;
 use function is_object;
@@ -30,6 +29,7 @@
  * arguments provided, making migration from Laravel's default authorization smooth.
  *
  * @template TUser of Authenticatable&Model
+ *
  * @extends Gate<object>
  */
 final class OpenFgaGate extends Gate implements OpenFgaGateInterface
@@ -67,11 +67,15 @@ public function check($abilities, $arguments = [])
         $argumentsArray = is_array($arguments) ? $arguments : [$arguments];
 
         // Handle single string ability first to avoid iterable/string confusion
-        if (is_string($abilities)) {
-            // For string abilities, check if this is an OpenFGA permission check
-            if ($this->isOpenFgaPermission($argumentsArray)) {
-                return $this->checkOpenFgaPermission($abilities, $argumentsArray, null);
-            }
+        if (! is_string($abilities)) {
+            // Fall back to Laravel's default behavior
+            // Note: Laravel's Gate::check() is designed to handle varied argument types internally
+            return parent::check($abilities, $arguments);
+        }
+
+        // For string abilities, check if this is an OpenFGA permission check
+        if ($this->isOpenFgaPermission($argumentsArray)) {
+            return $this->checkOpenFgaPermission($abilities, $argumentsArray, null);
         }
 
         // Fall back to Laravel's default behavior
@@ -112,6 +116,22 @@ public function checkOpenFgaPermission(string $ability, mixed $arguments, ?Authe
         return $this->manager->check($userId, $ability, $object);
     }
 
+    /**
+     * Get a gate instance for the given user.
+     *
+     * @param  Authenticatable|mixed $user
+     * @return static
+     */
+    #[Override]
+    public function forUser($user)
+    {
+        return new static(
+            $this->manager,
+            $this->container,
+            static fn (): ?Authenticatable => $user instanceof Authenticatable ? $user : null,
+        );
+    }
+
     /**
      * Determine if arguments represent an OpenFGA permission check.
      *

tests/Unit/Authorization/OpenFgaGateTest.php

@@ -165,7 +165,7 @@ enum OpenFgaGateTest
             );
         });
 
-        it('handles custom user parameter', function (): void {
+        it('handles custom user via forUser method', function (): void {
             $customUser = TestFactories::createTestUser(
                 authId: TestConstants::ALTERNATIVE_USER_ID,
                 identifier: 999,
@@ -177,7 +177,8 @@ enum OpenFgaGateTest
                 ->with('user:999', 'admin', 'system:1')
                 ->andReturn(true);
 
-            $result = $this->gate->check('admin', 'system:1', $customUser);
+            // Use Laravel's standard way to check permissions for a different user
+            $result = $this->gate->forUser($customUser)->check('admin', 'system:1');
             expect($result)->toBeTrue();
         });
     });