When selecting a location for case log, only show active locations

app/models/form/setup/questions/location_id.rb

@@ -13,7 +13,7 @@ class Form::Setup::Questions::LocationId < ::Form::Question
     answer_opts = {}
     return answer_opts unless ActiveRecord::Base.connected?
 
-    Location.select(:id, :postcode, :name).each_with_object(answer_opts) do |location, hsh|
+    Location.select(:id, :postcode, :name).where("startdate <= ? or startdate IS NULL", Time.zone.now).each_with_object(answer_opts) do |location, hsh|
       hsh[location.id.to_s] = { "value" => location.postcode, "hint" => location.name }
       hsh
     end

spec/models/form/setup/questions/location_id_spec.rb

@@ -31,7 +31,74 @@ RSpec.describe Form::Setup::Questions::LocationId, type: :model do
     expect(question).not_to be_derived
   end
 
-  it "has the correct answer_options" do
+  context "when there are no locations" do
-    expect(question.answer_options).to eq({})
+    it "the answer_options is an empty hash" do
+      expect(question.answer_options).to eq({})
+    end
+  end
+
+  context "when getting available locations" do
+    let(:scheme) { FactoryBot.create(:scheme) }
+    let(:case_log) { FactoryBot.create(:case_log, scheme:, needstype: 2) }
+
+    context "when there are no locations" do
+      it "the displayed_answer_options is an empty hash" do
+        expect(question.displayed_answer_options(case_log)).to eq({})
+      end
+    end
+
+    context "when selected scheme has locations" do
+      before do
+        Timecop.freeze(Time.utc(2022, 5, 12))
+      end
+
+      after do
+        Timecop.unfreeze
+      end
+
+      context "and all the locations have a future startdate" do
+        before do
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2022, 5, 13))
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2023, 1, 1))
+        end
+
+        it "the displayed_answer_options is an empty hash" do
+          expect(question.displayed_answer_options(case_log)).to eq({})
+        end
+      end
+
+      context "and the locations have a no startdate" do
+        before do
+          FactoryBot.create(:location, scheme:, startdate: nil)
+          FactoryBot.create(:location, scheme:, startdate: nil)
+        end
+
+        it "the displayed_answer_options shows the locations" do
+          expect(question.displayed_answer_options(case_log).count).to eq(2)
+        end
+      end
+
+      context "and the locations have a past startdate" do
+        before do
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2022, 4, 10))
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2022, 5, 12))
+        end
+
+        it "the displayed_answer_options shows the locations" do
+          expect(question.displayed_answer_options(case_log).count).to eq(2)
+        end
+      end
+
+      context "and some locations have a past startdate" do
+        before do
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2022, 10, 10))
+          FactoryBot.create(:location, scheme:, startdate: Time.utc(2022, 5, 12))
+        end
+
+        it "the displayed_answer_options shows the active location" do
+          expect(question.displayed_answer_options(case_log).count).to eq(1)
+        end
+      end
+    end
   end
 end