Skip to content
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.

Sign up for GitHub

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

[7.x] Add plain text only notifications #33781

Merged
merged 3 commits into from
Aug 7, 2020
Merged

[7.x] Add plain text only notifications #33781

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f9731a1
Add test for html only notifications
pawelmysior Aug 7, 2020
e883eff
Add ability to send plain text only notifications
pawelmysior Aug 7, 2020
4c56637
Improve class name
pawelmysior Aug 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Improve class name
  • Loading branch information
@pawelmysior
pawelmysior committed Aug 7, 2020
commit 4c56637321d7f88a69e20c395c6a9b20012b51b7
6 changes: 3 additions & 3 deletions tests/Integration/Notifications/SendingMailNotificationsTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -251,7 +251,7 @@ public function testMailIsSentUsingMailable()

public function testMailIsSentUsingMailMessageWithHtmlAndPlain()
{
$notification = new TestMailNotificationWithPlain;
$notification = new TestMailNotificationWithHtmlAndPlain;
$notification->id = Str::uuid()->toString();

$user = NotifiableUser::forceCreate([
Expand All @@ -270,7 +270,7 @@ public function testMailIsSentUsingMailMessageWithHtmlAndPlain()

$message->shouldReceive('to')->once()->with(['taylor@laravel.com']);

$message->shouldReceive('subject')->once()->with('Test Mail Notification With Plain');
$message->shouldReceive('subject')->once()->with('Test Mail Notification With Html And Plain');

$closure($message);

Expand Down Expand Up @@ -428,7 +428,7 @@ public function toMail($notifiable)
}
}

class TestMailNotificationWithPlain extends Notification
class TestMailNotificationWithHtmlAndPlain extends Notification
{
public function via($notifiable)
{
Expand Down