-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Functional tests for sending emails (via both Swiftmailer and Symfony Mailer) #150
Conversation
8dfcb68
to
cfec094
Compare
cfec094
to
7f388da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added comas adding a little bit of noise to the whole PR
|
||
protected function setUp(): void | ||
{ | ||
self::bootKernel(['environment' => 'test_with_swiftmailer']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as we are forcing given envs in these tests, what will happen in test env (both adapters are turned on). Can we have a test for both of them enabled, but with not default adapter set?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If no default adapter is set, then the swiftmailer adapter would be used (so it would be the same test like this one 💃). We can think about extending this test suite, but maybe it can be done in a separate PR?
It was needed to make the build green 💃 |
…les0123) This PR was merged into the 1.7-dev branch. Discussion ---------- | Q | A | --------------- | ----- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | mentioned in #150 (comment) | License | MIT Commits ------- 75e74e7 Add test for Swiftmailer adapter as the default one
Until now, there are no real functional tests in the bundle. I believe we need to finally change it and preferably do it before the next minor release 🚀
Therefore, this PR provides some basic tests that check the content of sent emails (both with Swiftmailer and Symfony Mailer). The most important changes:
test
,test_with_swiftmailer
,test_with_symfony_mailer
andtest_no_mailer
) for easier CI setupMore tests -> better 🎉