e38196df6f
* Fix typo * Add SMTP configs to Settings::General * Expand Settings's show page * Expand Settings::General's constants * Move smtp_enabled? logic to ApplicationMailer * Apply suggestions from code review Co-authored-by: Ridhwana <Ridhwana.Khan16@gmail.com> * Add missing descriptions and placeholders * Remove production guard clause * Change delivery_method to a callback * Create Settings::SMTP * Run migration * Move constants * Remove SMTP from Settings::General * Create SMTPSettingsController * Add back guard clause * Change which perform_deliveries configuration to use from * Update config/environments/production.rb * Rename migration to singular * Run migration again * Fix name * Alphabetize and add validation for authentication * Move settings and enabled? logic to Settings::SMTP * Change after_action to before_action * Fix broken spec * Fix broken spec * Create SMTP specs * Provide default for port * Create spec for ApplicationMailer * Fix broken spec * Move smtp_enabled? * Search and replace Co-authored-by: Ridhwana <Ridhwana.Khan16@gmail.com>
93 lines
3 KiB
Ruby
93 lines
3 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe Follow, type: :model do
|
|
let(:user) { create(:user) }
|
|
let(:tag) { create(:tag) }
|
|
let(:user_2) { create(:user) }
|
|
|
|
describe "validations" do
|
|
subject { user.follow(user_2) }
|
|
|
|
it { is_expected.to validate_inclusion_of(:subscription_status).in_array(%w[all_articles none]) }
|
|
it { is_expected.to validate_presence_of(:followable_id) }
|
|
it { is_expected.to validate_presence_of(:followable_type) }
|
|
it { is_expected.to validate_presence_of(:follower_id) }
|
|
it { is_expected.to validate_presence_of(:follower_type) }
|
|
it { is_expected.to validate_presence_of(:subscription_status) }
|
|
end
|
|
|
|
it "follows user" do
|
|
user.follow(user_2)
|
|
expect(user.following?(user_2)).to eq(true)
|
|
end
|
|
|
|
it "calculates points with explicit and implicit combined" do
|
|
user.follow(tag)
|
|
follow = described_class.last
|
|
follow.explicit_points = 2.0
|
|
follow.implicit_points = 3.0
|
|
follow.save
|
|
expect(follow.points).to eq(5.0)
|
|
end
|
|
|
|
context "when enqueuing jobs" do
|
|
it "enqueues create channel job" do
|
|
expect do
|
|
described_class.create(follower: user, followable: user_2)
|
|
end.to change(Follows::CreateChatChannelWorker.jobs, :size).by(1)
|
|
end
|
|
|
|
it "enqueues send notification worker" do
|
|
expect do
|
|
described_class.create(follower: user, followable: user_2)
|
|
end.to change(Follows::SendEmailNotificationWorker.jobs, :size).by(1)
|
|
end
|
|
end
|
|
|
|
context "when creating and inline" do
|
|
it "touches the follower user while creating" do
|
|
timestamp = 1.day.ago
|
|
user.update_columns(updated_at: timestamp, last_followed_at: timestamp)
|
|
described_class.create!(follower: user, followable: user_2)
|
|
|
|
user.reload
|
|
expect(user.updated_at).to be > timestamp
|
|
expect(user.last_followed_at).to be > timestamp
|
|
end
|
|
|
|
it "doesn't create a channel when a followable is an org" do
|
|
expect do
|
|
sidekiq_perform_enqueued_jobs do
|
|
described_class.create!(follower: user, followable: create(:organization))
|
|
end
|
|
end.not_to change(ChatChannel, :count)
|
|
end
|
|
|
|
it "doesn't create a chat channel when users don't follow mutually" do
|
|
expect do
|
|
sidekiq_perform_enqueued_jobs do
|
|
described_class.create!(follower: user, followable: user_2)
|
|
end
|
|
end.not_to change(ChatChannel, :count)
|
|
end
|
|
|
|
it "creates a chat channel when users follow mutually" do
|
|
described_class.create!(follower: user_2, followable: user)
|
|
expect do
|
|
sidekiq_perform_enqueued_jobs do
|
|
described_class.create!(follower: user, followable: user_2)
|
|
end
|
|
end.to change(ChatChannel, :count).by(1)
|
|
end
|
|
|
|
it "sends an email notification" do
|
|
allow(ForemInstance).to receive(:smtp_enabled?).and_return(true)
|
|
user_2.update_column(:email_follower_notifications, true)
|
|
expect do
|
|
Sidekiq::Testing.inline! do
|
|
described_class.create!(follower: user, followable: user_2)
|
|
end
|
|
end.to change(EmailMessage, :count).by(1)
|
|
end
|
|
end
|
|
end
|