Get code coverage to 100%

subjective-php · Jun 9, 2018 · c2ffdb8 · c2ffdb8
1 parent ca822d2
commit c2ffdb8
Show file tree

Hide file tree

Showing 3 changed files with 85 additions and 0 deletions.
diff --git a/tests/DeriveHeaderStrategyTest.php b/tests/DeriveHeaderStrategyTest.php
@@ -50,6 +50,30 @@ public function rowNotIsHeaderRow()
         $this->assertFalse($strategy->isHeaderRow($fileObject->fgetcsv()));
     }
 
+    /**
+     * @test
+     * @covers ::createDataRow
+     */
+    public function createDataRow()
+    {
+        $fileObject = $this->getFileObject();
+        $strategy = new DeriveHeaderStrategy();
+        $strategy->getHeaders($fileObject);
+        $fileObject->fgetcsv();//skip header line
+        $this->assertSame(
+            [
+                'id' => 'bk101',
+                'author' => 'Gambardella, Matthew',
+                'title' => 'XML Developer\'s Guide',
+                'genre' => 'Computer',
+                'price' => '44.95',
+                'publish_date' => '2000-10-01',
+                'description' => 'An in-depth look at creating applications with XML.',
+            ],
+            $strategy->createDataRow($fileObject->fgetcsv())
+        );
+    }
+
     private function getFileObject() : SplFileObject
     {
         $fileObject = new SplFileObject(__DIR__ . '/_files/pipe_delimited.txt');

diff --git a/tests/NoHeaderStrategyTest.php b/tests/NoHeaderStrategyTest.php
@@ -40,4 +40,35 @@ public function isHeaderRowAlwaysReturnsFalse()
         $this->assertFalse($strategy->isHeaderRow($fileObject->fgetcsv()));
         $this->assertFalse($strategy->isHeaderRow($fileObject->fgetcsv()));
     }
+
+    /**
+     * @test
+     * @covers ::createDataRow
+     */
+    public function createDataRow()
+    {
+        $row = [
+            'bk101',
+            'Gambardella, Matthew',
+            'XML Developer\'s Guide',
+            'Computer',
+            '44.95',
+            '2000-10-01',
+            'An in-depth look at creating applications with XML.',
+        ];
+        $strategy = new NoHeaderStrategy();
+        $this->assertSame(
+            [
+                'bk101',
+                'Gambardella, Matthew',
+                'XML Developer\'s Guide',
+                'Computer',
+                '44.95',
+                '2000-10-01',
+                'An in-depth look at creating applications with XML.',
+            ],
+            $strategy->createDataRow($row)
+        );
+    }
+
 }
diff --git a/tests/ProvidedHeaderStrategyTest.php b/tests/ProvidedHeaderStrategyTest.php
@@ -47,6 +47,36 @@ public function rowIsNotHeaderRow()
         $this->assertFalse($strategy->isHeaderRow($fileObject->fgetcsv()));
     }
 
+    /**
+     * @test
+     * @covers ::createDataRow
+     */
+    public function createDataRow()
+    {
+        $row = [
+            'bk101',
+            'Gambardella, Matthew',
+            'XML Developer\'s Guide',
+            'Computer',
+            '44.95',
+            '2000-10-01',
+            'An in-depth look at creating applications with XML.',
+        ];
+        $strategy = $this->getStrategy();
+        $this->assertSame(
+            [
+                'id' => 'bk101',
+                'author' => 'Gambardella, Matthew',
+                'title' => 'XML Developer\'s Guide',
+                'genre' => 'Computer',
+                'price' => '44.95',
+                'publish_date' => '2000-10-01',
+                'description' => 'An in-depth look at creating applications with XML.',
+            ],
+            $strategy->createDataRow($row)
+        );
+    }
+
     private function getFileObject() : SplFileObject
     {
         $fileObject = new SplFileObject(__DIR__ . '/_files/basic.csv');