[BC BREAK] remove TestMailer::sentTestEmails()

use TestMailer::sentEmails()->all() instead
zenstruck · Oct 7, 2021 · 59b0ce5 · 59b0ce5
1 parent 1ad786e
commit 59b0ce5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 16 deletions.
diff --git a/src/TestMailer.php b/src/TestMailer.php
@@ -4,7 +4,6 @@
 
 use Symfony\Component\Mailer\Event\MessageEvent;
 use Symfony\Component\Mailer\Event\MessageEvents;
-use Symfony\Component\Mime\Email;
 
 /**
  * @author Kevin Bond <kevinbond@gmail.com>
@@ -48,20 +47,6 @@ public function sentEmails(): SentEmails
         return SentEmails::fromEvents(self::$events);
     }
 
-    /**
-     * @return TestEmail[]
-     */
-    public function sentTestEmails(string $testEmailClass = TestEmail::class): array
-    {
-        // todo deprecate/remove
-
-        if (!\is_a($testEmailClass, TestEmail::class, true)) {
-            throw new \InvalidArgumentException(\sprintf('$testEmailClass must be a class that\'s an instance of "%s".', TestEmail::class));
-        }
-
-        return \array_map(static fn(Email $email) => new $testEmailClass($email), $this->sentEmails()->all());
-    }
-
     /**
      * Reset the collected emails.
      */

diff --git a/tests/InteractsWithMailerTest.php b/tests/InteractsWithMailerTest.php
@@ -151,9 +151,11 @@ public function sent_test_email_argument_must_be_an_instance_of_test_email(): vo
     {
         self::bootKernel();
 
+        self::$container->get('mailer')->send(new Email1());
+
         $this->expectException(\InvalidArgumentException::class);
 
-        $this->mailer()->sentTestEmails(\stdClass::class);
+        $this->mailer()->sentEmails()->all(\stdClass::class);
     }
 
     /**