bug #36888 [Mailer] Fix mandrill raw http request setting from email/name (JohJohan)

This PR was merged into the 4.4 branch.

Discussion
----------

[Mailer] Fix mandrill raw http request setting from email/name

| Q             | A
| ------------- | ---
| Branch?       | 4.4, 5.0, 5.1
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #36879
| License       | MIT
| Doc PR        | None

As describe in symfony/symfony#36879 there is a bug in sending raw http request to mandrill it will not set from email/name correct.

As you can see i make sure to set `from_email` and `from_name` correct now and changed the unit test to check correct you can see the doc that the format is correct https://mandrillapp.com/api/docs/messages.curl.html#method-send-raw

Commits
-------

6128dd0b75 ticket_36879 - Fix mandrill raw http request setting from email/name

Author: fabpot

Mailchimp/Tests/Transport/MandrillHttpTransportTest.php

@@ -57,8 +57,9 @@ public function testSend()
             $body = json_decode($options['body'], true);
             $message = $body['raw_message'];
             $this->assertSame('KEY', $body['key']);
+            $this->assertSame('Fabien', $body['from_name']);
+            $this->assertSame('[email protected]', $body['from_email']);
             $this->assertSame('[email protected]', $body['to'][0]);
-            $this->assertSame('Fabien <[email protected]>', $body['from_email']);
 
             $this->assertStringContainsString('Subject: Hello!', $message);
             $this->assertStringContainsString('To: Saif Eddin <[email protected]>', $message);

Mailchimp/Transport/MandrillHttpTransport.php

@@ -49,7 +49,8 @@ protected function doSendHttp(SentMessage $message): ResponseInterface
                 'to' => array_map(function (Address $recipient): string {
                     return $recipient->getAddress();
                 }, $envelope->getRecipients()),
-                'from_email' => $envelope->getSender()->toString(),
+                'from_email' => $envelope->getSender()->getAddress(),
+                'from_name' => $envelope->getSender()->getName(),
                 'raw_message' => $message->toString(),
             ],
         ]);

MandrillHttpTransport.php

@@ -49,7 +49,8 @@ protected function doSendHttp(SentMessage $message): ResponseInterface
                 'to' => array_map(function (Address $recipient): string {
                     return $recipient->getAddress();
                 }, $envelope->getRecipients()),
-                'from_email' => $envelope->getSender()->toString(),
+                'from_email' => $envelope->getSender()->getAddress(),
+                'from_name' => $envelope->getSender()->getName(),
                 'raw_message' => $message->toString(),
             ],
         ]);

MandrillHttpTransportTest.php

@@ -57,8 +57,9 @@ public function testSend()
             $body = json_decode($options['body'], true);
             $message = $body['raw_message'];
             $this->assertSame('KEY', $body['key']);
+            $this->assertSame('Fabien', $body['from_name']);
+            $this->assertSame('[email protected]', $body['from_email']);
             $this->assertSame('[email protected]', $body['to'][0]);
-            $this->assertSame('Fabien <[email protected]>', $body['from_email']);
 
             $this->assertStringContainsString('Subject: Hello!', $message);
             $this->assertStringContainsString('To: Saif Eddin <[email protected]>', $message);