Refactor test reset endpoint to use PatientDeleter

Jira-Issue: MAV-7067

Author: alistair-white-horne-tng

app/controllers/api/testing/teams_controller.rb

@@ -57,43 +57,10 @@ def destroy
       )
     )
 
-    patient_ids = team.patients.pluck(:id)
-
-    log_destroy(
-      PatientLocation.joins_team_locations.where(team_locations: { team_id: })
-    )
-
-    log_destroy(AccessLogEntry.where(patient_id: patient_ids))
-    log_destroy(ArchiveReason.where(patient_id: patient_ids))
-    log_destroy(AttendanceRecord.where(patient_id: patient_ids))
-    log_destroy(ConsentNotification.where(patient_id: patient_ids))
-    log_destroy(GillickAssessment.where(patient_id: patient_ids))
-    log_destroy(Note.where(patient_id: patient_ids))
     # In local dev we can end up with NotifyLogEntries without a patient
     log_destroy(NotifyLogEntry.where(patient_id: nil))
-    log_destroy(NotifyLogEntry.where(patient_id: patient_ids))
-    log_destroy(PatientChangeset.where(patient_id: patient_ids))
-    log_destroy(PatientLocation.where(patient_id: patient_ids))
-    log_destroy(PatientMergeLogEntry.where(patient_id: patient_ids))
-    log_destroy(PatientSpecificDirection.where(patient_id: patient_ids))
-    log_destroy(PDSSearchResult.where(patient_id: patient_ids))
-    log_destroy(PreScreening.where(patient_id: patient_ids))
-    log_destroy(SchoolMove.where(patient_id: patient_ids))
-    log_destroy(SchoolMoveLogEntry.where(patient_id: patient_ids))
-    log_destroy(VaccinationRecord.where(patient_id: patient_ids))
-    log_destroy(Triage.where(patient_id: patient_ids))
-    log_destroy(ImportantNotice.where(patient_id: patient_ids))
-
-    log_destroy(ParentRelationship.where(patient_id: patient_ids))
-    log_destroy(
-      PatientProgrammeVaccinationsSearch.where(patient_id: patient_ids)
-    )
-    log_destroy(Patient.where(id: patient_ids))
 
-    log_destroy(
-      Consent.where(parent: Parent.where.missing(:parent_relationships))
-    )
-    log_destroy(Parent.where.missing(:parent_relationships))
+    log_destroy_patients(patients: team.patients)
 
     log_destroy(Batch.where(team:))
 
@@ -103,10 +70,6 @@ def destroy
 
     log_destroy(Triage.where(team:))
 
-    # These should have been deleted anyway due to the foreign key cascade, but
-    # just to be safe.
-    log_destroy(PatientTeam.where(team:))
-
     TeamCachedCounts.new(team).reset_all!
 
     log_destroy(Session.for_team(team))
@@ -186,4 +149,10 @@ def log_destroy(query)
     )
     @log_time = Time.zone.now
   end
+
+  def log_destroy_patients(patients:)
+    PatientDeleter.call(patients:)
+    response.stream.write("PatientDeleter.call(patients: team.patients)")
+    @log_time = Time.zone.now
+  end
 end

spec/controllers/api/testing/teams_controller_spec.rb

@@ -4,7 +4,11 @@
   include ActiveJob::TestHelper
   include ImportsHelper
 
-  before { Flipper.enable(:testing_api) }
+  before do
+    Flipper.enable(:testing_api)
+    allow(Rails.env).to receive(:production?).and_return(false)
+  end
+
   after { Flipper.disable(:testing_api) }
 
   around { |example| travel_to(Date.new(2025, 7, 31)) { example.run } }