Merge branch 'coverlet-coverage:master' into development

Author: JosephSun2003

.gitignore

@@ -315,3 +315,4 @@ coverage.*.cobertura.xml
 coverage.*.opencover.xml
 
 FolderProfile.pubxml
+/NuGet.config

Directory.Build.props

@@ -38,11 +38,16 @@
     <VSTestLogger Include="html%3BLogFileName=TestResults-$(TargetFramework)-$(MSBuildProjectName).html" />
   </ItemGroup>
 
-  <PropertyGroup>
+  <PropertyGroup Condition=" '$(OS)' != 'Windows_NT' ">
     <VSTestResultsDirectory>$(RepoRoot)artifacts/testresults/$(Configuration.ToLowerInvariant())</VSTestResultsDirectory>
     <VSTestLogger>@(VSTestLogger)</VSTestLogger>
   </PropertyGroup>
 
+    <PropertyGroup Condition=" '$(OS)' == 'Windows_NT' ">
+    <VSTestResultsDirectory>$(RepoRoot)artifacts\testresults\$(Configuration.ToLowerInvariant())</VSTestResultsDirectory>
+    <VSTestLogger>@(VSTestLogger)</VSTestLogger>
+  </PropertyGroup>
+
   <PropertyGroup>
     <!--
         Do not change versions since we need to support VSTest DataCollectors. We need to load assembly version 1.6.0 to properly work
@@ -69,9 +74,9 @@
             <MicrosoftBuildTaskVersion>15.7.179</MicrosoftBuildTaskVersion>
             <MicrosoftBuildTaskUtilitiesCoreVersion>15.7.179</MicrosoftBuildTaskUtilitiesCoreVersion>
             <NuGetBuildTasksPackageVersion>6.9.0-rc.86</NuGetBuildTasksPackageVersion>
-            <MicrosoftBuildTaskSystemReflectionMetaData>1.4.2</MicrosoftBuildTaskSystemReflectionMetaData> 
+            <MicrosoftBuildTaskSystemReflectionMetaData>1.4.2</MicrosoftBuildTaskSystemReflectionMetaData>
             <MicrosoftBuildTaskSystemCollectionImmutable>1.5.0</MicrosoftBuildTaskSystemCollectionImmutable>  >= 1.3.1
-    -->       
+    -->
   </PropertyGroup>
 
 </Project>

Documentation/GlobalTool.md

@@ -114,6 +114,9 @@ coverlet <ASSEMBLY> --target <TARGET> --targetargs <TARGETARGS> --output "/custo
 
 The above command will write the results to the supplied path, if no file extension is specified it'll use the standard extension of the selected output format. To specify a directory instead, simply append a `/` to the end of the value.
 
+>[!TIP]
+>Use only folder name whenever multiple coverage output formats are used.
+
 ```bash
 coverlet <ASSEMBLY> --target <TARGET> --targetargs <TARGETARGS> --output "/custom/directory/" -f json -f lcov
 ```

Documentation/VSTestIntegration.md

@@ -70,7 +70,17 @@ You can change the output directory using the standard `dotnet test` switch `--r
 
 :warning:At the moment VSTest integration **doesn't support all features** of msbuild and .NET tool, for instance show result on console, report merging and threshold validation.
 We're working to fill the gaps.
->*PS: if you don't have any other way to merge reports(for instance your report generator doesn't support multi coverage file) you can for the moment exploit a trick reported by one of our contributor Daniel Paz(@p4p3) <https://github.com/tonerdo/coverlet/pull/225#issuecomment-573896446>*
+
+> [!TIP]
+> *Some alternative solutions to merge coverage files*
+>
+> * use _dotnet-coverage_ tool and merge multiple coverage files
+>
+>   `dotnet-coverage merge artifacts/coverage/**/coverage.cobertura.xml -f cobertura -o artifacts/coverage/coverage.xml`*
+>
+> * use _dotnet-reportgenerator-globaltool_ to create a HTML report and a merged coverage file
+>
+>   `reportgenerator -reports:"**/*.cobertura.xml" -targetdir:"artifacts\reports.cobertura" -reporttypes:"HtmlInline_AzurePipelines_Dark;Cobertura"`
 
 ### Default option (if you don't specify a runsettings file)
 

eng/publish-coverage-results.yml

@@ -26,7 +26,7 @@ steps:
   artifact: CoverageResults_$(Agent.Os)_$(BuildConfiguration)
   condition: ${{parameters.condition}}
 
-- task: PublishCodeCoverageResults@1
+- task: PublishCodeCoverageResults@2
   displayName: 'Publish code coverage'
   condition: ${{parameters.condition}}
   inputs:

src/coverlet.core/Helpers/InstrumentationHelper.cs

@@ -25,7 +25,7 @@ internal class InstrumentationHelper : IInstrumentationHelper
     private readonly ISourceRootTranslator _sourceRootTranslator;
     private ILogger _logger;
     private static readonly RegexOptions s_regexOptions =
-      RegexOptions.Multiline | RegexOptions.Compiled | RegexOptions.IgnoreCase;
+      RegexOptions.Multiline | RegexOptions.Compiled;
 
     public InstrumentationHelper(IProcessExitHandler processExitHandler, IRetryHelper retryHelper, IFileSystem fileSystem, ILogger logger, ISourceRootTranslator sourceRootTranslator)
     {
@@ -420,7 +420,7 @@ public bool IsTypeIncluded(string module, string type, string[] includeFilters)
     }
 
     public bool IsLocalMethod(string method)
-        => new Regex(WildcardToRegex("<*>*__*|*"), s_regexOptions, TimeSpan.FromSeconds(10)).IsMatch(method);
+        => Regex.IsMatch(method, WildcardToRegex("<*>*__*|*"));
 
     public void SetLogger(ILogger logger)
     {
@@ -442,7 +442,7 @@ private static bool IsTypeFilterMatch(string module, string type, string[] filte
         typePattern = WildcardToRegex(typePattern);
         modulePattern = WildcardToRegex(modulePattern);
 
-        if (new Regex(typePattern, s_regexOptions, TimeSpan.FromSeconds(10)).IsMatch(type) && new Regex(modulePattern, s_regexOptions, TimeSpan.FromSeconds(10)).IsMatch(module))
+        if (Regex.IsMatch(type, typePattern) && Regex.IsMatch(module, modulePattern))
           return true;
       }
 

test/Directory.Build.props

@@ -3,6 +3,6 @@
   <Import Project="$([MSBuild]::GetPathOfFileAbove('Directory.Build.props', '$(MSBuildThisFileDirectory)../'))" />
   <PropertyGroup>
     <IsTestProject>true</IsTestProject>
-    <NoWarn>$(NoWarn);NU1301</NoWarn>
+    <NoWarn>$(NoWarn);NU1301;IDE0007</NoWarn>
   </PropertyGroup>
 </Project>

test/coverlet.core.tests/Helpers/InstrumentationHelperTests.cs

@@ -112,23 +112,29 @@ public void TestBackupOriginalModule()
       Assert.True(File.Exists(backupPath));
     }
 
-    [Fact]
-    public void TestIsValidFilterExpression()
+    [Theory]
+    [InlineData("[*]*")]
+    [InlineData("[*]*core")]
+    [InlineData("[assembly]*")]
+    [InlineData("[*]type")]
+    [InlineData("[assembly]type")]
+    [InlineData("[coverlet.*.tests?]*")]
+    [InlineData("[*]Coverlet.Core*")]
+    [InlineData("[coverlet.*]*")]
+    public void TestIsValidFilterExpression(string pattern)
+    {
+      Assert.True(_instrumentationHelper.IsValidFilterExpression(pattern));
+    }
+
+    [Theory]
+    [InlineData("[*]")]
+    [InlineData("[-]*")]
+    [InlineData("*")]
+    [InlineData("][")]
+    [InlineData(null)]
+    public void TestInValidFilterExpression(string pattern)
     {
-      Assert.True(_instrumentationHelper.IsValidFilterExpression("[*]*"));
-      Assert.True(_instrumentationHelper.IsValidFilterExpression("[*]*core"));
-      Assert.True(_instrumentationHelper.IsValidFilterExpression("[assembly]*"));
-      Assert.True(_instrumentationHelper.IsValidFilterExpression("[*]type"));
-      Assert.True(_instrumentationHelper.IsValidFilterExpression("[assembly]type"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("[*]"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("[-]*"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("*"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("]["));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("["));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("[assembly][*"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("[assembly]*]"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression("[]"));
-      Assert.False(_instrumentationHelper.IsValidFilterExpression(null));
+      Assert.False(_instrumentationHelper.IsValidFilterExpression(pattern));
     }
 
     [Fact]