Fixed: Record status for notifications on tests

(cherry picked from commit 3d05913534e40e1b9ff217798d806d0b7c170d2d)

src/NzbDrone.Core/Notifications/NotificationFactory.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using FluentValidation.Results;
 using NLog;
 using NzbDrone.Core.Messaging.Events;
 using NzbDrone.Core.ThingiProvider;
@@ -217,5 +218,26 @@ namespace NzbDrone.Core.Notifications
             definition.SupportsOnBookRetag = provider.SupportsOnBookRetag;
             definition.SupportsOnApplicationUpdate = provider.SupportsOnApplicationUpdate;
         }
+
+        public override ValidationResult Test(NotificationDefinition definition)
+        {
+            var result = base.Test(definition);
+
+            if (definition.Id == 0)
+            {
+                return result;
+            }
+
+            if (result == null || result.IsValid)
+            {
+                _notificationStatusService.RecordSuccess(definition.Id);
+            }
+            else
+            {
+                _notificationStatusService.RecordFailure(definition.Id);
+            }
+
+            return result;
+        }
     }
 }