Merge pull request #29 from daniellienert/feature/extend-interface

!!! FEATURE: Extend the QueueInterface with countReserved and countFailed

Author: daniellienert

Classes/Command/JobCommandController.php

@@ -18,6 +18,7 @@
 use Neos\Cache\Frontend\VariableFrontend;
 use Neos\Flow\Annotations as Flow;
 use Neos\Flow\Cli\CommandController;
+use Neos\Flow\Mvc\Exception\StopActionException;
 
 /**
  * Job command controller
@@ -62,6 +63,7 @@ class JobCommandController extends CommandController
      * @param int $limit If set, only the given amount of jobs are processed (successful or not) before the script exits
      * @param bool $verbose Output debugging information
      * @return void
+     * @throws StopActionException
      */
     public function workCommand($queue, $exitAfter = null, $limit = null, $verbose = false)
     {
@@ -123,11 +125,12 @@ public function workCommand($queue, $exitAfter = null, $limit = null, $verbose =
      * @param string $queue The name of the queue
      * @param integer $limit Number of jobs to list (some queues only support a limit of 1)
      * @return void
+     * @throws JobQueueException
      */
     public function listCommand($queue, $limit = 1)
     {
         $jobs = $this->jobManager->peek($queue, $limit);
-        $totalCount = $this->queueManager->getQueue($queue)->count();
+        $totalCount = $this->queueManager->getQueue($queue)->countReady();
         foreach ($jobs as $job) {
             $this->outputLine('<b>%s</b>', [$job->getLabel()]);
         }

Classes/Command/QueueCommandController.php

@@ -11,9 +11,11 @@
  * source code.
  */
 
+use Flowpack\JobQueue\Common\Exception;
 use Flowpack\JobQueue\Common\Queue\QueueManager;
 use Neos\Flow\Annotations as Flow;
 use Neos\Flow\Cli\CommandController;
+use Neos\Flow\Mvc\Exception\StopActionException;
 use Neos\Utility\TypeHandling;
 
 /**
@@ -40,14 +42,15 @@ class QueueCommandController extends CommandController
      * Displays all configured queues, their type and the number of messages that are ready to be processed.
      *
      * @return void
+     * @throws Exception
      */
     public function listCommand()
     {
         $rows = [];
         foreach ($this->queueConfigurations as $queueName => $queueConfiguration) {
             $queue = $this->queueManager->getQueue($queueName);
             try {
-                $numberOfMessages = $queue->count();
+                $numberOfMessages = $queue->countReady();
             } catch (\Exception $e) {
                 $numberOfMessages = '-';
             }
@@ -63,6 +66,7 @@ public function listCommand()
      *
      * @param string $queue Name of the queue to describe (e.g. "some-queue")
      * @return void
+     * @throws Exception
      */
     public function describeCommand($queue)
     {
@@ -83,6 +87,8 @@ public function describeCommand($queue)
      *
      * @param string $queue Name of the queue to initialize (e.g. "some-queue")
      * @return void
+     * @throws Exception
+     * @throws StopActionException
      */
     public function setupCommand($queue)
     {
@@ -106,6 +112,8 @@ public function setupCommand($queue)
      * @param string $queue Name of the queue to flush (e.g. "some-queue")
      * @param bool $force This flag is required in order to avoid accidental flushes
      * @return void
+     * @throws Exception
+     * @throws StopActionException
      */
     public function flushCommand($queue, $force = false)
     {
@@ -134,6 +142,7 @@ public function flushCommand($queue, $force = false)
      * @param string $payload Arbitrary payload, for example a serialized instance of a class implementing JobInterface
      * @param string $options JSON encoded, for example '{"some-option": "some-value"}'
      * @return void
+     * @throws Exception
      */
     public function submitCommand($queue, $payload, $options = null)
     {

Classes/Queue/FakeQueue.php

@@ -146,7 +146,23 @@ public function peek($limit = 1)
     /**
      * @inheritdoc
      */
-    public function count()
+    public function countReady(): int
+    {
+        return 0;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function countReserved(): int
+    {
+        return 0;
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function countFailed(): int
     {
         return 0;
     }

Classes/Queue/QueueInterface.php

@@ -102,13 +102,25 @@ public function finish($messageId);
     public function peek($limit = 1);
 
     /**
-     * Count ready messages in the queue
+     * Get a count of ready messages currently in the queue.
      *
-     * Get a count of messages currently in the queue.
+     * @return int The number of ready messages in the queue
+     */
+    public function countReady(): int;
+
+    /**
+     * Get a count of reserved messages currently in the queue.
+     *
+     * @return int The number of reserved messages in the queue
+     */
+    public function countReserved(): int;
+
+    /**
+     * Get a count of failed messages currently in the queue.
      *
-     * @return integer The number of messages in the queue
+     * @return int The number of failed messages in the queue
      */
-    public function count();
+    public function countFailed(): int;
 
     /**
      * Removes all messages from this queue
@@ -119,4 +131,4 @@ public function count();
      * @return void
      */
     public function flush();
-}
+}

Tests/Functional/AbstractQueueTest.php

@@ -144,10 +144,10 @@ public function releasePutsMessageBackToQueue()
         $messageId = $this->queue->submit('A message');
 
         $this->queue->waitAndReserve(1);
-        $this->assertSame(0, $this->queue->count());
+        $this->assertSame(0, $this->queue->countReady());
 
         $this->queue->release($messageId);
-        $this->assertSame(1, $this->queue->count());
+        $this->assertSame(1, $this->queue->countReady());
     }
 
     /**
@@ -167,6 +167,8 @@ public function releaseIncreasesNumberOfReleases()
         $this->queue->release($messageId);
         $message = $this->queue->waitAndReserve(1);
         $this->assertSame(2, $message->getNumberOfReleases());
+
+        $this->queue->abort($messageId);
     }
 
     /**
@@ -179,26 +181,70 @@ public function abortRemovesMessageFromActiveQueue()
         $this->queue->waitAndReserve(1);
 
         $this->queue->abort($messageId);
-        $this->assertSame(0, $this->queue->count());
+        $this->assertSame(0, $this->queue->countReady());
         $this->assertNull($this->queue->waitAndTake(1));
     }
 
     /**
      * @test
      */
-    public function countReturnsZeroByDefault()
+    public function countReadyReturnsZeroByDefault()
     {
-        $this->assertSame(0, $this->queue->count());
+        $this->assertSame(0, $this->queue->countReady());
     }
 
     /**
      * @test
      */
-    public function countReturnsNumberOfReadyJobs()
+    public function countReadyReturnsNumberOfReadyJobs()
     {
         $this->queue->submit('First message');
         $this->queue->submit('Second message');
 
-        $this->assertSame(2, $this->queue->count());
+        $this->assertSame(2, $this->queue->countReady());
+    }
+
+    /**
+     * @test
+     */
+    public function countFailedReturnsZeroByDefault()
+    {
+        $this->assertSame(0, $this->queue->countFailed());
+    }
+
+    /**
+     * @test
+     */
+    public function countFailedReturnsNumberOfFailedMessages()
+    {
+        $messageId = $this->queue->submit('A message');
+
+        $this->queue->waitAndReserve(1);
+        $this->assertSame(0, $this->queue->countFailed());
+
+        $this->queue->abort($messageId);
+        $this->assertSame(1, $this->queue->countFailed());
+    }
+
+    /**
+     * @test
+     */
+    public function countReservedReturnsZeroByDefault()
+    {
+        $this->assertSame(0, $this->queue->countReserved());
+    }
+
+    /**
+     * @test
+     */
+    public function countReservedReturnsNumberOfReservedMessages()
+    {
+        $messageId = $this->queue->submit('A message');
+
+        $this->queue->waitAndReserve(1);
+        $this->assertSame(1, $this->queue->countReserved());
+
+        $this->queue->abort($messageId);
+        $this->assertSame(0, $this->queue->countReserved());
     }
-}
+}

Tests/Unit/Fixtures/TestQueue.php

@@ -30,13 +30,18 @@ class TestQueue implements QueueInterface
     /**
      * @var string[]
      */
-    protected $processingMessages = [];
+    protected $reservedMessages = [];
 
     /**
      * @var string[]
      */
     protected $failedMessages = [];
 
+    /**
+     * @var string[]
+     */
+    protected $processingMessages = [];
+
     /**
      * @var int[]
      */
@@ -205,11 +210,27 @@ public function peek($limit = 1)
     /**
      * @inheritdoc
      */
-    public function count()
+    public function countReady():int
     {
         return count($this->readyMessages);
     }
 
+    /**
+     * @inheritdoc
+     */
+    public function countReserved(): int
+    {
+        return count($this->reservedMessages);
+    }
+
+    /**
+     * @inheritdoc
+     */
+    public function countFailed(): int
+    {
+        return count($this->failedMessages);
+    }
+
     /**
      * @inheritdoc
      */