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

(cherry picked from commit f6c05d4456a5667398319e249614e2eed115621e)

src/NzbDrone.Core/Download/DownloadClientFactory.cs

@@ -74,10 +74,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;
         }

src/NzbDrone.Core/ImportLists/ImportListFactory.cs

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