From 4fa71f71a09dfc3a313bfd8f276960ff560c3e33 Mon Sep 17 00:00:00 2001
From: sephrat <34862846+sephrat@users.noreply.github.com>
Date: Wed, 20 Apr 2022 13:30:03 +0200
Subject: [PATCH] Better error message when test email fails due to missing
 recipient

---
 src/Ombi/Controllers/V1/External/TesterController.cs | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/Ombi/Controllers/V1/External/TesterController.cs b/src/Ombi/Controllers/V1/External/TesterController.cs
index f18547e0f..6ee0ebe22 100644
--- a/src/Ombi/Controllers/V1/External/TesterController.cs
+++ b/src/Ombi/Controllers/V1/External/TesterController.cs
@@ -282,11 +282,18 @@ namespace Ombi.Controllers.V1.External
         {
             try
             {
+                var currentUser = await GetCurrentUserAsync();
+
+                if (!currentUser.Email.HasValue())
+                {
+                    throw new Exception($"User '{currentUser.UserName}' has no email address set on their user profile.");
+                }
+
                 var message = new NotificationMessage
                 {
                     Message = "This is just a test! Success!",
                     Subject = $"Ombi: Test",
-                    To = (await GetCurrentUserAsync()).Email,
+                    To = currentUser.Email,
                 };
 
                 message.Other.Add("PlainTextBody", "This is just a test! Success!");
@@ -539,7 +546,7 @@ namespace Ombi.Controllers.V1.External
             {
 
                 var user = await UserManager.Users.Include(x => x.UserNotificationPreferences).FirstOrDefaultAsync(x => x.UserName == HttpContext.User.Identity.Name);
-                
+
 
                 var status = await WhatsAppApi.SendMessage(new WhatsAppModel {
                     From = settings.From,