Improve unit tests to check send is called with the correct parameters

Author: ajibarra

tests/TestCase/Job/SendMailJobTest.php

@@ -58,8 +58,23 @@ public function setUp(): void
      */
     public function testExecute()
     {
+        $job = $this->getMockBuilder(SendMailJob::class)
+            ->onlyMethods(['getTransport'])
+            ->getMock();
         $message = $this->createMessage(DebugTransport::class, [], $this->message);
-        $actual = $this->job->execute($message);
+        $emailMessage = new \Cake\Mailer\Message();
+        $data = json_decode($message->getArgument('emailMessage'), true);
+        $emailMessage->createFromArray($data);
+        $transport = $this->getMockBuilder(DebugTransport::class)->getMock();
+        $transport->expects($this->once())
+            ->method('send')
+            ->with($emailMessage)
+            ->willReturn(['message' => 'test', 'headers' => []]);
+        $job->expects($this->once())
+            ->method('getTransport')
+            ->with(DebugTransport::class, [])
+            ->willReturn($transport);
+        $actual = $job->execute($message);
         $this->assertSame(Processor::ACK, $actual);
     }