Fixed: Ensure failing providers are marked as failed when testing all (#3977)

(cherry picked from commit f6c05d4456a5667398319e249614e2eed115621e)

Co-authored-by: Bogdan <mynameisbogdan@users.noreply.github.com>

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;
         }