Merge pull request #2516 from mroderick/add-tests-for-2515

test: Add tests for upcoming scope ordering fix (PR #2515)

Author: olleolleolle

spec/models/concerns/listable_spec.rb

@@ -15,6 +15,15 @@
           expect(Workshop.today_and_upcoming).to match_array(future_workshops)
         end
       end
+
+      it 'returns workshops ordered by date_and_time ascending (soonest first)' do
+        Fabricate.times(2, :past_workshop)
+        workshop_in_3_days = Fabricate(:workshop, date_and_time: 3.days.from_now)
+        workshop_tomorrow = Fabricate(:workshop, date_and_time: 1.day.from_now)
+        workshop_in_5_days = Fabricate(:workshop, date_and_time: 5.days.from_now)
+
+        expect(Workshop.today_and_upcoming).to eq([workshop_tomorrow, workshop_in_3_days, workshop_in_5_days])
+      end
     end
 
     context '#upcoming' do
@@ -24,6 +33,15 @@
 
         expect(Workshop.upcoming).to match_array(future_workshops)
       end
+
+      it 'returns workshops ordered by date_and_time ascending (soonest first)' do
+        Fabricate.times(2, :past_workshop)
+        workshop_in_3_days = Fabricate(:workshop, date_and_time: 3.days.from_now)
+        workshop_tomorrow = Fabricate(:workshop, date_and_time: 1.day.from_now)
+        workshop_in_5_days = Fabricate(:workshop, date_and_time: 5.days.from_now)
+
+        expect(Workshop.upcoming).to eq([workshop_tomorrow, workshop_in_3_days, workshop_in_5_days])
+      end
     end
 
     context '#past' do