Defensive code to prevent a crash #191

Author: onizet

src/Html2OpenXml/Expressions/TextExpression.cs

@@ -81,7 +81,8 @@ public override IEnumerable<OpenXmlElement> Interpret (ParsingContext context)
             // doesn't ends/starts with a whitespace
             if (text.Length == 0 && prevIsPhrasing && nextIsPhrasing
                 && (endsWithSpace || startsWithSpace)
-                && !(node.PreviousSibling!.TextContent[node.PreviousSibling!.TextContent.Length - 1].IsWhiteSpaceCharacter()
+                && !(node.PreviousSibling!.TextContent.Length == 0
+                    || node.PreviousSibling!.TextContent[node.PreviousSibling!.TextContent.Length - 1].IsWhiteSpaceCharacter()
                     || node.NextSibling!.TextContent[0].IsWhiteSpaceCharacter()
                 ))
             {

test/HtmlToOpenXml.Tests/WhitespaceTests.cs

@@ -39,12 +39,13 @@ public void ConsecutiveDivs_ReturnsMultipleParagraphs ()
         [TestCase("<span>   Hello \r\n World!   </span>")]
         [TestCase("<span>   Hello\r\n\r\nWorld!   </span>")]
         [TestCase("<div><u>Hello World! </u></div>")]
+        [TestCase("<hr>\n <span>Hello World!</span>")]
         public void Multiline_ReturnsCollapsedText (string html)
         {
             var elements = converter.Parse(html);
-            Assert.That(elements, Has.Count.EqualTo(1));
+            Assert.That(elements, Has.Count.GreaterThanOrEqualTo(1));
             Assert.That(elements, Has.All.TypeOf<Paragraph>());
-            Assert.That(elements[0].InnerText, Is.EqualTo("Hello World!"));
+            Assert.That(elements.Last().InnerText, Is.EqualTo("Hello World!"));
         }
 
         [TestCase("h1")]