Use delete_all instead of destroy_all

This should improve performance of the endpoint in the testing API used
to clear out a team by avoiding the need to run the Rails callbacks.

Author: thomasleese

app/controllers/api/testing/teams_controller.rb

@@ -59,10 +59,9 @@ def destroy
         log_destroy(Consent.where(team:))
         log_destroy(Triage.where(team:))
 
-        Patient
-          .where(id: patient_ids)
-          .includes(:parents)
-          .in_batches { log_destroy(it) }
+        log_destroy(ParentRelationship.where(patient_id: patient_ids))
+        log_destroy(Patient.where(id: patient_ids))
+        log_destroy(Parent.where.missing(:parent_relationships))
 
         batches = Batch.where(team:)
         log_destroy(VaccinationRecord.where(batch: batches))
@@ -101,7 +100,7 @@ def destroy
   def log_destroy(query)
     where_clause = query.where_clause
     @log_time ||= Time.zone.now
-    query.destroy_all
+    query.delete_all
     response.stream.write(
       "#{query.model.name}.where(#{where_clause.to_h}): #{Time.zone.now - @log_time}s\n"
     )