Merge pull request #163 from tonyhallett/isnullorempty-string-options

string options should not be checked against null

Author: tonyhallett

FineCodeCoverage/Core/Coverlet/DataCollector/CoverletDataCollectorUtil.cs

@@ -169,7 +169,7 @@ private string GetSettings()
 
         private string GetTestAdapterPathArg()
         {
-            if (coverageProject.Settings.CoverletCollectorDirectoryPath != null) {
+            if (!String.IsNullOrWhiteSpace(coverageProject.Settings.CoverletCollectorDirectoryPath)) {
                 var directoryPath = coverageProject.Settings.CoverletCollectorDirectoryPath.Trim();
                 if (Directory.Exists(directoryPath))
                 {

FineCodeCoverage/Core/OpenCover/OpenCoverUtil.cs

@@ -47,7 +47,7 @@ public void Initialize(string appDataFolder)
 
 		private string GetOpenCoverExePath(string customExePath)
         {
-			if(customExePath != null)
+			if(!String.IsNullOrWhiteSpace(customExePath))
             {
 				return customExePath;
             }