Skip to content

Commit

Permalink
Delete Regex timeout tests with RegexOptions.Debug (#61888)
Browse files Browse the repository at this point in the history
We don't need these tests (the Debug flag is only compiled into Debug builds and is there for diagnostic purposes), and it seems these have been taking too long sporadically. 
Closes #61794
  • Loading branch information
stephentoub authored Nov 22, 2021
1 parent 3745e38 commit 259b561
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -794,9 +794,7 @@ public void Match_DefaultTimeout_Throws(RegexOptions options)
// TODO: Figure out what to do with default timeouts for source generated regexes
[Theory]
[InlineData(RegexOptions.None)]
[InlineData(RegexOptions.None | RegexHelpers.RegexOptionDebug)]
[InlineData(RegexOptions.Compiled)]
[InlineData(RegexOptions.Compiled | RegexHelpers.RegexOptionDebug)]
public void Match_CachedPattern_NewTimeoutApplies(RegexOptions options)
{
const string PatternLeadingToLotsOfBacktracking = @"^(\w+\s?)*$";
Expand Down

0 comments on commit 259b561

Please sign in to comment.