Refactor Action class to use ActionState enum

Author: Safemood

src/Action.php

@@ -2,6 +2,8 @@
 
 namespace Safemood\Workflow;
 
+use Safemood\Workflow\Enums\ActionState;
+
 abstract class Action
 {
     private $state;
@@ -19,20 +21,20 @@ abstract public function handle(array &$context);
     /**
      * Get the state of the action.
      *
-     * @return mixed
+     * @return ActionState|null
      */
-    public function getState()
+    public function getState(): ?ActionState
     {
         return $this->state;
     }
 
     /**
      * Set the state of the action.
      *
-     * @param  mixed  $state
+     * @param  ActionState $state
      * @return void
      */
-    public function setState($state)
+    public function setState(ActionState $state): void
     {
         $this->state = $state;
     }
@@ -42,7 +44,7 @@ public function setState($state)
      *
      * @return \Exception|null
      */
-    public function getException()
+    public function getException(): ?\Exception
     {
         return $this->exception;
     }
@@ -53,7 +55,7 @@ public function getException()
      * @param  \Exception|null  $exception
      * @return void
      */
-    public function setException($exception)
+    public function setException(?\Exception $exception): void
     {
         $this->exception = $exception;
     }

src/Enums/ActionState.php

@@ -2,13 +2,10 @@
 
 namespace Safemood\Workflow\Enums;
 
-class ActionState
+enum ActionState: string
 {
-    const PENDING = 'pending';
-
-    const RUNNING = 'running';
-
-    const SUCCESS = 'success';
-
-    const FAILED = 'failed';
+    case PENDING = 'pending';
+    case RUNNING = 'running';
+    case SUCCESS = 'success';
+    case FAILED = 'failed';
 }