SesTransport: use correct Tags argument. (#42390)

Tietew · web-flow · commit d2429ab508f0 · 2022-05-16T10:44:04.000-05:00
diff --git a/src/Illuminate/Mail/Transport/SesTransport.php b/src/Illuminate/Mail/Transport/SesTransport.php
@@ -51,7 +51,7 @@ protected function doSend(SentMessage $message): void
         if ($message->getOriginalMessage() instanceof Message) {
             foreach ($message->getOriginalMessage()->getHeaders()->all() as $header) {
                 if ($header instanceof MetadataHeader) {
-                    $options['EmailTags'][] = ['Name' => $header->getKey(), 'Value' => $header->getValue()];
+                    $options['Tags'][] = ['Name' => $header->getKey(), 'Value' => $header->getValue()];
                 }
             }
         }
diff --git a/tests/Mail/MailSesTransportTest.php b/tests/Mail/MailSesTransportTest.php
@@ -10,6 +10,7 @@
 use Illuminate\View\Factory;
 use Mockery as m;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Mailer\Header\MetadataHeader;
 use Symfony\Component\Mime\Email;
 
 class MailSesTransportTest extends TestCase
@@ -55,6 +56,7 @@ public function testSend()
         $message->sender('myself@example.com');
         $message->to('me@example.com');
         $message->bcc('you@example.com');
+        $message->getHeaders()->add(new MetadataHeader('FooTag', 'TagValue'));
 
         $client = m::mock(SesClient::class);
         $sesResult = m::mock();
@@ -63,6 +65,11 @@ public function testSend()
             ->once()
             ->andReturn('ses-message-id');
         $client->shouldReceive('sendRawEmail')->once()
+            ->with(m::on(function ($arg) {
+                return $arg['Source'] === 'myself@example.com' &&
+                    $arg['Destinations'] === ['me@example.com', 'you@example.com'] &&
+                    $arg['Tags'] === [['Name' => 'FooTag', 'Value' => 'TagValue']];
+            }))
             ->andReturn($sesResult);
 
         (new SesTransport($client))->send($message);

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ protected function doSend(SentMessage $message): void`
`51`	`51`	`if ($message->getOriginalMessage() instanceof Message) {`
`52`	`52`	`foreach ($message->getOriginalMessage()->getHeaders()->all() as $header) {`
`53`	`53`	`if ($header instanceof MetadataHeader) {`
`54`		`- $options['EmailTags'][] = ['Name' => $header->getKey(), 'Value' => $header->getValue()];`
	`54`	`+ $options['Tags'][] = ['Name' => $header->getKey(), 'Value' => $header->getValue()];`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`	`}`