add tests for dom and section parser ignoring blank markup sections

bustle · Jan 16, 2019 · 100c5fb · 100c5fb
1 parent a5353ff
commit 100c5fb
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/tests/unit/parsers/dom-test.js b/tests/unit/parsers/dom-test.js
@@ -49,6 +49,7 @@ let expectations = [
   ['<p>first line</p>\n<p>second line</p>', ['first line','second line']],
   ['<p>first line</p>middle line<p>third line</p>', ['first line','middle line','third line']],
   ['<p>first line</p>second line', ['first line','second line']],
+  ['<p>first line</p><p></p><p>third line</p>', ['first line', 'third line']],
 
   ['<b>bold text</b>',['*bold text*']],
 

diff --git a/tests/unit/parsers/section-test.js b/tests/unit/parsers/section-test.js
@@ -173,6 +173,20 @@ test('#parse only runs text nodes through parserPlugins once', (assert) => {
   assert.equal(pluginRunCount, 1);
 });
 
+test('#parse ignores blank markup sections', assert => {
+  let container = buildDOM(`
+    <div><p>One</p><p></p><p>Three</p></div>
+  `);
+
+  let element = container.firstChild;
+  parser = new SectionParser(builder);
+  let sections = parser.parse(element);
+
+  assert.equal(sections.length, 2, 'Two sections');
+  assert.equal(sections[0].text, 'One');
+  assert.equal(sections[1].text, 'Three');
+});
+
 test("#parse handles single paragraph in list item", assert => {
   let container = buildDOM(`
     <ul><li><p>One</p></li>