tags where HtmlAgilityPack creates nested structure + var html = "
Level1
Level2
Level3
Level4
Level5
"; + + var config = new Config + { + GithubFlavored = true, + UnknownTags = Config.UnknownTagsOption.Bypass + }; + + return CheckConversion(html, config); + } + + [Fact] + public Task When_NestedSpans_FiveLevelsDeep_ThenConvertCorrectly() + { + // Tests moderately nested tags to ensure span bypass converter handles nesting + var html = "Level1Level2Level3Level4Level5"; + + var config = new Config + { + UnknownTags = Config.UnknownTagsOption.Bypass + }; + + return CheckConversion(html, config); + } + + [Fact] + public Task When_InterleavedParagraphsAndSpans_ThenConvertCorrectly() + { + // Tests the interleaved pattern common in malformed HTML
+ var html = " Text1 Text2 Text3 Text4 tags as found in user-generated content
+ var html = " Part1 Part2 Part3 Part4 Part5 Part6 Part7 Part8";
+
+ var config = new Config
+ {
+ GithubFlavored = true,
+ UnknownTags = Config.UnknownTagsOption.Bypass
+ };
+
+ return CheckConversion(html, config);
+ }
+
+ [Fact]
+ public Task When_UnclosedParagraphsWithSpansAndTextNodes_ThenConvertCorrectly()
+ {
+ // Tests mixed content: properly closed tags, text nodes, and unclosed nested tags
+ var html = @" Intro Section1 Section2 Section3";
+
+ var config = new Config
+ {
+ GithubFlavored = true,
+ UnknownTags = Config.UnknownTagsOption.Bypass
+ };
+
+ return CheckConversion(html, config);
+ }
+
+ [Fact]
+ public Task When_EmptyNestedParagraphs_ThenConvertCorrectly()
+ {
+ // Tests deeply nested empty tags
+ var html = "
tags + var html = "
A
B
C
D
E
"; + + var config = new Config + { + UnknownTags = Config.UnknownTagsOption.Bypass + }; + + return CheckConversion(html, config); + } + + [Fact] + public Task When_NestedParagraphs_TenLevelsDeep_ThenConvertCorrectly() + { + // Tests deeper nesting (10 levels) to ensure performance remains linear after fix + var html = "L1
L2
L3
L4
L5
L6
L7
L8
L9
L10
"; + + var config = new Config + { + GithubFlavored = true, + UnknownTags = Config.UnknownTagsOption.Bypass + }; + + return CheckConversion(html, config); + } } } diff --git a/src/ReverseMarkdown/Converters/P.cs b/src/ReverseMarkdown/Converters/P.cs index b90bc99..d6cfe20 100644 --- a/src/ReverseMarkdown/Converters/P.cs +++ b/src/ReverseMarkdown/Converters/P.cs @@ -16,9 +16,13 @@ public override string Convert(HtmlNode node) var indentation = IndentationFor(node); var newlineAfter = NewlineAfter(node); - var content = Converter.Config.CleanupUnnecessarySpaces ? TreatChildren(node).Trim() : TreatChildren(node); + var content = TreatChildren(node); + if (Converter.Config.CleanupUnnecessarySpaces) + { + content = content.Trim(); + } - return $"{indentation}{TreatChildren(node)}{newlineAfter}"; + return $"{indentation}{content}{newlineAfter}"; } private static string IndentationFor(HtmlNode node)