Add remove method

src/AbstractCollection.php

@@ -14,9 +14,24 @@ class AbstractCollection implements CollectionInterface
 
     public function add(mixed $item): void
     {
+        if (null === $item) {
+            return;
+        }
+
         $this->items[] = $item;
     }
 
+    public function remove(mixed $item): void
+    {
+        $key = array_search($item, $this->items, true);
+
+        if ($key) {
+            unset($this->items[$key]);
+        }
+
+        $this->items = array_values($this->items);
+    }
+
     public function toArray(): array
     {
         return $this->items;

src/Interface/CollectionInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 declare(strict_types=1);
 
 namespace Loom\Utility\Collection\Interface;
@@ -6,5 +7,6 @@ namespace Loom\Utility\Collection\Interface;
 interface CollectionInterface
 {
     public function add(mixed $item): void;
+    public function remove(mixed $item): void;
     public function toArray(): array;
 }

tests/CollectionTest.php

@@ -19,20 +19,54 @@ class CollectionTest extends TestCase
         $this->assertEquals($expectedResult, $collection->toArray());
     }
 
+    #[DataProvider('removeDataProvider')]
+    public function testRemove(array $startingArray, mixed $itemToRemove, array $expectedResult): void
+    {
+        $collection = new Collection($startingArray);
+        $collection->remove($itemToRemove);
+
+        $this->assertEquals($expectedResult, $collection->toArray());
+    }
+
     public static function addDataProvider(): array
     {
         return [
             [
-                'startingItems' => ['apple', 'banana', 'cherry'],
+                'startingItems' => ['A', 'B', 'C'],
-                'itemToAdd' => 'grape',
+                'itemToAdd' => 'D',
-                'expectedResult' => ['apple', 'banana', 'cherry', 'grape'],
+                'expectedResult' => ['A', 'B', 'C', 'D'],
             ],
             [
                 'startingItems' => [],
                 'itemToAdd' => 1,
                 'expectedResult' => [1],
             ],
+            [
+                'startingItems' => [],
+                'itemToAdd' => null,
+                'expectedResult' => [],
+            ],
+        ];
+    }
 
+    public static function removeDataProvider(): array
+    {
+        return [
+            [
+                'startingArray' => ['A', 'B', 'C'],
+                'itemToRemove' => 'B',
+                'expectedResult' => ['A', 'C'],
+            ],
+            [
+                'startingArray' => ['A', 'B', 'C'],
+                'itemToRemove' => 'orange',
+                'expectedResult' => ['A', 'B', 'C'],
+            ],
+            [
+                'startingArray' => [],
+                'itemToRemove' => 'A',
+                'expectedResult' => [],
+            ],
         ];
     }
 }