Revert "[11.x] Use Str::wrap() instead of nesting Str::start() inside Str::finish()" (#54528)

* test(mail): 🧪 Add failing test

* fix(mail): 🐛 Make sure, already wrapped strings are not wrapped again

* Update Headers.php

---------

Co-authored-by: Taylor Otwell <[email protected]>

Author: shaedrich

src/Illuminate/Mail/Mailables/Headers.php

@@ -95,7 +95,7 @@ public function text(array $text)
     public function referencesString(): string
     {
         return (new Collection($this->references))
-            ->map(fn ($messageId) => Str::wrap($messageId, '<', '>'))
+            ->map(fn ($messageId) => Str::of($messageId)->start('<')->finish('>')->value())
             ->implode(' ');
     }
 }

tests/Mail/MailMailableHeadersTest.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Illuminate\Tests\Mail;
+
+use Illuminate\Mail\Mailables\Headers;
+use PHPUnit\Framework\TestCase;
+
+class MailMailableHeadersTest extends TestCase
+{
+    public function test()
+    {
+        $headers = new Headers(
+            '[email protected]',
+            [
+                '<[email protected]>',
+                '<[email protected]>',
+                '<[email protected]>',
+                '<[email protected]>',
+            ],
+        );
+
+        $this->assertSame(
+            '<[email protected]> <[email protected]> <[email protected]> <[email protected]>',
+            $headers->referencesString(),
+        );
+    }
+}