Merge pull request #59 from saifulferoz/feature/increase-test-coverage

test: increase test coverage

.scrutinizer.yml

@@ -1,4 +1,5 @@
 build:
+    image: default-bionic
     nodes:
         analysis:
             tests:

Tests/Subscriber/DoctrineSubscriberTest.php

@@ -16,6 +16,7 @@
 use PHPUnit\Framework\MockObject\MockObject;
 use PHPUnit\Framework\TestCase;
 use Xiidea\EasyAuditBundle\Subscriber\DoctrineSubscriber;
+use Xiidea\EasyAuditBundle\Tests\Fixtures\ORM\DummyEntity;
 use Xiidea\EasyAuditBundle\Tests\Fixtures\ORM\Movie;
 use Doctrine\Persistence\ObjectManager;
 
@@ -52,7 +53,7 @@ public function testCreateEventForAttributedEntity()
     public function testCreateEventForEntityNotConfiguredToTrack()
     {
         $subscriber = new DoctrineSubscriber(array());
-        $this->invokeCreatedEventCall($subscriber);
+        $this->invokeCreatedEventCall($subscriber, new DummyEntity());
     }
 
     public function testCreateEventForEntityConfiguredToTrack()
@@ -66,7 +67,6 @@ public function testCreateEventForEntityConfiguredToTrack()
 
     public function testCreateEventForEntityConfiguredToTrackAllEvents()
     {
-
         $subscriber = new DoctrineSubscriber(array('Xiidea\EasyAuditBundle\Tests\Fixtures\ORM\Movie' => array()));
 
         $this->invokeCreatedEventCall($subscriber);
@@ -75,7 +75,7 @@ public function testCreateEventForEntityConfiguredToTrackAllEvents()
     public function testUpdateEventForEntityNotConfiguredToTrack()
     {
         $subscriber = new DoctrineSubscriber(array());
-        $this->invokeUpdatedEventCall($subscriber);
+        $this->invokeUpdatedEventCall($subscriber, new DummyEntity());
     }
 
     public function testRemovedEventForEntityNotConfiguredToTrack()
@@ -95,21 +95,21 @@ public function testRemovedEventForEntityConfiguredToTrackAllEvent()
     /**
      * @param DoctrineSubscriber $subscriber
      */
-    private function invokeCreatedEventCall($subscriber)
+    private function invokeCreatedEventCall($subscriber, $entity = null)
     {
         $subscriber->setDispatcher($this->dispatcher);
-        $subscriber->postPersist(new LifecycleEventArgs(new Movie(), $this->entityManager));
+        $subscriber->postPersist(new LifecycleEventArgs($entity ?? new Movie(), $this->entityManager));
         $this->assertTrue(true);
     }
 
     /**
      * @param DoctrineSubscriber $subscriber
      */
-    private function invokeUpdatedEventCall($subscriber)
+    private function invokeUpdatedEventCall($subscriber, $entity = null)
     {
         $subscriber->setDispatcher($this->dispatcher);
 
-        $subscriber->postUpdate(new LifecycleEventArgs(new Movie(), $this->entityManager));
+        $subscriber->postUpdate(new LifecycleEventArgs($entity ?? new Movie(), $this->entityManager));
         $this->assertTrue(true);
     }