Skip to content

Commit 18dca2e

Browse files
ericabouafnicolas-grekas
authored andcommittedJan 7, 2025
[Webhook][RemoteEvent] fix SendgridPayloadConverter category support
1 parent 2b21ea5 commit 18dca2e

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed
 

‎RemoteEvent/SendgridPayloadConverter.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ public function convert(array $payload): AbstractMailerEvent
5151
$event->setDate($date);
5252
$event->setRecipientEmail($payload['email']);
5353
$event->setMetadata([]);
54-
$event->setTags($payload['category'] ?? []);
54+
$event->setTags((array) ($payload['category'] ?? []));
5555

5656
return $event;
5757
}

‎Tests/RemoteEvent/SendgridPayloadConverterTest.php

+16
Original file line numberDiff line numberDiff line change
@@ -112,4 +112,20 @@ public function testAsynchronousBounce()
112112
$this->assertInstanceOf(MailerDeliveryEvent::class, $event);
113113
$this->assertSame('123456', $event->getId());
114114
}
115+
116+
public function testWithStringCategory()
117+
{
118+
$converter = new SendgridPayloadConverter();
119+
120+
$event = $converter->convert([
121+
'event' => 'processed',
122+
'sg_message_id' => '123456',
123+
'timestamp' => '123456789',
124+
'email' => 'test@example.com',
125+
'category' => 'cat facts',
126+
]);
127+
128+
$this->assertInstanceOf(MailerDeliveryEvent::class, $event);
129+
$this->assertSame(['cat facts'], $event->getTags());
130+
}
115131
}

0 commit comments

Comments
 (0)
Please sign in to comment.