Improve uniqeu array functionality

Author: canvural

src/SchemaFaker/ArrayFaker.php

@@ -6,9 +6,9 @@
 
 use cebe\openapi\spec\Schema;
 use Faker\Provider\Base;
-
 use function array_unique;
-use function array_values;
+use function count;
+use function dump;
 
 /**
  * @internal
@@ -30,10 +30,20 @@ public static function generate(Schema $schema): array
 
         for ($i = 0; $i < $itemSize; $i++) {
             $fakeData[] = $itemSchema->generate();
-        }
 
-        if ($schema->uniqueItems === true) {
-            $fakeData = array_values(array_unique($fakeData));
+            if ($schema->uniqueItems !== true) {
+                continue;
+            }
+
+            $uniqueData = array_unique($fakeData);
+
+            if (count($uniqueData) >= count($fakeData)) {
+                continue;
+            }
+
+            $i -= count($fakeData) - count($uniqueData);
+
+            $fakeData = $uniqueData;
         }
 
         return $fakeData;

tests/Unit/SchemaFaker/ArrayFakerTest.php

@@ -9,8 +9,13 @@
 use Vural\OpenAPIFaker\Tests\Unit\UnitTestCase;
 
 use function array_unique;
+use function count;
+use function mt_srand;
+use function random_int;
 use function Safe\sort;
 
+use const MT_RAND_PHP;
+
 /**
  * @uses \Vural\OpenAPIFaker\SchemaFaker\SchemaFaker
  * @uses \Vural\OpenAPIFaker\SchemaFaker\StringFaker
@@ -127,23 +132,26 @@ function it_handles_nested_arrays()
     /** @test */
     function it_can_generate_unique_elements()
     {
+        mt_srand(227, MT_RAND_PHP);
+
         $fakeData = ArrayFaker::generate(SchemaFactory::fromJson(
             <<< JSON
 {
   "type": "array",
   "items": {
     "type": "integer",
     "minimum": 1,
-    "maximum": 2
+    "maximum": 5
   },
-  "minItems": 1,
-  "maxItems": 2,
+  "minItems": 5,
+  "maxItems": 5,
   "uniqueItems": true
 }
 JSON
         ));
 
         self::assertIsArray($fakeData);
+        self::assertCount(5, $fakeData);
         self::assertSame($fakeData, array_unique($fakeData));
     }