Skip to content
Snippets Groups Projects
Commit 857e703a authored by Eugie Limpin's avatar Eugie Limpin :two:
Browse files

Remove unused scope

parent 43a762bb
No related branches found
No related tags found
No related merge requests found
This commit is part of merge request !83817. Comments created here will be created in the context of that merge request.
......@@ -48,10 +48,6 @@ class InProductMarketingEmail < ApplicationRecord
where(user: user, track: track, series: series)
end
scope :for_user_with_campaign, -> (user, campaign) do
where(user: user, campaign: campaign)
end
scope :without_track_and_series, -> (track, series) do
join_condition = for_user.and(for_track_and_series(track, series))
users_without_records(join_condition)
......
......@@ -113,31 +113,6 @@
end
end
describe '.for_user_with_campaign' do
let_it_be(:user) { create(:user) }
let_it_be(:in_product_marketing_email) { create(:in_product_marketing_email, :campaign, user: user) }
let(:campaign) { in_product_marketing_email.campaign }
subject(:scope) { described_class.for_user_with_campaign(user, campaign).first }
context 'when record for user with given campaign exists' do
it { is_expected.to eq(in_product_marketing_email) }
end
context 'when user is different' do
let(:user) { build_stubbed(:user) }
it { is_expected.to be_nil }
end
context 'when campaign is different' do
let(:campaign) { 'another_campaign' }
it { is_expected.to be_nil }
end
end
describe '.save_cta_click' do
let(:user) { create(:user) }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment