Update based on PR feedback

Author: tdashton

README.md

@@ -33,12 +33,6 @@ composer install
 composer test-coverage
 ```
 
-To run tests in a docker container (without coverage):
-
-```bash
-docker run -it --rm --workdir /app --volume .:/app php:8.1 php vendor/bin/phpunit tests/
-```
-
 ## Usage
 
 The library defines interfaces to deal with collections and also boilerplate code with default implementations.

docs/collection-interfaces.md

@@ -51,7 +51,7 @@ $collection->add($item1)->add($item2);
 public function chunk(int $size): array
 ```
 
-This method [mirrors the behavior of `array_chunk`](https://www.php.net/manual/function.array-chunk.php) and returns a zero indexed numeric array of the current Collection based on the chunk size provided.
+This method [mirrors the behavior of `array_chunk`](https://www.php.net/manual/function.array-chunk.php) and returns a zero indexed numeric array of the current collection based on the chunk size provided.
 
 ### diff
 

docs/collection-trait.md

@@ -97,7 +97,7 @@ Internally it is call the `ArrayIterator::append` and returning the instance to
 
 ## chunk
 
-This method [mirrors the behavior of `array_chunk`](https://www.php.net/manual/function.array-chunk.php) and returns a zero indexed numeric array of the current Collection based on the chunk size provided.
+Internally this method chunks [a copy](https://www.php.net/manual/arrayobject.getarraycopy.php) of the collection with the [`array_chunk` php function](https://www.php.net/manual/function.array-chunk.php), returning a zero indexed array of collections of the same type as the initial one.
 
 ## diff
 
@@ -123,7 +123,7 @@ Internally, this method will iterate through each item of a collection, optional
 
 ## eachChunk
 
-This method chunks the collection and executes the given anonymous function with each chunk.
+Internally, this method calls the [chunk](#chunk) function and then executes the passed anonymous function with each chunk.
 
 ## empty
 

src/CollectionTrait.php

@@ -43,7 +43,7 @@ public function chunk(int $size): array
 
         return array_map(
             static fn(array $chunk) => self::fromIterable($chunk),
-            array_chunk($this->toArray(), $size)
+            array_chunk($this->getArrayCopy(), $size)
         );
     }
 

tests/CollectionTest.php

@@ -486,11 +486,11 @@ public function testReverse(): void
     }
 
     #[DataProvider('chunkDataProvider')]
-    public function testChunk(Collection $collectionToChunk, int $chunkSize, array $expectedChunks): void
+    public function testChunk(Collection $collection, int $chunkSize, array $expectedChunks): void
     {
         self::assertEquals(
             $expectedChunks,
-            $collectionToChunk->chunk($chunkSize)
+            $collection->chunk($chunkSize)
         );
     }
 
@@ -603,14 +603,16 @@ public static function eachDataProvider(): array
     }
 
     #[DataProvider('chunkEachDataProvider')]
-    public function testChunkEach(Collection $collectionToChunk, int $chunkSize, array $expectedChunks): void
+    public function testChunkEach(Collection $collection, int $chunkSize, array $expectedChunks): void
     {
-        $expectedLambda = function(CollectionStub $collection) use ($expectedChunks): void {
-            static $i = 0;
-            self::assertEquals($collection, $expectedChunks[$i++]);
-        };
+        $i = 0;
 
-        $collectionToChunk->eachChunk($chunkSize, $expectedLambda);
+        $collection->eachChunk(
+            $chunkSize,
+            function(CollectionStub $collection) use (&$i, $expectedChunks): void {
+                self::assertEquals($collection, $expectedChunks[$i++]);
+            }
+        );
     }
 
     public static function chunkEachDataProvider(): array