fix: update StateEnum.toArray and clean up test comments per review feedback (#7)

Agent-Logs-Url: https://github.com/maniaba/CodeIgniter4/sessions/81c8ece0-932b-44e0-9757-d80c5cb2fc6a

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: maniaba <[email protected]>

Author: Copilot

tests/_support/Enum/StateEnum.php

@@ -13,17 +13,13 @@
 
 namespace Tests\Support\Enum;
 
-/**
- * An enum that also defines toArray(), used to test that UnitEnum handling
- * takes precedence over toArray() in Entity::normalizeValue().
- */
 enum StateEnum: string
 {
     case DRAFT     = 'draft';
     case PUBLISHED = 'published';
 
     public function toArray(): array
     {
-        return array_column(self::cases(), 'value');
+        return self::cases();
     }
 }

tests/system/Entity/EntityTest.php

@@ -1046,21 +1046,16 @@ public function testCastEnumSetWithUnitEnumObject(): void
         $this->assertSame(ColorEnum::RED, $entity->color);
     }
 
+    /**
+     * @see https://github.com/codeigniter4/CodeIgniter4/issues/10136
+     */
     public function testInjectRawDataWithEnumThatHasToArrayMethod(): void
     {
-        // Regression test for https://github.com/codeigniter4/CodeIgniter4/issues/10136
-        // Enums implementing toArray() must still be handled by the UnitEnum branch in
-        // normalizeValue(), so hasChanged() does not incorrectly report a change after
-        // injectRawData() stores the same enum value.
         $entity = new class () extends Entity {};
 
         $entity->injectRawData(['state' => StateEnum::DRAFT]);
 
-        // toRawArray() returns raw attributes, so the enum object is returned as-is.
         $this->assertSame(StateEnum::DRAFT, $entity->toRawArray()['state']);
-
-        // The key assertion: normalizeValue() must treat the enum as a UnitEnum
-        // (not call toArray() on it), so the original and current normalized forms match.
         $this->assertFalse($entity->hasChanged('state'));
     }