fix(mapper): case-insensitive null string check in Integer/Float casters, add factory integration tests for nullable path

Author: laylatichy

packages/mapper/src/Casters/FloatCaster.php

@@ -35,6 +35,10 @@ public static function configure(PropertyReflector $property, Context $context):
 
     public function cast(mixed $input): ?float
     {
+        if (is_string($input)) {
+            $input = mb_strtolower($input);
+        }
+
         if ($this->nullable && ($input === null || $input === '' || $input === 'null')) {
             return null;
         }

packages/mapper/src/Casters/IntegerCaster.php

@@ -35,6 +35,10 @@ public static function configure(PropertyReflector $property, Context $context):
 
     public function cast(mixed $input): ?int
     {
+        if (is_string($input)) {
+            $input = mb_strtolower($input);
+        }
+
         if ($this->nullable && ($input === null || $input === '' || $input === 'null')) {
             return null;
         }

tests/Integration/Mapper/CasterFactoryTest.php

@@ -24,8 +24,11 @@ public function test_for_property(): void
         $class = reflect(ObjectWithSerializerProperties::class);
 
         $this->assertInstanceOf(IntegerCaster::class, $factory->forProperty($class->getProperty('intProp')));
+        $this->assertInstanceOf(IntegerCaster::class, $factory->forProperty($class->getProperty('nullableIntProp')));
         $this->assertInstanceOf(FloatCaster::class, $factory->forProperty($class->getProperty('floatProp')));
+        $this->assertInstanceOf(FloatCaster::class, $factory->forProperty($class->getProperty('nullableFloatProp')));
         $this->assertInstanceOf(BooleanCaster::class, $factory->forProperty($class->getProperty('boolProp')));
+        $this->assertInstanceOf(BooleanCaster::class, $factory->forProperty($class->getProperty('nullableBoolProp')));
         $this->assertInstanceOf(NativeDateTimeCaster::class, $factory->forProperty($class->getProperty('nativeDateTimeImmutableProp')));
         $this->assertInstanceOf(NativeDateTimeCaster::class, $factory->forProperty($class->getProperty('nativeDateTimeProp')));
         $this->assertInstanceOf(NativeDateTimeCaster::class, $factory->forProperty($class->getProperty('nativeDateTimeInterfaceProp')));

tests/Integration/Mapper/Casters/FloatCasterTest.php

@@ -41,6 +41,8 @@ public function nullable_cast_with_values(mixed $input, float $expected): void
     #[TestWith([null])]
     #[TestWith([''])]
     #[TestWith(['null'])]
+    #[TestWith(['NULL'])]
+    #[TestWith(['Null'])]
     public function nullable_cast_returns_null_for_empty_input(mixed $input): void
     {
         $caster = new FloatCaster(nullable: true);

tests/Integration/Mapper/Casters/IntegerCasterTest.php

@@ -41,6 +41,8 @@ public function nullable_cast_with_values(mixed $input, int $expected): void
     #[TestWith([null])]
     #[TestWith([''])]
     #[TestWith(['null'])]
+    #[TestWith(['NULL'])]
+    #[TestWith(['Null'])]
     public function nullable_cast_returns_null_for_empty_input(mixed $input): void
     {
         $caster = new IntegerCaster(nullable: true);

tests/Integration/Mapper/Fixtures/ObjectWithSerializerProperties.php

@@ -18,10 +18,16 @@ final class ObjectWithSerializerProperties
 
     public int $intProp = 1;
 
+    public ?int $nullableIntProp = null;
+
     public float $floatProp = 0.1;
 
+    public ?float $nullableFloatProp = null;
+
     public bool $boolProp = true;
 
+    public ?bool $nullableBoolProp = null;
+
     public array $arrayProp = ['a'];
 
     #[SerializeWith(DoubleStringSerializer::class)]