Merge branch 'main' into dependabot/bundler/aws-sdk-s3-1.195.0

Author: rsmithlal

spec/factories/better_together/messages.rb

@@ -1,7 +1,11 @@
 # frozen_string_literal: true
 
 FactoryBot.define do
-  factory :message do
-    content { 'MyText' }
+  factory('better-together/message',
+          class: 'BetterTogether::Message',
+          aliases: %i[better_together_message message]) do
+    content { Faker::Lorem.paragraph }
+    association :sender, factory: :person
+    association :conversation
   end
 end

spec/mailers/better_together/conversation_mailer_spec.rb

@@ -3,19 +3,40 @@
 require 'rails_helper'
 
 module BetterTogether
-  RSpec.describe ConversationMailer, type: :mailer do
-    # describe 'new_message_notification' do
-    #   let(:mail) { ConversationMailer.new_message_notification }
+  RSpec.describe ConversationMailer, type: :mailer do # rubocop:todo Metrics/BlockLength
+    describe 'new_message_notification' do # rubocop:todo Metrics/BlockLength
+      let!(:host_platform) { create(:platform, :host) }
+      let(:sender) { create(:user) }
+      let(:recipient) { create(:user) }
+      let(:conversation) { create(:conversation, creator: sender.person) }
+      let(:message) { create(:message, conversation: conversation, sender: sender.person) }
 
-    #   it 'renders the headers' do
-    #     expect(mail.subject).to eq('New message notification')
-    #     expect(mail.to).to eq(['[email protected]'])
-    #     expect(mail.from).to eq(['[email protected]'])
-    #   end
+      let(:mail) do
+        ConversationMailer.with(message: message, recipient: recipient.person)
+                          .new_message_notification
+      end
 
-    #   it 'renders the body' do
-    #     expect(mail.body.encoded).to match('Hi')
-    #   end
-    # end
+      it 'renders the headers' do
+        expect(mail.subject).to eq("[#{host_platform.name}] New message in conversation \"#{conversation.title}\"")
+        expect(mail.to).to eq([recipient.email])
+        expect(mail.from).to eq(['[email protected]'])
+      end
+
+      it 'renders the body' do
+        expect(mail.body.encoded).to have_content("Hello #{recipient.person.name}")
+        expect(mail.body.encoded).to have_content("#{sender.person.name}:")
+        expect(mail.body.encoded).to have_content(message.content.to_plain_text)
+      end
+
+      it 'sends a message notification email' do
+        expect { mail.deliver_now }
+          .to change { ActionMailer::Base.deliveries.count }.by(1)
+      end
+
+      it 'sends the message notification to the correct email address' do
+        mail.deliver_now
+        expect(ActionMailer::Base.deliveries.last.to).to include(recipient.email)
+      end
+    end
   end
 end

spec/mailers/previews/better_together/conversation_mailer_preview.rb

@@ -1,11 +1,21 @@
 # frozen_string_literal: true
 
 module BetterTogether
-  # Preview all emails at http://localhost:3000/rails/mailers/better_together/conversation_mailer_mailer
+  # Preview all emails at http://localhost:3000/rails/mailers/better_together/conversation_mailer
   class ConversationMailerPreview < ActionMailer::Preview
-    # Preview this email at http://localhost:3000/rails/mailers/better_together/conversation_mailer_mailer/new_message_notification
+    include FactoryBot::Syntax::Methods
+    include BetterTogether::ApplicationHelper
+
+    # Preview this email at http://localhost:3000/rails/mailers/better_together/conversation_mailer/new_message_notification
     def new_message_notification
-      ConversationMailer.new_message_notification
+      host_platform || create(:platform)
+      sender = create(:user)
+      recipient = create(:user)
+      conversation = create(:conversation, creator: sender.person)
+      message = create(:message, conversation: conversation, sender: sender.person)
+
+      ConversationMailer.with(message: message, recipient: recipient.person)
+                        .new_message_notification
     end
   end
 end