Allow Notify exception to propagate from request info.

hmcts · Feb 12, 2025 · 9d00b77 · 9d00b77
1 parent 2dc52eb
commit 9d00b77
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/src/main/java/uk/gov/hmcts/probate/service/InformationRequestCorrespondenceService.java b/src/main/java/uk/gov/hmcts/probate/service/InformationRequestCorrespondenceService.java
@@ -19,14 +19,8 @@ public class InformationRequestCorrespondenceService {
     private final NotificationService notificationService;
 
     public List<Document> emailInformationRequest(CaseDetails caseDetails) throws NotificationClientException {
-        try {
-            final Document notification = notificationService.sendEmail(CASE_STOPPED_REQUEST_INFORMATION, caseDetails);
-            log.info("Successful response for request for information email for case id {} ", caseDetails.getId());
-            return List.of(notification);
-        } catch (NotificationClientException e) {
-            log.error(e.getMessage());
-            // this feels wrong - do we not want to alert the caller that the email sending has failed?
-            return List.of();
-        }
+        final Document notification = notificationService.sendEmail(CASE_STOPPED_REQUEST_INFORMATION, caseDetails);
+        log.info("Successful response for request for information email for case id {} ", caseDetails.getId());
+        return List.of(notification);
     }
 }