Initialize property and guard hasMethod() (#320)

Author: behzadsp

src/Concerns/DecorateActions.php

@@ -4,7 +4,7 @@
 
 trait DecorateActions
 {
-    protected mixed $action;
+    protected mixed $action = null;
 
     public function setAction($action): self
     {
@@ -30,7 +30,7 @@ protected function getProperty(string $property)
 
     protected function hasMethod(string $method): bool
     {
-        return method_exists($this->action, $method);
+        return isset($this->action) && method_exists($this->action, $method);
     }
 
     protected function callMethod(string $method, array $parameters = [])

tests/DecorateActionsTest.php

@@ -0,0 +1,36 @@
+<?php
+
+use Lorisleiva\Actions\ActionRequest;
+
+beforeEach(function () {
+    $this->request = new class extends ActionRequest {
+        public function authorize(): bool { return true; }
+        public function rules(): array   { return []; }
+        public function handle()          { /* no-op */ }
+    };
+
+    $method            = new \ReflectionMethod($this->request, 'hasMethod');
+    $method->setAccessible(true);
+    $this->hasMethod = $method;
+});
+
+it('returns false and does not throw when action is uninitialized', function () {
+    expect($this->hasMethod->invoke($this->request, 'anything'))
+        ->toBeFalse();
+});
+
+it('returns true for an existing method after setAction()', function () {
+    // A dummy action object declaring customFoo()
+    $dummyAction = new class {
+        public function customFoo(): string
+        {
+            return 'bar';
+        }
+    };
+
+    // Initialize the request’s $action
+    $this->request->setAction($dummyAction);
+
+    expect($this->hasMethod->invoke($this->request, 'customFoo'))
+        ->toBeTrue();
+});