Remove tests directory from code coverage. Add test for bad property. Add test for bad class.

Author: ByScripts

phpunit.xml.dist

@@ -17,7 +17,7 @@
     </testsuites>
     <filter>
         <whitelist processUncoveredFilesFromWhitelist="true">
-            <directory suffix="Test.php">tests</directory>
+            <directory>src</directory>
         </whitelist>
     </filter>
 </phpunit>

tests/EntityTest.php

@@ -2,6 +2,7 @@
 
 namespace Byscripts\StaticEntity\Tests;
 
+use Byscripts\StaticEntity\Tests\Fixtures\BadProperty;
 use Byscripts\StaticEntity\Tests\Fixtures\Civility;
 use Byscripts\StaticEntity\Tests\Fixtures\WebBrowser;
 use PHPUnit\Framework\TestCase;
@@ -131,4 +132,13 @@ public function testFindIds()
 
         $this->assertEquals([Civility::MR, Civility::MRS], $ids);
     }
+
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage has no property
+     */
+    public function testBadProperty()
+    {
+        BadProperty::getAll();
+    }
 }

tests/Fixtures/BadClass.php

@@ -0,0 +1,21 @@
+<?php
+
+namespace Byscripts\StaticEntity\Tests\Fixtures;
+
+class BadClass
+{
+    private $foo;
+
+    static function getDataSet(): array
+    {
+        return [
+            1 => [
+                'foo' => 'bar',
+            ],
+            2 => [
+                'foo' => 'bar',
+                'bar' => 'baz',
+            ],
+        ];
+    }
+}

tests/Fixtures/BadProperty.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Byscripts\StaticEntity\Tests\Fixtures;
+
+use Byscripts\StaticEntity\AbstractStaticEntity;
+
+class BadProperty extends AbstractStaticEntity
+{
+    private $foo;
+
+    static function getDataSet(): array
+    {
+        return [
+            1 => [
+                'foo' => 'bar',
+            ],
+            2 => [
+                'foo' => 'bar',
+                'bar' => 'baz',
+            ],
+        ];
+    }
+}

tests/ProviderTest.php

@@ -3,6 +3,8 @@
 namespace Byscripts\StaticEntity\Tests;
 
 use Byscripts\StaticEntity\Provider;
+use Byscripts\StaticEntity\Tests\Fixtures\BadClass;
+use Byscripts\StaticEntity\Tests\Fixtures\BadProperty;
 use Byscripts\StaticEntity\Tests\Fixtures\Civility;
 use Byscripts\StaticEntity\Tests\Fixtures\WebBrowser;
 use PHPUnit\Framework\TestCase;
@@ -135,4 +137,22 @@ public function testFindIds()
 
         $this->assertEquals([Civility::MR, Civility::MRS], $ids);
     }
+
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage has no property
+     */
+    public function testBadProperty()
+    {
+        Provider::getAll(BadProperty::class);
+    }
+
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage must implements
+     */
+    public function testBadClass()
+    {
+        Provider::getAll(BadClass::class);
+    }
 }