Ensure parsed mobiledocs have a blank marker

bustle · Aug 12, 2015 · e6f656c · e6f656c
1 parent c404d3b
commit e6f656c
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 0 deletions.
diff --git a/src/js/parsers/mobiledoc.js b/src/js/parsers/mobiledoc.js
@@ -21,6 +21,13 @@ export default class MobiledocParser {
     this.markerTypes = this.parseMarkerTypes(markerTypes);
     this.parseSections(sections, post);
 
+    if (post.sections.isEmpty) {
+      let section = this.builder.createMarkupSection('p');
+      let marker = this.builder.createBlankMarker();
+      section.markers.append(marker);
+      post.sections.append(section);
+    }
+
     return post;
   }
 
@@ -67,6 +74,10 @@ export default class MobiledocParser {
     const section = this.builder.createMarkupSection(tagName);
     post.sections.append(section);
     this.parseMarkers(markers, section);
+    if (section.markers.isEmpty) {
+      let marker = this.builder.createBlankMarker();
+      section.markers.append(marker);
+    }
   }
 
   parseMarkers(markers, section) {

diff --git a/tests/unit/parsers/mobiledoc-test.js b/tests/unit/parsers/mobiledoc-test.js
@@ -25,6 +25,27 @@ test('#parse empty doc returns an empty post', (assert) => {
     version: MOBILEDOC_VERSION,
     sections: [[], []]
   };
+
+  let section = builder.createMarkupSection('P');
+  let marker  = builder.createBlankMarker();
+  section.markers.append(marker);
+  post.sections.append(section);
+  assert.deepEqual(parser.parse(mobiledoc),
+                   post);
+});
+
+test('#parse empty markup section returns an empty post', (assert) => {
+  let mobiledoc = {
+    version: MOBILEDOC_VERSION,
+    sections: [[], [
+      [1, 'h2', []]
+    ]]
+  };
+
+  let section = builder.createMarkupSection('h2');
+  let marker  = builder.createBlankMarker();
+  section.markers.append(marker);
+  post.sections.append(section);
   assert.deepEqual(parser.parse(mobiledoc),
                    post);
 });