Fixed: Ensure failing providers are marked as failed when testing all

src/NzbDrone.Core/Applications/ApplicationFactory.cs

@@ -62,10 +62,19 @@ namespace NzbDrone.Core.Applications
         {
             var result = base.Test(definition);
 
-            if ((result == null || result.IsValid) && definition.Id != 0)
+            if (definition.Id == 0)
+            {
+                return result;
+            }
+
+            if (result == null || result.IsValid)
             {
                 _applicationStatusService.RecordSuccess(definition.Id);
             }
+            else
+            {
+                _applicationStatusService.RecordFailure(definition.Id);
+            }
 
             return result;
         }

src/NzbDrone.Core/Download/DownloadClientFactory.cs

@@ -75,10 +75,19 @@ namespace NzbDrone.Core.Download
         {
             var result = base.Test(definition);
 
-            if ((result == null || result.IsValid) && definition.Id != 0)
+            if (definition.Id == 0)
+            {
+                return result;
+            }
+
+            if (result == null || result.IsValid)
             {
                 _downloadClientStatusService.RecordSuccess(definition.Id);
             }
+            else
+            {
+                _downloadClientStatusService.RecordFailure(definition.Id);
+            }
 
             return result;
         }