Merge pull request #24 from keboola/odin-escapedby

tests: add tests for escaped by parameter, fixes #23

Author: Halama

tests/CsvFileTest.php

@@ -101,6 +101,39 @@ public function testParse()
         self::assertEquals($expected, $rows);
     }
 
+    public function testParseEscapedBy()
+    {
+        $csvFile = new CsvFile(__DIR__ . '/data/escapingEscapedBy.csv', ",", '"', '\\');
+
+        $expected = [
+            [
+                'col1', 'col2',
+            ],
+            [
+                'line without enclosure', 'second column',
+            ],
+            [
+                'enclosure \" in column', 'hello \\\\',
+            ],
+            [
+                'line with enclosure', 'second column',
+            ],
+            [
+                'column with enclosure \", and comma inside text', 'second column enclosure in text \"',
+            ],
+            [
+                "columns with\nnew line", "columns with\ttab",
+            ],
+            [
+                "Columns with WINDOWS\r\nnew line", "second",
+            ],
+            [
+                'column with \n \t \\\\', 'second col',
+            ],
+        ];
+
+        self::assertEquals($expected, iterator_to_array($csvFile));
+    }
 
     public function testEmptyHeader()
     {

tests/data/escapingEscapedBy.csv

@@ -0,0 +1,10 @@
+col1,col2
+line without enclosure,second column
+"enclosure \" in column","hello \\"
+"line with enclosure","second column"
+"column with enclosure \", and comma inside text","second column enclosure in text \""
+"columns with
+new line","columns with	tab"
+"Columns with WINDOWS
+new line", "second"
+"column with \n \t \\","second col"