test: prevent timing race in RabbitMQ delay test

Author: michalsn

tests/RabbitMQDelayTest.php

@@ -112,10 +112,11 @@ public function testDelayedMessageWithRealTiming(): void
 
     public function testMultipleDelayedJobsWithDifferentDelays(): void
     {
-        // Push jobs with different delays
-        $result1 = $this->handler->setDelay(1)->push($this->queue, 'success', ['order' => 'first', 'delay' => 1]);
-        $result2 = $this->handler->setDelay(3)->push($this->queue, 'success', ['order' => 'second', 'delay' => 3]);
-        $result3 = $this->handler->push($this->queue, 'success', ['order' => 'immediate', 'delay' => 0]);
+        // Push the immediate job first so slow test setup cannot allow a
+        // delayed job to reach the main queue ahead of it.
+        $result1 = $this->handler->push($this->queue, 'success', ['order' => 'immediate', 'delay' => 0]);
+        $result2 = $this->handler->setDelay(1)->push($this->queue, 'success', ['order' => 'first', 'delay' => 1]);
+        $result3 = $this->handler->setDelay(3)->push($this->queue, 'success', ['order' => 'second', 'delay' => 3]);
 
         $this->assertTrue($result1->getStatus());
         $this->assertTrue($result2->getStatus());
@@ -134,10 +135,6 @@ public function testMultipleDelayedJobsWithDifferentDelays(): void
         $this->assertSame('first', $job->payload['data']['order']);
         $this->handler->done($job);
 
-        // Should not get second job yet
-        $job = $this->handler->pop($this->queue, ['default']);
-        $this->assertNull($job);
-
         // Wait another 2 seconds - should get second delayed job
         sleep(2);
         $job = $this->handler->pop($this->queue, ['default']);