[Mailer] Reflect sandbox state in MailjetApiTransport::__toString()

alexandre-daubois · alexandre-daubois · commit d484d6969055 · 2023-02-21T19:05:31.000+01:00
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailjet/Tests/Transport/MailjetApiTransportTest.php b/src/Symfony/Component/Mailer/Bridge/Mailjet/Tests/Transport/MailjetApiTransportTest.php
@@ -45,6 +45,10 @@ public static function getTransportData()
                 (new MailjetApiTransport(self::USER, self::PASSWORD))->setHost('example.com'),
                 'mailjet+api://example.com',
             ],
+            [
+                (new MailjetApiTransport(self::USER, self::PASSWORD, sandbox: true))->setHost('example.com'),
+                'mailjet+api://example.com?sandbox=true',
+            ]
         ];
     }
 
diff --git a/src/Symfony/Component/Mailer/Bridge/Mailjet/Transport/MailjetApiTransport.php b/src/Symfony/Component/Mailer/Bridge/Mailjet/Transport/MailjetApiTransport.php
@@ -66,7 +66,7 @@ public function __construct(string $publicKey, string $privateKey, HttpClientInt
 
     public function __toString(): string
     {
-        return sprintf('mailjet+api://%s', $this->getEndpoint());
+        return sprintf('mailjet+api://%s', $this->getEndpoint().($this->sandbox ? '?sandbox=true' : ''));
     }
 
     protected function doSendApi(SentMessage $sentMessage, Email $email, Envelope $envelope): ResponseInterface

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public function __construct(string $publicKey, string $privateKey, HttpClientInt`
`66`	`66`
`67`	`67`	`public function __toString(): string`
`68`	`68`	`{`
`69`		`- return sprintf('mailjet+api://%s', $this->getEndpoint());`
	`69`	`+ return sprintf('mailjet+api://%s', $this->getEndpoint().($this->sandbox ? '?sandbox=true' : ''));`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`protected function doSendApi(SentMessage $sentMessage, Email $email, Envelope $envelope): ResponseInterface`