Add unit test for ProjectMetricJob

3 files changed, 8 insertions, 5 deletions

diff --git a/app/Core/Base.php b/app/Core/Base.php
index 6931d93a..41f5d2e0 100644
--- a/app/Core/Base.php
+++ b/app/Core/Base.php
@@ -157,6 +157,7 @@ use Pimple\Container;
  * @property \Kanboard\Job\TaskLinkEventJob                      $taskLinkEventJob
  * @property \Kanboard\Job\ProjectFileEventJob                   $projectFileEventJob
  * @property \Kanboard\Job\NotificationJob                       $notificationJob
+ * @property \Kanboard\Job\ProjectMetricJob                      $projectMetricJob
  * @property \Psr\Log\LoggerInterface                            $logger
  * @property \PicoDb\Database                                    $db
  * @property \Symfony\Component\EventDispatcher\EventDispatcher  $dispatcher
diff --git a/app/ServiceProvider/JobProvider.php b/app/ServiceProvider/JobProvider.php
index 5b42794b..2194b11c 100644
--- a/app/ServiceProvider/JobProvider.php
+++ b/app/ServiceProvider/JobProvider.php
@@ -5,6 +5,7 @@ namespace Kanboard\ServiceProvider;
 use Kanboard\Job\CommentEventJob;
 use Kanboard\Job\NotificationJob;
 use Kanboard\Job\ProjectFileEventJob;
+use Kanboard\Job\ProjectMetricJob;
 use Kanboard\Job\SubtaskEventJob;
 use Kanboard\Job\TaskEventJob;
 use Kanboard\Job\TaskFileEventJob;
@@ -57,6 +58,10 @@ class JobProvider implements ServiceProviderInterface
             return new NotificationJob($c);
         });
 
+        $container['projectMetricJob'] = $container->factory(function ($c) {
+            return new ProjectMetricJob($c);
+        });
+
         return $container;
     }
 }
diff --git a/app/Subscriber/ProjectDailySummarySubscriber.php b/app/Subscriber/ProjectDailySummarySubscriber.php
index 7e3c11c3..eaa9d468 100644
--- a/app/Subscriber/ProjectDailySummarySubscriber.php
+++ b/app/Subscriber/ProjectDailySummarySubscriber.php
@@ -3,7 +3,6 @@
 namespace Kanboard\Subscriber;
 
 use Kanboard\Event\TaskEvent;
-use Kanboard\Job\ProjectMetricJob;
 use Kanboard\Model\TaskModel;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 
@@ -22,9 +21,7 @@ class ProjectDailySummarySubscriber extends BaseSubscriber implements EventSubsc
 
     public function execute(TaskEvent $event)
     {
-        if (isset($event['project_id'])) {
-            $this->logger->debug('Subscriber executed: '.__METHOD__);
-            $this->queueManager->push(ProjectMetricJob::getInstance($this->container)->withParams($event['project_id']));
-        }
+        $this->logger->debug('Subscriber executed: '.__METHOD__);
+        $this->queueManager->push($this->projectMetricJob->withParams($event['task']['project_id']));
     }
 }