Don't reset NR teams with no cutoff date

This handles a potentially problematic edge case

Jira-Issue: MAV-7080

Author: alistair-white-horne-tng

app/lib/mavis_cli/teams/reset_national_reporting.rb

@@ -75,10 +75,16 @@ def find_teams(workgroup)
             raise ArgumentError,
                   "Team #{workgroup} is not a national reporting team"
           end
+          if team.national_reporting_cut_off_date.nil?
+            raise ArgumentError,
+                  "Team #{workgroup} does not have a national reporting cut off date set; no data to delete"
+          end
 
           [team]
         else
-          Team.where(type: :national_reporting)
+          Team
+            .where(type: :national_reporting)
+            .where.not(national_reporting_cut_off_date: nil)
         end
       end
 

spec/features/cli_teams_reset_national_reporting_spec.rb

@@ -82,6 +82,16 @@
       }.to raise_error(ArgumentError, /not a national reporting team/)
     end
 
+    it "raises an error when the team has no national reporting cut off date" do
+      given_a_national_reporting_team_exists_without_a_cut_off_date
+      expect {
+        run_command_with_workgroup(@national_reporting_team.workgroup)
+      }.to raise_error(
+        ArgumentError,
+        /does not have a national reporting cut off date set/
+      )
+    end
+
     it "does not delete records that have been sent to the Imms API" do
       given_a_national_reporting_team_exists
       and_the_national_reporting_team_has_immunisation_imports_with_vaccination_records
@@ -141,6 +151,18 @@
       and_no_vaccination_records_are_deleted
       and_no_patients_are_deleted
     end
+
+    it "skips national_reporting teams that have no cut off date set" do
+      given_a_national_reporting_team_exists_without_a_cut_off_date
+      and_the_national_reporting_team_has_immunisation_imports_with_vaccination_records
+
+      when_i_run_the_command_for_all_teams
+
+      then_the_output_indicates_no_teams_found
+      and_no_immunisation_imports_are_deleted
+      and_no_vaccination_records_are_deleted
+      and_no_patients_are_deleted
+    end
   end
 
   private
@@ -160,6 +182,18 @@ def given_a_national_reporting_team_exists
   alias_method :and_a_national_reporting_team_exists,
                :given_a_national_reporting_team_exists
 
+  def given_a_national_reporting_team_exists_without_a_cut_off_date
+    @national_reporting_team =
+      create(
+        :team,
+        :national_reporting,
+        national_reporting_cut_off_date: nil,
+        programmes: [Programme.hpv, Programme.flu],
+        organisation: national_reporting_organisation,
+        workgroup: "national-reporting-team"
+      )
+  end
+
   def and_the_feature_flag_is_enabled
     Flipper.enable(:sync_national_reporting_to_imms_api)
   end