Merge pull request #154 from alexanderkozlenko/pr_msbuild_case

Make enumeration-based MSBuild properties case-insensitive

Author: tonerdo

src/coverlet.core/Reporters/ReporterFactory.cs

@@ -1,3 +1,4 @@
+using System;
 using System.Linq;
 using System.Collections.Generic;
 
@@ -18,6 +19,6 @@ public ReporterFactory(string format)
         }
 
         public IReporter CreateReporter()
-            => _reporters.FirstOrDefault(r => r.Format == _format);
+            => _reporters.FirstOrDefault(r => string.Equals(r.Format, _format, StringComparison.OrdinalIgnoreCase));
     }
 }

src/coverlet.msbuild.tasks/CoverageResultTask.cs

@@ -90,19 +90,19 @@ public override bool Execute()
 
                     if (_threshold > 0)
                     {
-                        if (linePercent < _threshold && thresholdTypes.Contains("line"))
+                        if (linePercent < _threshold && thresholdTypes.Contains("line", StringComparer.OrdinalIgnoreCase))
                         {
                             exceptionBuilder.AppendLine($"'{Path.GetFileNameWithoutExtension(module.Key)}' has a line coverage '{linePercent}%' below specified threshold '{_threshold}%'");
                             thresholdFailed = true;
                         }
 
-                        if (branchPercent < _threshold && thresholdTypes.Contains("branch"))
+                        if (branchPercent < _threshold && thresholdTypes.Contains("branch", StringComparer.OrdinalIgnoreCase))
                         {
                             exceptionBuilder.AppendLine($"'{Path.GetFileNameWithoutExtension(module.Key)}' has a branch coverage '{branchPercent}%' below specified threshold '{_threshold}%'");
                             thresholdFailed = true;
                         }
 
-                        if (methodPercent < _threshold && thresholdTypes.Contains("method"))
+                        if (methodPercent < _threshold && thresholdTypes.Contains("method", StringComparer.OrdinalIgnoreCase))
                         {
                             exceptionBuilder.AppendLine($"'{Path.GetFileNameWithoutExtension(module.Key)}' has a method coverage '{methodPercent}%' below specified threshold '{_threshold}%'");
                             thresholdFailed = true;