Type pedantry fixes.

Author: Crell

src/EnvMapper.php

@@ -20,7 +20,7 @@ class EnvMapper
      * @param bool $requireValues
      *   If true, any unmatched properties will result in an exception.  If false, unmatched properties
      *   will be ignored, which in most cases means they will be uninitialized.
-     * @param array<string, mixed>|null $source
+     * @param array<string, string|int|float>|null $source
      *   The array to map from.  If not specified, $_ENV will be used. Note that because the
      *   primary use case is environment variables, the input array MUST have keys that are UPPER_CASE
      *   strings.
@@ -71,12 +71,12 @@ public function map(string $class, bool $requireValues = false, ?array $source =
      * push them into well-typed numeric fields we need to cast them
      * appropriately.
      *
-     * @param string $val
+     * @param string|int|float $val
      *   The value to normalize.
      * @return int|float|string|bool
      *   The passed value, but now with the correct type.
      */
-    private function typeNormalize(string $val, \ReflectionProperty $rProp): int|float|string|bool|\BackedEnum
+    private function typeNormalize(string|int|float $val, \ReflectionProperty $rProp): int|float|string|bool|\BackedEnum
     {
         $rType = $rProp->getType();
         if ($rType instanceof \ReflectionNamedType) {
@@ -90,18 +90,18 @@ private function typeNormalize(string $val, \ReflectionProperty $rProp): int|flo
                 assert($backingType instanceof \ReflectionNamedType);
                 $isIntBacked = $backingType->getName() === 'int';
 
-                return $name::from($isIntBacked ? (int) $val : $val);
+                return $name::from($isIntBacked ? (int) $val : (string) $val);
             }
 
             return match ($name) {
                 'string' => $val,
                 'float' => is_numeric($val)
                     ? (float) $val
                     : throw TypeMismatch::create($rProp->getDeclaringClass()->getName(), $rProp->getName(), $val),
-                'int' => (is_numeric($val) && floor((float) $val) === (float) $val)
+                'int' => (is_numeric($val) && is_int($val + 0))
                     ? (int) $val
                     : throw TypeMismatch::create($rProp->getDeclaringClass()->getName(), $rProp->getName(), $val),
-                'bool' => in_array(strtolower($val), [1, '1', 'true', 'yes', 'on'], false),
+                'bool' => in_array(strtolower((string) $val), [1, '1', 'true', 'yes', 'on'], false),
                 default => throw TypeMismatch::create($rProp->getDeclaringClass()->getName(), $rProp->getName(), $val),
             };
         }

tests/EnvMapperTest.php

@@ -36,11 +36,11 @@ public function mapping_different_types_with_defaults_parses_correctly(): void
         /** @var SampleEnvironment $env */
         $env = $mapper->map(SampleEnvironment::class, source: $this->source);
 
-        self::assertNotNull($env->phpVersion);
-        self::assertNotNull($env->xdebug_mode);
-        self::assertNotNull($env->PATH);
-        self::assertNotNull($env->hostname);
-        self::assertNotNull($env->shlvl);
+        self::assertEquals($this->source['PHP_VERSION'], $env->phpVersion);
+        self::assertEquals($this->source['XDEBUG_MODE'], $env->xdebug_mode);
+        self::assertEquals($this->source['PATH'], $env->PATH);
+        self::assertEquals($this->source['HOSTNAME'], $env->hostname);
+        self::assertEquals($this->source['SHLVL'], $env->shlvl);
         self::assertSame('01234', $env->zipCode);
         self::assertSame(true, $env->bool);
         self::assertEquals(StringBackedEnum::Foo, $env->stringBackedEnum);