Improve Test suite

Author: nyamsprod

src/Parameters.php

@@ -404,7 +404,11 @@ public function mergePairs(Parameters|iterable ...$others): self
      */
     public function sanitize(): self
     {
-        $this->members = array_map(fn (Item $item): Item => $item->parameters->clear(), $this->members);
+        $this->members = array_map(function (Item $item): Item {
+            $item->parameters->clear();
+            
+            return $item;
+        }, $this->members);
 
         return $this;
     }

src/StructuredFieldTest.php

@@ -16,13 +16,13 @@ abstract class StructuredFieldTest extends TestCase
      * @dataProvider httpWgDataProvider
      * @test
      */
-    public function it_can_pass_http_wg_tests(TestUnit $test): void
+    public function it_can_pass_http_wg_tests(TestRecord $test): void
     {
         if ($test->mustFail) {
             $this->expectException(SyntaxError::class);
         }
 
-        $structuredField = TestFieldType::from($test->type)->newStructuredField(implode(',', $test->raw));
+        $structuredField = TestDataType::from($test->type)->newStructuredField(implode(',', $test->raw));
 
         if (!$test->mustFail) {
             self::assertSame(implode(',', $test->canonical), $structuredField->toHttpValue());
@@ -31,12 +31,12 @@ public function it_can_pass_http_wg_tests(TestUnit $test): void
 
     /**
      * @throws JsonException
-     * @return iterable<string, array<TestUnit>>
+     * @return iterable<string, array<TestRecord>>
      */
     public function httpWgDataProvider(): iterable
     {
         foreach ($this->paths as $path) {
-            foreach (TestSuite::fromPath($path) as $test) {
+            foreach (TestRecordCollection::fromPath($path) as $test) {
                 yield $test->name => [$test];
             }
         }

src/TestFieldType.php renamed to src/TestDataType.php

@@ -4,7 +4,7 @@
 
 namespace Bakame\Http\StructuredFields;
 
-enum TestFieldType: string
+enum TestDataType: string
 {
     case Dictionary = 'dictionary';
     case List = 'list';

src/TestUnit.php renamed to src/TestRecord.php

@@ -4,7 +4,7 @@
 
 namespace Bakame\Http\StructuredFields;
 
-final class TestUnit
+final class TestRecord
 {
     public function __construct(
         public readonly string $name,

src/TestSuite.php renamed to src/TestRecordCollection.php

@@ -16,17 +16,17 @@
 use function stream_get_contents;
 
 /**
- * @implements IteratorAggregate<string, TestUnit>
+ * @implements IteratorAggregate<string, TestRecord>
  */
-final class TestSuite implements IteratorAggregate
+final class TestRecordCollection implements IteratorAggregate
 {
     private function __construct(
-        /** @var array<string, TestUnit> */
+        /** @var array<string, TestRecord> */
         private array $elements = []
     ) {
     }
 
-    public function add(TestUnit $test): void
+    public function add(TestRecord $test): void
     {
         if (isset($this->elements[$test->name])) {
             throw new RuntimeException('Already existing test name `'.$test->name.'`');
@@ -69,14 +69,14 @@ public static function fromPath(string $path, $context = null): self
         $suite = new self();
         foreach ($records as $offset => $record) {
             $record['name'] = basename($path).' #'.($offset + 1).': '.$record['name'];
-            $suite->add(TestUnit::fromDecoded($record));
+            $suite->add(TestRecord::fromDecoded($record));
         }
 
         return $suite;
     }
 
     /**
-     * @return Iterator<string, TestUnit>
+     * @return Iterator<string, TestRecord>
      */
     public function getIterator(): Iterator
     {