Covered extra paths for Exiftool mapper test

Signed-off-by: Tom Van Herreweghe <[email protected]>

Author: Tom Van Herreweghe

lib/PHPExif/Mapper/Exiftool.php

@@ -172,7 +172,11 @@ public function mapRawData(array $data)
                 $key = $this->map[self::GPSLATITUDE];
 
                 $mappedData[$key] = $gpsLocation;
+            } else {
+                unset($mappedData[$this->map[self::GPSLATITUDE]]);
             }
+        } else {
+            unset($mappedData[$this->map[self::GPSLATITUDE]]);
         }
 
         return $mappedData;

tests/PHPExif/Mapper/ExiftoolMapperTest.php

@@ -186,5 +186,54 @@ public function testMapRawDataCorrectlyFormatsNumericGPSData()
         $this->assertCount(1, $result);
         $this->assertEquals($expected, reset($result));
     }
-    
+
+    /**
+     * @group mapper
+     * @covers \PHPExif\Mapper\Exiftool::mapRawData
+     */
+    public function testMapRawDataCorrectlyIgnoresIncorrectGPSData()
+    {
+        $this->mapper->setNumeric(false);
+        $result = $this->mapper->mapRawData(
+            array(
+                'GPSLatitude'     => '40.333452381',
+                'GPSLatitudeRef'  => 'North',
+                'GPSLongitude'    => '20.167314814',
+                'GPSLongitudeRef' => 'West',
+            )
+        );
+
+        $this->assertCount(0, $result);
+    }
+
+    /**
+     * @group mapper
+     * @covers \PHPExif\Mapper\Exiftool::mapRawData
+     */
+    public function testMapRawDataCorrectlyIgnoresIncompleteGPSData()
+    {
+        $result = $this->mapper->mapRawData(
+            array(
+                'GPSLatitude'     => '40.333452381',
+                'GPSLatitudeRef'  => 'North',
+            )
+        );
+
+        $this->assertCount(0, $result);
+    }
+
+    /**
+     * @group mapper
+     * @covers \PHPExif\Mapper\Exiftool::setNumeric
+     */
+    public function testSetNumericInProperty()
+    {
+        $reflProperty = new \ReflectionProperty(get_class($this->mapper), 'numeric');
+        $reflProperty->setAccessible(true);
+
+        $expected = true;
+        $this->mapper->setNumeric($expected);
+
+        $this->assertEquals($expected, $reflProperty->getValue($this->mapper));
+    }
 }