diff --git a/src/NzbDrone.Core.Test/HealthCheck/Checks/DownloadClientCheckFixture.cs b/src/NzbDrone.Core.Test/HealthCheck/Checks/DownloadClientCheckFixture.cs index 1615214a7..9f80edd63 100644 --- a/src/NzbDrone.Core.Test/HealthCheck/Checks/DownloadClientCheckFixture.cs +++ b/src/NzbDrone.Core.Test/HealthCheck/Checks/DownloadClientCheckFixture.cs @@ -4,6 +4,7 @@ using NUnit.Framework; using NzbDrone.Core.Download; using NzbDrone.Core.HealthCheck.Checks; using NzbDrone.Core.Test.Framework; +using NzbDrone.Test.Common; namespace NzbDrone.Core.Test.HealthCheck.Checks { @@ -33,6 +34,8 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks .Returns(new IDownloadClient[] { downloadClient.Object }); Subject.Check().ShouldBeError(); + + ExceptionVerification.ExpectedErrors(1); } [Test] diff --git a/src/NzbDrone.Core/HealthCheck/Checks/DownloadClientCheck.cs b/src/NzbDrone.Core/HealthCheck/Checks/DownloadClientCheck.cs index 624057ee1..8a13ad82d 100644 --- a/src/NzbDrone.Core/HealthCheck/Checks/DownloadClientCheck.cs +++ b/src/NzbDrone.Core/HealthCheck/Checks/DownloadClientCheck.cs @@ -1,5 +1,6 @@ using System; using System.Linq; +using NLog; using NzbDrone.Core.Download; namespace NzbDrone.Core.HealthCheck.Checks @@ -7,10 +8,12 @@ namespace NzbDrone.Core.HealthCheck.Checks public class DownloadClientCheck : HealthCheckBase { private readonly IProvideDownloadClient _downloadClientProvider; + private readonly Logger _logger; - public DownloadClientCheck(IProvideDownloadClient downloadClientProvider) + public DownloadClientCheck(IProvideDownloadClient downloadClientProvider, Logger logger) { _downloadClientProvider = downloadClientProvider; + _logger = logger; } public override HealthCheck Check() @@ -29,9 +32,10 @@ namespace NzbDrone.Core.HealthCheck.Checks downloadClient.GetItems(); } } - catch (Exception e) + catch (Exception ex) { - return new HealthCheck(GetType(), HealthCheckResult.Error, "Unable to communicate with download client " + e.Message); + _logger.Error("Unable to communicate with download client: ", ex); + return new HealthCheck(GetType(), HealthCheckResult.Error, "Unable to communicate with download client: " + ex.Message); } return new HealthCheck(GetType());