+ There is a new comment by <%= @comment.user.full_name %> on your post <%= @comment.post.title %> +
+To see this comment, <%= link_to "Click here", post_url(@comment.post) %>
+Have a great day!
+ + + diff --git a/config/environments/test.rb b/config/environments/test.rb index 3b7f2e37..e0027ca3 100644 --- a/config/environments/test.rb +++ b/config/environments/test.rb @@ -9,7 +9,6 @@ ENV["POSTS_PER_PAGE"] = "15" Rails.application.configure do # Settings specified here will take precedence over those in config/application.rb. - # For Devise config.action_mailer.default_url_options = { host: 'localhost:3000' } @@ -46,6 +45,7 @@ Rails.application.configure do # The :test delivery method accumulates sent emails in the # ActionMailer::Base.deliveries array. config.action_mailer.delivery_method = :test + config.active_job.queue_adapter = :test # Print deprecation notices to the stderr. config.active_support.deprecation = :stderr diff --git a/spec/mailers/previews/user_mailer_preview.rb b/spec/mailers/previews/user_mailer_preview.rb new file mode 100644 index 00000000..3148bfc9 --- /dev/null +++ b/spec/mailers/previews/user_mailer_preview.rb @@ -0,0 +1,6 @@ +# Preview all emails at http://localhost:3000/rails/mailers/user_mailer +class UserMailerPreview < ActionMailer::Preview + def notify_post_owner + UserMailer.notify_post_owner(comment: Comment.first) + end +end diff --git a/spec/mailers/user_mailer_spec.rb b/spec/mailers/user_mailer_spec.rb new file mode 100644 index 00000000..c64ee6e4 --- /dev/null +++ b/spec/mailers/user_mailer_spec.rb @@ -0,0 +1,22 @@ +require "rails_helper" + +RSpec.describe UserMailer, type: :mailer do + describe "notify_comment" do + let(:user) { FactoryBot.create(:user, email: "notified@example.com", notifications_enabled: true) } + let(:post) { FactoryBot.create(:post, user: user)} + let(:comment) { FactoryBot.create(:comment, post: post) } + let(:mail) { UserMailer.notify_post_owner(comment: comment) } + + it "renders the headers" do + expect(mail.subject).to eq("[#{ENV.fetch('APP_NAME')}] - New comment on #{post.title}") + expect(mail.to).to eq(["notified@example.com"]) + expect(mail.from).to eq(["notifications@example.com"]) + end + + it "renders the body" do + expect(mail.body.encoded).to include("Hello, #{user.full_name}") + expect(mail.body.encoded).to include("There is a new comment by") + expect(mail.body.encoded).to include('Annoyed ? You can turn off notifications here') + end + end +end diff --git a/spec/support/action_mailer.rb b/spec/support/action_mailer.rb new file mode 100644 index 00000000..b9563a3b --- /dev/null +++ b/spec/support/action_mailer.rb @@ -0,0 +1,5 @@ +RSpec.configure do |config| + config.before(:each) do + ActionMailer::Base.deliveries.clear + end +end diff --git a/spec/system/comments_spec.rb b/spec/system/comments_spec.rb index 350c9cd4..539880ac 100644 --- a/spec/system/comments_spec.rb +++ b/spec/system/comments_spec.rb @@ -108,4 +108,34 @@ feature 'comments', type: :system, js: true do expect(Comment.where(post_id: post.id).count).to eq(comments_count + 1) end -end \ No newline at end of file + + it 'notifies post owner when comment is posted' do + log_in_as user + visit post_path(post) + allow(UserMailer).to receive_message_chain(:notify_post_owner, :deliver_later) + + comment_body = 'this is a comment!' + + find(newCommentBodySelector).fill_in with: comment_body + click_button 'Submit' + visit post_path(post) + + expect(UserMailer).to have_received(:notify_post_owner) + end + + it 'does not notify the post owner if he refused notifications' do + post_owner = FactoryBot.create(:user, notifications_enabled: false) + post = FactoryBot.create(:post, user: post_owner) + log_in_as user + visit post_path(post) + allow(UserMailer).to receive_message_chain(:notify_post_owner, :deliver_later) + + comment_body = 'this is a comment!' + + find(newCommentBodySelector).fill_in with: comment_body + click_button 'Submit' + visit post_path(post) + + expect(UserMailer).not_to have_received(:notify_post_owner) + end +end diff --git a/spec/system/user_sign_up_spec.rb b/spec/system/user_sign_up_spec.rb index 8ac4241f..0f604cde 100644 --- a/spec/system/user_sign_up_spec.rb +++ b/spec/system/user_sign_up_spec.rb @@ -77,7 +77,7 @@ feature 'sign up', type: :system do expect(page).to have_css('.alert') end - scenario 'and disables notifications' do + scenario 'with disabled notifications' do visit new_user_registration_path fill_in 'Full name', with: user.full_name fill_in 'Email', with: user.email