remove regex timeout for IsLocalMethod and IsTypeFilterMatch (#1689)

* remove regex timeout for IsLocalMethod and IsTypeFilterMatch

* use static regex method

* update PublishCodeCoverageResults version

* support windows path separator

Author: Bertk

.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>

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]