diff --git a/lib/user_segments.rb b/lib/user_segments.rb index bf30f4416..ae6d345be 100644 --- a/lib/user_segments.rb +++ b/lib/user_segments.rb @@ -10,7 +10,7 @@ class UserSegments not_supported_on_current_budget].freeze def self.all_users - User.active + User.active.where.not(confirmed_at: nil) end def self.administrators diff --git a/spec/models/newsletter_spec.rb b/spec/models/newsletter_spec.rb index 275bd39fd..14fa3a454 100644 --- a/spec/models/newsletter_spec.rb +++ b/spec/models/newsletter_spec.rb @@ -50,6 +50,7 @@ describe Newsletter do describe "#list_of_recipient_emails" do before do create(:user, newsletter: true, email: "newsletter_user@consul.dev") + create(:user, newsletter: true, email: "newsletter_unconfirmed_user@consul.dev", confirmed_at: nil) create(:user, newsletter: false, email: "no_news_user@consul.dev") create(:user, email: "erased_user@consul.dev").erase newsletter.update!(segment_recipient: "all_users")