Merge pull request #15 from box/add_test_for_skipping_empty_rows

Add test for skipping empty rows
box · Apr 4, 2015 · 538f610 · 538f610
2 parents 66e5792 + 93cdd39
commit 538f610
Showing 1 changed file with 16 additions and 0 deletions.
diff --git a/tests/Spout/Writer/CSVTest.php b/tests/Spout/Writer/CSVTest.php
@@ -76,6 +76,22 @@ public function testWriteShouldSupportNullValues()
         $this->assertEquals('csv--11,,csv--13', $writtenContent, 'The null values should be replaced by empty values');
     }
 
+    /**
+     * @return void
+     */
+    public function testWriteShouldSkipEmptyRows()
+    {
+        $allRows = [
+            ['csv--11', 'csv--12'],
+            [],
+            ['csv--31', 'csv--32'],
+        ];
+        $writtenContent = $this->writeToCsvFileAndReturnWrittenContent($allRows, 'csv_with_empty_rows.csv');
+        $writtenContent = $this->trimWrittenContent($writtenContent);
+
+        $this->assertEquals("csv--11,csv--12\ncsv--31,csv--32", $writtenContent, 'Empty rows should be skipped');
+    }
+
     /**
      * @return void
      */