diff --git a/lib/user_segments.rb b/lib/user_segments.rb index 096715b26..7bbc8668c 100644 --- a/lib/user_segments.rb +++ b/lib/user_segments.rb @@ -50,7 +50,7 @@ class UserSegments end def self.user_segment_emails(users_segment) - UserSegments.send(users_segment).newsletter.pluck(:email).compact + UserSegments.send(users_segment).newsletter.order(:created_at).pluck(:email).compact end private diff --git a/spec/lib/user_segments_spec.rb b/spec/lib/user_segments_spec.rb index 1a4a965d6..4f1f9ef75 100644 --- a/spec/lib/user_segments_spec.rb +++ b/spec/lib/user_segments_spec.rb @@ -190,4 +190,15 @@ describe UserSegments do end end + describe "#user_segment_emails" do + it "returns list of emails sorted by user creation date" do + create(:user, email: "first@email.com", created_at: 1.day.ago) + create(:user, email: "last@email.com") + + emails = described_class.user_segment_emails(:all_users) + expect(emails.first).to eq "first@email.com" + expect(emails.last).to eq "last@email.com" + end + end + end