From 49d3c27647cd3917171c7c1d594e5743d3541a2e Mon Sep 17 00:00:00 2001
From: ta264 <ta264@users.noreply.github.com>
Date: Sun, 10 May 2020 11:36:23 +0100
Subject: [PATCH] Fix mono version checks and mono test checks

---
 azure-pipelines.yml                                      | 2 +-
 .../HealthCheck/Checks/MonoVersionCheckFixture.cs        | 9 +++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/azure-pipelines.yml b/azure-pipelines.yml
index e080e5e1e..bd6a96401 100644
--- a/azure-pipelines.yml
+++ b/azure-pipelines.yml
@@ -385,7 +385,7 @@ stages:
           testResultsFormat: 'NUnit'
           testResultsFiles: '**/TestResult.xml'
           testRunTitle: '$(testName) Unit Tests'
-          failTaskOnFailedTests: false
+          failTaskOnFailedTests: true
 
   - stage: Integration
     displayName: Integration
diff --git a/src/NzbDrone.Core.Test/HealthCheck/Checks/MonoVersionCheckFixture.cs b/src/NzbDrone.Core.Test/HealthCheck/Checks/MonoVersionCheckFixture.cs
index e33402e41..bdcc0caef 100644
--- a/src/NzbDrone.Core.Test/HealthCheck/Checks/MonoVersionCheckFixture.cs
+++ b/src/NzbDrone.Core.Test/HealthCheck/Checks/MonoVersionCheckFixture.cs
@@ -18,7 +18,6 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks
                   .Returns(new Version(version));
         }
 
-        [TestCase("5.18")]
         [TestCase("5.20")]
         public void should_return_ok(string version)
         {
@@ -27,7 +26,6 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks
             Subject.Check().ShouldBeOk();
         }
 
-        [TestCase("5.16")]
         public void should_return_notice(string version)
         {
             GivenOutput(version);
@@ -35,8 +33,6 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks
             Subject.Check().ShouldBeNotice();
         }
 
-        [TestCase("5.4")]
-        [TestCase("5.8")]
         public void should_return_warning(string version)
         {
             GivenOutput(version);
@@ -57,6 +53,11 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks
         [TestCase("4.2")]
         [TestCase("4.4.0")]
         [TestCase("4.4.1")]
+        [TestCase("4.6.2")]
+        [TestCase("5.4")]
+        [TestCase("5.8")]
+        [TestCase("5.16")]
+        [TestCase("5.18")]
         public void should_return_error(string version)
         {
             GivenOutput(version);