Merge pull request Radarr#4 from Radarr/patch/add-failed-builds

Add artifacts even if build fails but only if less than 5 tests failed.

Author: AeonLucid

RadarrAPI/Release/AppVeyor/AppVeyorReleaseSource.cs

@@ -61,8 +61,7 @@ protected override async Task DoFetchReleasesAsync()
                 // - unsuccesful builds,
                 // - tagged builds (duplicate).
                 if (build.PullRequestId.HasValue ||
-                    build.IsTag ||
-                    build.Status != "success") continue;
+                    build.IsTag) continue;
 
                 var buildExtendedData = await _httpClient.GetStringAsync($"https://ci.appveyor.com/api/projects/{AccountName}/{ProjectSlug}/build/{build.Version}");
                 var buildExtended = JsonConvert.DeserializeObject<AppVeyorProjectLastBuild>(buildExtendedData).Build;
@@ -71,7 +70,6 @@ protected override async Task DoFetchReleasesAsync()
                 var buildJob = buildExtended.Jobs.FirstOrDefault();
                 if (buildJob == null ||
                     buildJob.ArtifactsCount == 0 ||
-                    buildJob.Status != "success" ||
                     !buildExtended.Started.HasValue) continue;
 
                 // Grab artifacts
@@ -183,4 +181,4 @@ protected override async Task DoFetchReleasesAsync()
             }
         }
     }
-}
+}

RadarrAPI/Release/AppVeyor/Responses/AppVeyorJob.cs

@@ -11,6 +11,9 @@ public class AppVeyorJob
         [JsonProperty("artifactsCount", Required = Required.Always)]
         public int ArtifactsCount { get; set; }
 
+        [JsonProperty("failedTestsCount", Required = Required.Always)]
+        public int FailedTestsCount { get; set; }
+
         [JsonProperty("status", Required = Required.Always)]
         public string Status { get; set; }