Remove unnecessary forward compatibility, incorrect Pure statement

Author: navarr

src/IterableToArray.php

@@ -9,9 +9,6 @@
 
 namespace Navarr\Utils;
 
-use JetBrains\PhpStorm\Pure;
-use Traversable;
-
 use function is_array;
 use function iterator_to_array;
 
@@ -31,45 +28,12 @@ class IterableToArray
      * </p>
      * @return array<T> An array containing the elements of the iterable.
      */
-    #[Pure]
     public static function convert(iterable $iterable, bool $preserve_keys = true): array
     {
         if (is_array($iterable)) {
             return $iterable;
         }
 
-        if ($iterable instanceof Traversable) {
-            return iterator_to_array($iterable, $preserve_keys);
-        }
-
-        // Fallback for supposedly impossible scenario
-        return self::fallbackConvert($iterable, $preserve_keys);
-    }
-
-    /**
-     * Fallback to convert an iterable into an array
-     *
-     * This should, theoretically, never be used.  However, for the sake of forward-compatibility, it exists and is
-     * tested.
-     *
-     * It is extracted into its own private method to allow for unit testing.
-     *
-     * @template T
-     * @param iterable<T> $iterable
-     * @param bool $preserve_keys
-     * @return array<T>
-     */
-    #[Pure]
-    private static function fallbackConvert(iterable $iterable, bool $preserve_keys): array
-    {
-        $result = [];
-        foreach ($iterable as $key => $value) {
-            if ($preserve_keys) {
-                $result[$key] = $value;
-                continue;
-            }
-            $result[] = $value;
-        }
-        return $result;
+        return iterator_to_array($iterable, $preserve_keys);
     }
 }

tests/IterableToArrayTest.php

@@ -43,42 +43,4 @@ public function testConvertWithIteratorDoesNotPreserveKeys()
         $this->assertEquals(['b'], $result);
         $this->assertNotEquals(['a' => 'b'], $result);
     }
-
-    /**
-     * @dataProvider fallbackDataProvider
-     */
-    public function testFallback(iterable $iterator, bool $preserve_keys, array $expectedResult)
-    {
-        $reflectionMethod = new ReflectionMethod(IterableToArray::class, 'fallbackConvert');
-        $reflectionMethod->setAccessible(true);
-        $result = $reflectionMethod->invoke(null, $iterator, $preserve_keys);
-
-        $this->assertEquals($expectedResult, $result);
-    }
-
-    public function fallbackDataProvider(): array
-    {
-        return [
-            'Array w/ keys' => [
-                ['a' => 'b', 'b' => 'c'],
-                true,
-                ['a' => 'b', 'b' => 'c'],
-            ],
-            'Array w/out keys' => [
-                ['a' => 'b', 'b' => 'c'],
-                false,
-                ['b', 'c'],
-            ],
-            'Iterable w/ keys' => [
-                new ArrayIterator(['a' => 'b', 'b' => 'c']),
-                true,
-                ['a' => 'b', 'b' => 'c'],
-            ],
-            'Iterable w/out keys' => [
-                new ArrayIterator(['a' => 'b', 'b' => 'c']),
-                false,
-                ['b', 'c'],
-            ],
-        ];
-    }
 }