Prevent saving of null TestResultDelivery pref (#7018)

CDCgov · Dec 6, 2023 · 2fbc7cc · 2fbc7cc
1 parent e1c651c
commit 2fbc7cc
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/backend/src/main/java/gov/cdc/usds/simplereport/api/queue/QueueMutationResolver.java b/backend/src/main/java/gov/cdc/usds/simplereport/api/queue/QueueMutationResolver.java
@@ -73,7 +73,9 @@ public String addPatientToQueue(
             symptomOnset,
             noSymptoms);
 
-    personService.updateTestResultDeliveryPreference(patientId, testResultDelivery);
+    if (testResultDelivery != null) {
+      personService.updateTestResultDeliveryPreference(patientId, testResultDelivery);
+    }
 
     return to.getInternalId()
         .toString(); // this return is unused in the UI. it used to be PatientLinkInternalId
@@ -99,6 +101,8 @@ public void updateTimeOfTestQuestions(
     testOrderService.updateTimeOfTestQuestions(
         patientId, pregnancy, symptomsMap, symptomOnset, noSymptoms, genderOfSexualPartners);
 
-    personService.updateTestResultDeliveryPreference(patientId, testResultDelivery);
+    if (testResultDelivery != null) {
+      personService.updateTestResultDeliveryPreference(patientId, testResultDelivery);
+    }
   }
 }