133 lines
4.1 KiB
Ruby
133 lines
4.1 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe EmailsOnPushWorker do
|
|
include RepoHelpers
|
|
include EmailHelpers
|
|
include EmailSpec::Matchers
|
|
|
|
let(:project) { create(:project) }
|
|
let(:user) { create(:user) }
|
|
let(:data) { Gitlab::DataBuilder::Push.build_sample(project, user) }
|
|
let(:recipients) { user.email }
|
|
let(:perform) { subject.perform(project.id, recipients, data.stringify_keys) }
|
|
let(:email) { ActionMailer::Base.deliveries.last }
|
|
|
|
subject { EmailsOnPushWorker.new }
|
|
|
|
describe "#perform" do
|
|
context "when push is a new branch" do
|
|
before do
|
|
data_new_branch = data.stringify_keys.merge("before" => Gitlab::Git::BLANK_SHA)
|
|
|
|
subject.perform(project.id, recipients, data_new_branch)
|
|
end
|
|
|
|
it "sends a mail with the correct subject" do
|
|
expect(email.subject).to include("Pushed new branch")
|
|
end
|
|
|
|
it "sends the mail to the correct recipient" do
|
|
expect(email.to).to eq([user.email])
|
|
end
|
|
end
|
|
|
|
context "when push is a deleted branch" do
|
|
before do
|
|
data_deleted_branch = data.stringify_keys.merge("after" => Gitlab::Git::BLANK_SHA)
|
|
|
|
subject.perform(project.id, recipients, data_deleted_branch)
|
|
end
|
|
|
|
it "sends a mail with the correct subject" do
|
|
expect(email.subject).to include("Deleted branch")
|
|
end
|
|
|
|
it "sends the mail to the correct recipient" do
|
|
expect(email.to).to eq([user.email])
|
|
end
|
|
end
|
|
|
|
context "when push is a force push to delete commits" do
|
|
before do
|
|
data_force_push = data.stringify_keys.merge(
|
|
"after" => data[:before],
|
|
"before" => data[:after]
|
|
)
|
|
|
|
subject.perform(project.id, recipients, data_force_push)
|
|
end
|
|
|
|
it "sends a mail with the correct subject" do
|
|
expect(email.subject).to include('adds bar folder and branch-test text file')
|
|
end
|
|
|
|
it "mentions force pushing in the body" do
|
|
expect(email).to have_body_text("force push")
|
|
end
|
|
|
|
it "sends the mail to the correct recipient" do
|
|
expect(email.to).to eq([user.email])
|
|
end
|
|
end
|
|
|
|
context "when there are no errors in sending" do
|
|
before { perform }
|
|
|
|
it "sends a mail with the correct subject" do
|
|
expect(email.subject).to include('adds bar folder and branch-test text file')
|
|
end
|
|
|
|
it "does not mention force pushing in the body" do
|
|
expect(email).not_to have_body_text("force push")
|
|
end
|
|
|
|
it "sends the mail to the correct recipient" do
|
|
expect(email.to).to eq([user.email])
|
|
end
|
|
end
|
|
|
|
context "when there is an SMTP error" do
|
|
before do
|
|
reset_delivered_emails!
|
|
allow(Notify).to receive(:repository_push_email).and_raise(Net::SMTPFatalError)
|
|
allow(subject).to receive_message_chain(:logger, :info)
|
|
perform
|
|
end
|
|
|
|
it "gracefully handles an input SMTP error" do
|
|
expect(ActionMailer::Base.deliveries.count).to eq(0)
|
|
end
|
|
end
|
|
|
|
context "when there are multiple recipients" do
|
|
let(:recipients) do
|
|
1.upto(5).map { |i| user.email.sub('@', "+#{i}@") }.join("\n")
|
|
end
|
|
|
|
before do
|
|
# This is a hack because we modify the mail object before sending, for efficency,
|
|
# but the TestMailer adapter just appends the objects to an array. To clone a mail
|
|
# object, create a new one!
|
|
# https://github.com/mikel/mail/issues/314#issuecomment-12750108
|
|
allow_any_instance_of(Mail::TestMailer).to receive(:deliver!).and_wrap_original do |original, mail|
|
|
original.call(Mail.new(mail.encoded))
|
|
end
|
|
|
|
reset_delivered_emails!
|
|
end
|
|
|
|
it "sends the mail to each of the recipients" do
|
|
perform
|
|
expect(ActionMailer::Base.deliveries.count).to eq(5)
|
|
expect(ActionMailer::Base.deliveries.map(&:to).flatten).to contain_exactly(*recipients.split)
|
|
end
|
|
|
|
it "only generates the mail once" do
|
|
expect(Notify).to receive(:repository_push_email).once.and_call_original
|
|
expect(Premailer::Rails::CustomizedPremailer).to receive(:new).once.and_call_original
|
|
perform
|
|
end
|
|
end
|
|
end
|
|
end
|