Fix unit test

dotnet · Sep 14, 2023 · 40298bd · 40298bd
1 parent 83dfa61
commit 40298bd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...stem.Text.RegularExpressions/src/System/Text/RegularExpressions/RegexFindOptimizations.cs b/...stem.Text.RegularExpressions/src/System/Text/RegularExpressions/RegexFindOptimizations.cs
@@ -142,7 +142,7 @@ public RegexFindOptimizations(RegexNode root, RegexOptions options)
             // If there are multiple leading strings, we can search for any of them.
             if (compiled)
             {
-                if (RegexPrefixAnalyzer.FindPrefixes(root) is { Length: > 1 and < 8 } prefixes)
+                if (RegexPrefixAnalyzer.FindPrefixes(root) is { Length: > 1 } prefixes)
                 {
                     LeadingPrefixes = prefixes;
                     FindMode = FindNextStartingPositionMode.LeadingStrings_LeftToRight;

diff --git a/src/libraries/System.Text.RegularExpressions/tests/UnitTests/RegexPrefixAnalyzerTests.cs b/src/libraries/System.Text.RegularExpressions/tests/UnitTests/RegexPrefixAnalyzerTests.cs
@@ -74,15 +74,16 @@ public void FindFirstCharClass_StressDeep()
         [InlineData("abc|def", new[] { "abc", "def" })]
         [InlineData("abc|def|(ghi|jklm)", new[] { "abc", "def", "ghi", "jklm" })]
         [InlineData("abc[def]ghi", new[] { "abcdghi", "abceghi", "abcfghi" })]
-        [InlineData("abc[def]ghi|[jkl]", new[] { "abcdghi", "abceghi", "abcfghi", "j", "k", "l" })]
-        [InlineData("abcdefg|h", new[] { "abcdefg", "h" })]
+        [InlineData("abc[def]ghi|[jkl]m", new[] { "abcdghi", "abceghi", "abcfghi", "jm", "km", "lm" })]
         [InlineData("agggtaaa|tttaccct", new[] { "agggtaaa", "tttaccct" })]
         [InlineData("[cgt]gggtaaa|tttaccc[acg]", new[] { "cgggtaaa", "ggggtaaa", "tgggtaaa", "tttaccca", "tttacccc", "tttacccg" })]
         [InlineData("a[act]ggtaaa|tttacc[agt]t", new[] { "aaggtaaa", "acggtaaa", "atggtaaa", "tttaccat", "tttaccgt", "tttacctt" })]
         [InlineData("ag[act]gtaaa|tttac[agt]ct", new[] { "agagtaaa", "agcgtaaa", "agtgtaaa", "tttacact", "tttacgct", "tttactct" })]
         [InlineData("agg[act]taaa|ttta[agt]cct", new[] { "aggataaa", "aggctaaa", "aggttaaa", "tttaacct", "tttagcct", "tttatcct" })]
         [InlineData("\b(abc|def)\b", new[] { "abc", "def" })]
         [InlineData("^(abc|def)$", new[] { "abc", "def" })]
+        [InlineData("abcdefg|h", null)]
+        [InlineData("abc[def]ghi|[jkl]", null)]
         public void FindPrefixes(string pattern, string[] expectedSet)
         {
             RegexTree tree = RegexParser.Parse(pattern, RegexOptions.None, CultureInfo.InvariantCulture);