Fixed: Health Check produced warning if Sonarr binaries folder was not writable even when the external script update mechanism was selected.

fixes #964

src/NzbDrone.Core.Test/HealthCheck/Checks/UpdateCheckFixture.cs

@@ -6,6 +6,7 @@ using NzbDrone.Common.EnvironmentInfo;
 using NzbDrone.Core.Configuration;
 using NzbDrone.Core.HealthCheck.Checks;
 using NzbDrone.Core.Test.Framework;
+using NzbDrone.Core.Update;
 
 namespace NzbDrone.Core.Test.HealthCheck.Checks
 {
@@ -47,5 +48,28 @@ namespace NzbDrone.Core.Test.HealthCheck.Checks
 
             Subject.Check().ShouldBeError();
         }
+
+        [Test]
+        public void should_not_return_error_when_app_folder_is_write_protected_and_external_script_enabled()
+        {
+            MonoOnly();
+
+            Mocker.GetMock<IConfigFileProvider>()
+                  .Setup(s => s.UpdateAutomatically)
+                  .Returns(true);
+
+            Mocker.GetMock<IConfigFileProvider>()
+                  .Setup(s => s.UpdateMechanism)
+                  .Returns(UpdateMechanism.Script);
+
+            Mocker.GetMock<IAppFolderInfo>()
+                  .Setup(s => s.StartUpFolder)
+                  .Returns(@"/opt/nzbdrone");
+
+            Mocker.GetMock<NzbDrone.Common.Disk.IDiskProvider>()
+                  .Verify(c => c.FolderWritable(Moq.It.IsAny<string>()), Times.Never());
+
+            Subject.Check().ShouldBeOk();
+        }
     }
 }

src/NzbDrone.Core/HealthCheck/Checks/UpdateCheck.cs

@@ -27,7 +27,8 @@ namespace NzbDrone.Core.HealthCheck.Checks
 
         public override HealthCheck Check()
         {
-            if (OsInfo.IsWindows || _configFileProvider.UpdateAutomatically)
+            if ((OsInfo.IsWindows || _configFileProvider.UpdateAutomatically) &&
+                _configFileProvider.UpdateMechanism == UpdateMechanism.BuiltIn)
             {
                 if (!_diskProvider.FolderWritable(_appFolderInfo.StartUpFolder))
                 {