Skip to content

Commit fd490b0

Browse files
committed
[FrameworkBundle] Use MailerAssertionsTrait in KernelTestCase
1 parent 1df9b14 commit fd490b0

File tree

4 files changed

+3
-5
lines changed

4 files changed

+3
-5
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,7 @@ CHANGELOG
5555
* Made `framework.session.handler_id` accept a DSN
5656
* Marked the `RouterDataCollector` class as `@final`.
5757
* [BC Break] The `framework.messenger.buses.<name>.middleware` config key is not deeply merged anymore.
58+
* Moved `MailerAssertionsTrait` in `KernelTestCase`
5859

5960
4.3.0
6061
-----

Test/KernelTestCase.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@
2323
*/
2424
abstract class KernelTestCase extends TestCase
2525
{
26+
use MailerAssertionsTrait;
27+
2628
protected static $class;
2729

2830
/**

Test/MailerAssertionsTrait.php

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -118,10 +118,6 @@ public static function getMailerMessage(int $index = 0, string $transport = null
118118

119119
private static function getMessageMailerEvents(): MessageEvents
120120
{
121-
if (!self::getClient()->getRequest()) {
122-
static::fail('Unable to make email assertions. Did you forget to make an HTTP request?');
123-
}
124-
125121
if (!$logger = self::$container->get('mailer.logger_message_listener')) {
126122
static::fail('A client must have Mailer enabled to make email assertions. Did you forget to require symfony/mailer?');
127123
}

Test/WebTestCase.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
abstract class WebTestCase extends KernelTestCase
2323
{
2424
use WebTestAssertionsTrait;
25-
use MailerAssertionsTrait;
2625

2726
protected function tearDown(): void
2827
{

0 commit comments

Comments
 (0)