Add sync_national_reporting_to_imms_api feature ...

... flag

We're planning to dual-run the national reporting upload with teams, and during
that period we'll want to ensure we don't actually send the vaccination records
to the imms api.

Jira-Issue: MAV-2733

Author: misaka

app/models/concerns/vaccination_record_sync_to_nhs_immunisations_api_concern.rb

@@ -7,7 +7,11 @@ module VaccinationRecordSyncToNHSImmunisationsAPIConcern
     scope :with_correct_source_for_nhs_immunisations_api,
           -> do
             includes(:patient).then do
-              it.sourced_from_service.or(it.sourced_from_bulk_upload)
+              if Flipper.enabled?(:sync_national_reporting_to_imms_api)
+                it.sourced_from_service.or(it.sourced_from_bulk_upload)
+              else
+                it.sourced_from_service
+              end
             end
           end
 
@@ -34,7 +38,11 @@ module VaccinationRecordSyncToNHSImmunisationsAPIConcern
   end
 
   def correct_source_for_nhs_immunisations_api?
-    (sourced_from_service? || sourced_from_bulk_upload?)
+    sourced_from_service? ||
+      (
+        Flipper.enabled?(:sync_national_reporting_to_imms_api) &&
+          sourced_from_bulk_upload?
+      )
   end
 
   def sync_status

config/feature_flags.yml

@@ -43,6 +43,10 @@ reporting_api: Enables the Commissioner reporting component to authenticate to M
   Authorization Code Flow (https://datatracker.ietf.org/doc/html/rfc6749#section-4.1), and retrieve
   statistics from /api/reporting/
 
+sync_national_reporting_to_imms_api: |
+  Sync immunisations records uploaded as part of national reporting to the NHS
+  Immunisations API.
+
 testing_api: Basic API useful for automated testing.
 
 mmrv: Adds support for MMRV vaccinations

spec/features/import_vaccination_records_bulk_spec.rb

@@ -86,6 +86,7 @@ def and_sending_to_nhs_immunisations_api_is_enabled
     Flipper.enable(:imms_api_integration)
     Flipper.enable(:imms_api_sync_job, Programme.flu)
     Flipper.enable(:imms_api_sync_job, Programme.hpv)
+    Flipper.enable(:sync_national_reporting_to_imms_api)
 
     @stubbed_post_request =
       stub_immunisations_api_post(Random.uuid, Random.uuid)

spec/models/concerns/vaccination_record_sync_to_nhs_immunisations_api_concern_spec.rb

@@ -74,6 +74,8 @@
   describe "with_correct_source_for_nhs_immunisations_api scope" do
     subject { VaccinationRecord.with_correct_source_for_nhs_immunisations_api }
 
+    before { Flipper.enable(:sync_national_reporting_to_imms_api) }
+
     let!(:vaccination_record) do
       create(:vaccination_record, programme:, session:)
     end
@@ -90,6 +92,16 @@
       end
 
       it { should include(vaccination_record) }
+
+      context "with the sync_national_reporting_to_imms_api feature flag disabled" do
+        before { Flipper.disable(:sync_national_reporting_to_imms_api) }
+
+        let!(:vaccination_record) do
+          create(:vaccination_record, :sourced_from_bulk_upload, programme:)
+        end
+
+        it { should_not include(vaccination_record) }
+      end
     end
 
     context "when vaccination record was part of a historical upload" do
@@ -112,6 +124,8 @@
   describe "#correct_source_to_nhs_immunisations_api?" do
     subject { vaccination_record.correct_source_for_nhs_immunisations_api? }
 
+    before { Flipper.enable(:sync_national_reporting_to_imms_api) }
+
     context "when the vaccination record is eligible to sync" do
       it { should be true }
     end
@@ -139,6 +153,12 @@
       end
 
       it { should be true }
+
+      context "with the sync_national_reporting_to_imms_api feature flag disabled" do
+        before { Flipper.disable(:sync_national_reporting_to_imms_api) }
+
+        it { should be false }
+      end
     end
 
     context "a vaccination record created because patient is already vaccinated" do