remove dependency in dev to webforge/collections, because its really just used in one place

Author: pscheit

composer.json

@@ -15,9 +15,9 @@
   "require-dev": {
     "mikey179/vfsstream": "^1.6@stable",
     "mockery/mockery": "@stable",
-    "webforge/collections": "^1.0@dev",
     "friendsofphp/php-cs-fixer": "^3",
-    "phpunit/phpunit": "^10.5"
+    "phpunit/phpunit": "^10.5",
+    "doctrine/collections": "^2.3"
   },
   "autoload": {
     "psr-0": {"Webforge\\Common": ["src/php/", "tests/php/"]}
@@ -30,7 +30,7 @@
   },
   "extra": {
     "branch-alias": {
-      "dev-master": "2.0.x-dev"
+      "dev-master": "3.0.x-dev"
     }
   }
 }

composer.lock

@@ -3,8 +3,8 @@
 namespace Webforge\Common;
 
 use ArrayIterator;
+use Doctrine\Common\Collections\ArrayCollection;
 use stdClass;
-use Webforge\Collections\TraversableCollection;
 
 class UtilTest extends \PHPUnit\Framework\TestCase
 {
@@ -64,17 +64,12 @@ public function testCastArray($item, $expected): void
         );
     }
 
-    public static function provideCastArray()
+    public static function provideCastArray(): array
     {
         $tests = [];
-
-        $test = function () use (&$tests): void {
-            $tests[] = func_get_args();
-        };
-
-        $test($iterator = new ArrayIterator([1, 2, 3]), [1, 2, 3]);
-        $test([1, 2, 3], [1, 2, 3]);
-        $test(new TraversableCollection(['0' => 'nil', '1' => 'one']), ['0' => 'nil', '1' => 'one']);
+        $tests[] = [$iterator = new ArrayIterator([1, 2, 3]), [1, 2, 3]];
+        $tests[] = [[1, 2, 3], [1, 2, 3]];
+        $tests[] = [new ArrayCollection(['0' => 'nil', '1' => 'one']), ['0' => 'nil', '1' => 'one']];
 
         return $tests;
     }