Allow strings|int in collection constructor (#25)

Author: silasjoisten

src/Domain/Value/Field/FieldCollection.php

@@ -26,12 +26,16 @@ final class FieldCollection implements \Countable, \IteratorAggregate
     private array $items = [];
 
     /**
-     * @param list<Field> $items
+     * @param list<Field|string> $items
      */
     public function __construct(
         array $items = [],
     ) {
         foreach ($items as $item) {
+            if (\is_string($item)) {
+                $item = new Field($item);
+            }
+
             $this->add($item);
         }
     }

src/Domain/Value/IdCollection.php

@@ -26,12 +26,16 @@ final class IdCollection implements \Countable, \IteratorAggregate
     private array $items = [];
 
     /**
-     * @param list<Id> $items
+     * @param list<Id|int> $items
      */
     public function __construct(
         array $items = [],
     ) {
         foreach ($items as $item) {
+            if (\is_int($item)) {
+                $item = new Id($item);
+            }
+
             $this->add($item);
         }
     }

src/Domain/Value/Resolver/RelationCollection.php

@@ -26,12 +26,16 @@ final class RelationCollection implements \Countable, \IteratorAggregate
     private array $items = [];
 
     /**
-     * @param list<Relation> $items
+     * @param list<Relation|string> $items
      */
     public function __construct(
         array $items = [],
     ) {
         foreach ($items as $item) {
+            if (\is_string($item)) {
+                $item = new Relation($item);
+            }
+
             $this->add($item);
         }
     }

src/Domain/Value/Tag/TagCollection.php

@@ -26,12 +26,16 @@ final class TagCollection implements \Countable, \IteratorAggregate
     private array $items = [];
 
     /**
-     * @param list<Tag> $items
+     * @param list<string|Tag> $items
      */
     public function __construct(
         array $items = [],
     ) {
         foreach ($items as $item) {
+            if (\is_string($item)) {
+                $item = new Tag($item);
+            }
+
             $this->add($item);
         }
     }

tests/Unit/Domain/Value/Field/FieldCollectionTest.php

@@ -25,6 +25,18 @@ final class FieldCollectionTest extends TestCase
 {
     use FakerTrait;
 
+    /**
+     * @test
+     */
+    public function constructWithString(): void
+    {
+        $faker = self::faker();
+        $collection = new FieldCollection([$faker->word(), $faker->word()]);
+
+        self::assertCount(2, $collection);
+        self::assertContainsOnlyInstancesOf(Field::class, $collection);
+    }
+
     /**
      * @test
      */

tests/Unit/Domain/Value/IdCollectionTest.php

@@ -25,6 +25,18 @@ final class IdCollectionTest extends TestCase
 {
     use FakerTrait;
 
+    /**
+     * @test
+     */
+    public function constructWithString(): void
+    {
+        $faker = self::faker();
+        $collection = new IdCollection([$faker->numberBetween(1), $faker->numberBetween(1)]);
+
+        self::assertCount(2, $collection);
+        self::assertContainsOnlyInstancesOf(Id::class, $collection);
+    }
+
     /**
      * @test
      */

tests/Unit/Domain/Value/Resolver/RelationCollectionTest.php

@@ -25,6 +25,18 @@ final class RelationCollectionTest extends TestCase
 {
     use FakerTrait;
 
+    /**
+     * @test
+     */
+    public function constructWithString(): void
+    {
+        $faker = self::faker();
+        $collection = new RelationCollection([$faker->relation(), $faker->relation()]);
+
+        self::assertCount(2, $collection);
+        self::assertContainsOnlyInstancesOf(Relation::class, $collection);
+    }
+
     /**
      * @test
      */

tests/Unit/Domain/Value/Tag/TagCollectionTest.php

@@ -25,6 +25,18 @@ final class TagCollectionTest extends TestCase
 {
     use FakerTrait;
 
+    /**
+     * @test
+     */
+    public function constructWithString(): void
+    {
+        $faker = self::faker();
+        $collection = new TagCollection([$faker->word(), $faker->word()]);
+
+        self::assertCount(2, $collection);
+        self::assertContainsOnlyInstancesOf(Tag::class, $collection);
+    }
+
     /**
      * @test
      */