Permit no payload

bustle · Oct 1, 2015 · d1f36f4 · d1f36f4
1 parent 7b90ad6
commit d1f36f4
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 0 deletions.
diff --git a/lib/dom-renderer.js b/lib/dom-renderer.js
@@ -97,6 +97,9 @@ export default class DOMRenderer {
     if (!card) {
       throw new Error(`Cannot render unknown card named ${name}`);
     }
+    if (!payload) {
+      payload = {};
+    }
     let element = createElement('div');
     card.display.setup(element, {}, {name}, payload);
     return element;

diff --git a/tests/unit/dom-renderer-test.js b/tests/unit/dom-renderer-test.js
@@ -176,6 +176,37 @@ test('renders a mobiledoc with card section', (assert) => {
   assert.equal(sectionEl.innerHTML, payload.name);
 });
 
+test('renders a mobiledoc with card section and no payload', (assert) => {
+  assert.expect(3);
+  let cardName = 'title-card';
+  let TitleCard = {
+    name: cardName,
+    display: {
+      setup(element, options, env, setupPayload) {
+        assert.deepEqual({}, setupPayload);
+        element.innerHTML = '';
+      }
+    }
+  };
+  let mobiledoc = {
+    version: MOBILEDOC_VERSION,
+    sections: [
+      [],      // markers
+      [        // sections
+        [10, cardName]
+      ]
+    ]
+  };
+  let rendered = renderer.render(mobiledoc, document.createElement('div'), {
+    [cardName]: TitleCard
+  });
+  assert.equal(rendered.childNodes.length, 1,
+               'renders 1 section');
+  let sectionEl = rendered.childNodes[0];
+
+  assert.equal(sectionEl.innerHTML, '');
+});
+
 test('renders a mobiledoc with default image section', (assert) => {
   assert.expect(3);
   let cardName = 'image';