Fix: Already existing enum

antoninmasek · Aug 15, 2023 · dac7434 · dac7434
1 parent 7652b8c
commit dac7434
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/Casters/EnumCaster.php b/src/Casters/EnumCaster.php
@@ -14,6 +14,10 @@ public function cast(mixed $value): mixed
             return null;
         }
 
+        if ($value instanceof $this->class_name) {
+            return $value;
+        }
+
         return $this->class_name::from($value);
     }
 }
diff --git a/tests/SimpleHydratorTest.php b/tests/SimpleHydratorTest.php
@@ -373,7 +373,7 @@ public function testItThrowsExceptionWhenAssociativeArrayIsUsedWithCollectionFro
         Human::collectionFromArray($this->data);
     }
 
-    public function testItCanParseEnum()
+    public function testItCanParseEnumFromValue()
     {
         $data = [
             'brand' => 'Chevrolet',
@@ -388,6 +388,21 @@ public function testItCanParseEnum()
         $this->assertSame(Color::YELLOW, $camaro->color);
     }
 
+    public function testItCanParseEnum()
+    {
+        $data = [
+            'brand' => 'Chevrolet',
+            'color' => Color::YELLOW,
+            'type' => 'Camaro',
+            'keys' => null,
+        ];
+
+        /** @var Car $camaro */
+        $camaro = Hydrator::hydrate(Car::class, $data);
+
+        $this->assertSame(Color::YELLOW, $camaro->color);
+    }
+
     public function testItFailsWithInvalidEnum()
     {
         $data = [