Merge pull request #70 from dbould/increase-test-coverage

Increase test coverage
PHPJasper · Oct 9, 2017 · d951d44 · d951d44
2 parents 3a02a59 + 4ef11fa
commit d951d44
Show file tree

Hide file tree

Showing 3 changed files with 437 additions and 4 deletions.
diff --git a/phpunit.xml b/phpunit.xml
@@ -12,8 +12,7 @@
 
     <filter>
         <whitelist>
-            <directory>src/PHPJasper</directory>
-            <directory>src/JasperStarter</directory>
+            <directory suffix=".php">./src</directory>
         </whitelist>
     </filter>
 

diff --git a/tests/PHPJasper/PHPJasperTest.php b/tests/PHPJasper/PHPJasperTest.php
@@ -103,6 +103,27 @@ public function testExecuteWithCompile()
         $jasper = new PHPJasper();
         $jasper->compile('hello_world.jrxml')->execute();
     }
+
+    public function testExecute()
+    {
+        $jasper = new PHPJasper();
+        $actual = $jasper->compile(__DIR__ . '/test.jrxml')->execute();
+
+        $this->assertInternalType('array', $actual);
+    }
+
+    public function testResourceDirectoryException()
+    {
+        $this->expectException(\PHPJasper\Exception\InvalidResourceDirectory::class);
+
+        $jasper = new PHPJasper();
+        $jasperReflection = new \ReflectionClass(get_class($jasper));
+        $property = $jasperReflection->getProperty('pathExecutable');
+        $property->setAccessible(true);
+        $property->setValue($jasper,'');
+
+        $jasper->compile(__DIR__ . '/test.jrxml')->execute();
+    }
 
     public function testListParametersWithWrongInput()
     {
@@ -129,11 +150,10 @@ public function testProcessWithWrongFormat()
             'format' => 'mp3'
         ]);
     }
-    
+
     public function testProcess()
     {
         $jasper = new PHPJasper();
         $this->assertInstanceOf(PHPJasper::class, $jasper->process('hello_world.jrxml', ""));
     }
-
 }