refactor: add Closure return type #1184

spiral · Jan 3, 2025 · a5044a5 · a5044a5
1 parent c2b4575
commit a5044a5
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/tests/JobTest.php b/tests/JobTest.php
@@ -121,7 +121,7 @@ private function getEmail(): Email
     private function expectRenderer(Email $email): void
     {
         $this->renderer->expects('render')->withArgs(
-            function (Message $message) {
+            function (Message $message): bool {
                 $this->assertSame($message->getSubject(), 'test');
                 return true;
             }

diff --git a/tests/JobsQueueTest.php b/tests/JobsQueueTest.php
@@ -33,7 +33,7 @@ public function testQueue(): void
         $mail->setBCC('admin2@google.com');
 
         $queue->expects('push')->withArgs(
-            function ($job, $data, Options $options) use ($mail) {
+            function ($job, $data, Options $options) use ($mail): bool {
                 $this->assertSame(MailQueue::JOB_NAME, $job);
                 $this->assertSame($data, MessageSerializer::pack($mail));
                 $this->assertSame('mailer', $options->getQueue());

diff --git a/tests/QueueTest.php b/tests/QueueTest.php
@@ -43,7 +43,7 @@ public function testQueue(): void
         $mail->setBCC('admin2@google.com');
 
         $this->queue->expects('push')->withArgs(
-            function ($job, $data, Options $options) use ($mail) {
+            function ($job, $data, Options $options) use ($mail): bool {
                 $this->assertSame(MailQueue::JOB_NAME, $job);
                 $this->assertSame($data, MessageSerializer::pack($mail));
                 $this->assertSame('mailer', $options->getQueue());
@@ -68,21 +68,21 @@ public function testQueueWithDelay(): void
         $mail3->setDelay(200);
 
         $this->queue->expects('push')->once()->withArgs(
-            function ($job, $data, Options $options) {
+            function ($job, $data, Options $options): bool {
                 $this->assertSame(30, $options->getDelay());
                 return true;
             }
         );
 
         $this->queue->expects('push')->once()->withArgs(
-            function ($job, $data, Options $options) {
+            function ($job, $data, Options $options): bool {
                 $this->assertSame(100, $options->getDelay());
                 return true;
             }
         );
 
         $this->queue->expects('push')->once()->withArgs(
-            function ($job, $data, Options $options) {
+            function ($job, $data, Options $options): bool {
                 $this->assertSame(200, $options->getDelay());
                 return true;
             }