[Webhook][RemoteEvent] fix SendgridPayloadConverter category support

ericabouaf · nicolas-grekas · commit 186daa540bac · 2025-01-07T12:11:15.000+01:00
diff --git a/src/Symfony/Component/Mailer/Bridge/Sendgrid/RemoteEvent/SendgridPayloadConverter.php b/src/Symfony/Component/Mailer/Bridge/Sendgrid/RemoteEvent/SendgridPayloadConverter.php
@@ -51,7 +51,7 @@ public function convert(array $payload): AbstractMailerEvent
         $event->setDate($date);
         $event->setRecipientEmail($payload['email']);
         $event->setMetadata([]);
-        $event->setTags($payload['category'] ?? []);
+        $event->setTags((array) ($payload['category'] ?? []));
 
         return $event;
     }
diff --git a/src/Symfony/Component/Mailer/Bridge/Sendgrid/Tests/RemoteEvent/SendgridPayloadConverterTest.php b/src/Symfony/Component/Mailer/Bridge/Sendgrid/Tests/RemoteEvent/SendgridPayloadConverterTest.php
@@ -112,4 +112,20 @@ public function testAsynchronousBounce()
         $this->assertInstanceOf(MailerDeliveryEvent::class, $event);
         $this->assertSame('123456', $event->getId());
     }
+
+    public function testWithStringCategory()
+    {
+        $converter = new SendgridPayloadConverter();
+
+        $event = $converter->convert([
+            'event' => 'processed',
+            'sg_message_id' => '123456',
+            'timestamp' => '123456789',
+            'email' => 'test@example.com',
+            'category' => 'cat facts',
+        ]);
+
+        $this->assertInstanceOf(MailerDeliveryEvent::class, $event);
+        $this->assertSame(['cat facts'], $event->getTags());
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public function convert(array $payload): AbstractMailerEvent`
`51`	`51`	`$event->setDate($date);`
`52`	`52`	`$event->setRecipientEmail($payload['email']);`
`53`	`53`	`$event->setMetadata([]);`
`54`		`- $event->setTags($payload['category'] ?? []);`
	`54`	`+ $event->setTags((array) ($payload['category'] ?? []));`
`55`	`55`
`56`	`56`	`return $event;`
`57`	`57`	`}`