test: bad header sanitized

Author: yajra

src/CsvFile.php

@@ -135,8 +135,7 @@ public function getHeaders(?int $length = null): array
         if ($headers === false) {
             return [];
         }
-        $headers = array_map(fn ($header) => str_replace(["\r", "\n"], '', $header), $headers);
 
-        return $headers;
+        return array_map(fn ($header) => str_replace(["\r", "\n"], '', $header), $headers);
     }
 }

tests/Feature/CsvFileTest.php

@@ -55,3 +55,12 @@
         ->and($content)->toContain('3,"Jane, Doe",3'.PHP_EOL)
         ->and($content)->toContain('3,"Jane"" Doe",3'.PHP_EOL);
 });
+
+test('it can sanitize headers', function () {
+    $headers = CsvFile::make(__DIR__.'/../data/bad-header.dat', 'r')->getHeaders();
+    expect($headers)->toBe([
+        'NAME',
+        'EMAIL',
+        'PHONE',
+    ]);
+});

tests/data/bad-header.dat

@@ -0,0 +1,6 @@
+NAME,EMAIL,PHONE
+
+John Doe,[email protected]
+Jane Doe,[email protected]
+"Mr. ""John"" Doe",[email protected]
+"Ms. Doe, Jane", [email protected]