kanbanworkflowstimelinescrumrubyroadmapproject-planningproject-managementopenprojectangularissue-trackerifcgantt-chartganttbug-trackerboardsbcf
You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
429 lines
13 KiB
429 lines
13 KiB
#-- encoding: UTF-8
|
|
|
|
#-- copyright
|
|
# OpenProject is an open source project management software.
|
|
# Copyright (C) 2012-2021 the OpenProject GmbH
|
|
#
|
|
# This program is free software; you can redistribute it and/or
|
|
# modify it under the terms of the GNU General Public License version 3.
|
|
#
|
|
# OpenProject is a fork of ChiliProject, which is a fork of Redmine. The copyright follows:
|
|
# Copyright (C) 2006-2013 Jean-Philippe Lang
|
|
# Copyright (C) 2010-2013 the ChiliProject Team
|
|
#
|
|
# This program is free software; you can redistribute it and/or
|
|
# modify it under the terms of the GNU General Public License
|
|
# as published by the Free Software Foundation; either version 2
|
|
# of the License, or (at your option) any later version.
|
|
#
|
|
# This program is distributed in the hope that it will be useful,
|
|
# but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with this program; if not, write to the Free Software
|
|
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
|
|
#
|
|
# See COPYRIGHT and LICENSE files for more details.
|
|
#++
|
|
|
|
require 'spec_helper'
|
|
|
|
describe UserMailer, type: :mailer do
|
|
subject(:deliveries) { ActionMailer::Base.deliveries }
|
|
|
|
let(:type_standard) { FactoryBot.build_stubbed(:type_standard) }
|
|
let(:user) { FactoryBot.build_stubbed(:user) }
|
|
let(:journal) do
|
|
FactoryBot.build_stubbed(:work_package_journal).tap do |j|
|
|
allow(j)
|
|
.to receive(:data)
|
|
.and_return(FactoryBot.build_stubbed(:journal_work_package_journal))
|
|
end
|
|
end
|
|
let(:work_package) do
|
|
FactoryBot.build_stubbed(:work_package,
|
|
type: type_standard)
|
|
end
|
|
|
|
let(:recipient) { FactoryBot.build_stubbed(:user) }
|
|
|
|
before do
|
|
allow(work_package).to receive(:reload).and_return(work_package)
|
|
|
|
allow(journal).to receive(:journable).and_return(work_package)
|
|
allow(journal).to receive(:user).and_return(user)
|
|
|
|
allow(Setting).to receive(:mail_from).and_return('john@doe.com')
|
|
allow(Setting).to receive(:host_name).and_return('mydomain.foo')
|
|
allow(Setting).to receive(:protocol).and_return('http')
|
|
allow(Setting).to receive(:default_language).and_return('en')
|
|
end
|
|
|
|
shared_examples_for 'mail is sent' do
|
|
let(:letters_sent_count) { 1 }
|
|
let(:mail) { deliveries.first }
|
|
let(:html_body) { mail.body.parts.detect { |p| p.content_type.include? 'text/html' }.body.encoded }
|
|
|
|
it 'actually sends a mail' do
|
|
expect(deliveries.size).to eql(letters_sent_count)
|
|
end
|
|
|
|
it 'is sent to the recipient' do
|
|
expect(deliveries.first.to).to include(recipient.mail)
|
|
end
|
|
|
|
it 'is sent from the configured address' do
|
|
expect(deliveries.first.from).to match_array([Setting.mail_from])
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'multiple mails are sent' do |set_letters_sent_count|
|
|
it_behaves_like 'mail is sent' do
|
|
let(:letters_sent_count) { set_letters_sent_count }
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'mail is not sent' do
|
|
it 'sends no mail' do
|
|
expect(deliveries).to be_empty
|
|
end
|
|
end
|
|
|
|
shared_examples_for 'does not send mails to author' do
|
|
let(:user) { FactoryBot.build_stubbed(:user) }
|
|
|
|
context 'when mail is for another user' do
|
|
it_behaves_like 'mail is sent'
|
|
end
|
|
|
|
context 'when mail is for author' do
|
|
let(:recipient) { user }
|
|
|
|
it_behaves_like 'mail is not sent'
|
|
end
|
|
end
|
|
|
|
describe '#with_deliveries' do
|
|
context 'with false' do
|
|
before do
|
|
described_class.with_deliveries(false) do
|
|
described_class.test_mail(recipient).deliver_now
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'mail is not sent'
|
|
end
|
|
|
|
context 'with true' do
|
|
before do
|
|
described_class.with_deliveries(true) do
|
|
described_class.test_mail(recipient).deliver_now
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'mail is sent'
|
|
end
|
|
end
|
|
|
|
describe '#test_mail' do
|
|
let(:test_email) { 'bob.bobbi@example.com' }
|
|
let(:recipient) { FactoryBot.build_stubbed(:user, firstname: 'Bob', lastname: 'Bobbi', mail: test_email) }
|
|
|
|
before do
|
|
described_class.test_mail(recipient).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'has the expected subject' do
|
|
expect(deliveries.first.subject)
|
|
.to eql 'OpenProject Test'
|
|
end
|
|
|
|
it 'includes the url to the instance' do
|
|
expect(deliveries.first.body.encoded)
|
|
.to match Regexp.new("OpenProject URL: #{Setting.protocol}://#{Setting.host_name}")
|
|
end
|
|
end
|
|
|
|
# the name method uses a format setting to determine how to concatenate first name
|
|
# and last name whereby an unescaped comma will lead to have two email addresses
|
|
# defined instead of one (['Bobbi', 'bob.bobbi@example.com'] vs. ['bob.bobbi@example.com'])
|
|
context 'with the user name setting prone to trip up email address separation',
|
|
with_settings: { user_format: :lastname_coma_firstname } do
|
|
it_behaves_like 'mail is sent'
|
|
end
|
|
end
|
|
|
|
describe '#work_package_watcher_changed' do
|
|
let(:watcher_changer) { user }
|
|
|
|
before do
|
|
described_class.work_package_watcher_changed(work_package, recipient, watcher_changer, 'added').deliver_now
|
|
described_class.work_package_watcher_changed(work_package, recipient, watcher_changer, 'removed').deliver_now
|
|
end
|
|
|
|
include_examples 'multiple mails are sent', 2
|
|
|
|
it 'contains the WP subject in the mail subject' do
|
|
expect(deliveries.first.subject).to include(work_package.subject)
|
|
end
|
|
end
|
|
|
|
describe '#wiki_content_added' do
|
|
let(:wiki_content) { FactoryBot.create(:wiki_content) }
|
|
|
|
before do
|
|
described_class.wiki_content_added(recipient, wiki_content, user).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent'
|
|
|
|
it_behaves_like 'does not send mails to author'
|
|
end
|
|
|
|
describe '#wiki_content_updated' do
|
|
let(:wiki_content) { FactoryBot.create(:wiki_content) }
|
|
|
|
before do
|
|
described_class.wiki_content_updated(recipient, wiki_content, user).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent'
|
|
|
|
it 'links to the latest version diff page' do
|
|
expect(deliveries.first.body.encoded).to include 'diff/1'
|
|
end
|
|
|
|
it_behaves_like 'does not send mails to author'
|
|
end
|
|
|
|
describe '#message_posted' do
|
|
let(:message) do
|
|
FactoryBot.build_stubbed(:message).tap do |msg|
|
|
allow(msg)
|
|
.to receive(:project)
|
|
.and_return(msg.forum.project)
|
|
end
|
|
end
|
|
|
|
before do
|
|
described_class.message_posted(recipient, message, user).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'carries a message_id' do
|
|
expect(deliveries.first.message_id)
|
|
.to eql(described_class.generate_message_id(message, recipient))
|
|
end
|
|
|
|
it 'has no references' do
|
|
expect(deliveries.first.references)
|
|
.to be_nil
|
|
end
|
|
|
|
it 'includes a link to the message' do
|
|
expect(html_body)
|
|
.to have_link(message.subject,
|
|
href: topic_url(message, host: Setting.host_name, r: message.id, anchor: "message-#{message.id}"))
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'does not send mails to author'
|
|
end
|
|
|
|
describe '#account_information' do
|
|
let(:pwd) { "pAsswORd" }
|
|
|
|
before do
|
|
described_class.account_information(recipient, pwd).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'includes the password' do
|
|
expect(html_body)
|
|
.to have_content(pwd)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#news_added' do
|
|
let(:news) { FactoryBot.build_stubbed(:news) }
|
|
|
|
before do
|
|
described_class.news_added(recipient, news, user).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'carries a message_id' do
|
|
expect(mail.message_id)
|
|
.to eql(described_class.generate_message_id(news, recipient))
|
|
end
|
|
end
|
|
|
|
it_behaves_like 'does not send mails to author'
|
|
end
|
|
|
|
describe '#news_comment_added' do
|
|
let(:news) { FactoryBot.build_stubbed(:news) }
|
|
let(:comment) { FactoryBot.build_stubbed(:comment, commented: news) }
|
|
|
|
before do
|
|
described_class.news_comment_added(recipient, comment, user).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent'
|
|
|
|
it_behaves_like 'does not send mails to author'
|
|
end
|
|
|
|
describe '#password_lost' do
|
|
let(:token) { FactoryBot.build_stubbed(:recovery_token) }
|
|
let(:recipient) { token.user }
|
|
|
|
before do
|
|
described_class.password_lost(token).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'includes a link to reset' do
|
|
url = account_lost_password_url(host: Setting.host_name, token: token.value)
|
|
|
|
expect(html_body)
|
|
.to have_link(url,
|
|
href: url)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#user_signed_up' do
|
|
let(:token) { FactoryBot.build_stubbed(:invitation_token) }
|
|
let(:recipient) { token.user }
|
|
|
|
before do
|
|
described_class.user_signed_up(token).deliver_now
|
|
end
|
|
|
|
it_behaves_like 'mail is sent' do
|
|
it 'includes a link to activate' do
|
|
url = account_activate_url(host: Setting.host_name, token: token.value)
|
|
|
|
expect(html_body)
|
|
.to have_link(url,
|
|
href: url)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '#message_id' do
|
|
let(:project) { FactoryBot.build_stubbed(:project) }
|
|
let(:message) do
|
|
FactoryBot.build_stubbed(:message).tap do |m|
|
|
allow(m)
|
|
.to receive(:project)
|
|
.and_return project
|
|
end
|
|
end
|
|
let(:message2) do
|
|
FactoryBot.build_stubbed(:message).tap do |m|
|
|
allow(m)
|
|
.to receive(:project)
|
|
.and_return project
|
|
end
|
|
end
|
|
let(:author) { FactoryBot.build_stubbed(:user) }
|
|
|
|
describe 'same user' do
|
|
subject do
|
|
message_ids = [message, message2].map do |m|
|
|
described_class.message_posted(user, m, author).message_id
|
|
end
|
|
|
|
message_ids.uniq.count
|
|
end
|
|
|
|
it { expect(subject).to eq(2) }
|
|
end
|
|
|
|
describe 'same timestamp' do
|
|
let(:user2) { FactoryBot.build_stubbed(:user) }
|
|
|
|
subject do
|
|
message_ids = [user, user2].map do |user|
|
|
described_class.message_posted(user, message, author).message_id
|
|
end
|
|
|
|
message_ids.uniq.count
|
|
end
|
|
|
|
it { expect(subject).to eq(2) }
|
|
end
|
|
end
|
|
|
|
describe 'localization' do
|
|
context 'with the user having a language configured',
|
|
with_settings: { available_languages: %w[en de],
|
|
default_language: 'en',
|
|
emails_header: {
|
|
"de" => 'deutscher header',
|
|
"en" => 'english header'
|
|
} } do
|
|
let(:recipient) do
|
|
FactoryBot.build_stubbed(:user, language: 'de')
|
|
end
|
|
|
|
before do
|
|
described_class.account_information(recipient, 'pwd').deliver_now
|
|
end
|
|
|
|
it 'uses the recipients language' do
|
|
expect(ActionMailer::Base.deliveries.last.body.parts.detect { |p| p.content_type.include? 'text/html' }.body.encoded)
|
|
.to include I18n.t(:mail_body_account_information, locale: :de)
|
|
end
|
|
|
|
it 'does not alter I18n.locale' do
|
|
expect(I18n.locale)
|
|
.to be :en
|
|
end
|
|
|
|
it 'include the user language header' do
|
|
expect(ActionMailer::Base.deliveries.last.body.parts.detect { |p| p.content_type.include? 'text/html' }.body.encoded)
|
|
.to include 'deutscher header'
|
|
end
|
|
end
|
|
|
|
context 'with the user having no language configured',
|
|
with_settings: { available_languages: %w[en de],
|
|
default_language: 'en',
|
|
emails_header: {
|
|
"de" => 'deutscher header',
|
|
"en" => 'english header'
|
|
} } do
|
|
let(:recipient) do
|
|
FactoryBot.build_stubbed(:user, language: '')
|
|
end
|
|
|
|
before do
|
|
I18n.locale = :de
|
|
|
|
described_class.account_information(recipient, 'pwd').deliver_now
|
|
end
|
|
|
|
it 'uses the default language' do
|
|
expect(ActionMailer::Base.deliveries.last.body.parts.detect { |p| p.content_type.include? 'text/html' }.body.encoded)
|
|
.to include I18n.t(:mail_body_account_information, locale: :en)
|
|
end
|
|
|
|
it 'include the default language header' do
|
|
expect(ActionMailer::Base.deliveries.last.body.parts.detect { |p| p.content_type.include? 'text/html' }.body.encoded)
|
|
.to include 'english header'
|
|
end
|
|
|
|
it 'does not alter I18n.locale' do
|
|
expect(I18n.locale)
|
|
.to be :de
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|