Set the renderTree renderNode element for a list item

fixes #130

src/js/editor/post.js

@@ -561,10 +561,10 @@ class PostEditor {
    *
    *     const range = editor.cursor.offsets;
    *     const markup = markerRange.headMarker.markups[0];
-   *     editor.run((postEditor) => {
+   *     editor.run(postEditor => {
    *       postEditor.removeMarkupFromRange(range, markup);
    *     });
-   *     // Will result some markers possibly being split, and the markup
+   *     // Will result in some markers possibly being split, and the markup
    *     // being removed from all markers between the split.
    *
    * The return value will be all markers between the split, the same return

src/js/renderers/editor-dom.js

@@ -190,6 +190,7 @@ class Visitor {
     renderNode.element = element;
 
     attachRenderNodeElementToDOM(renderNode, element, null);
+    renderNode.renderTree.elements.set(element, renderNode);
 
     if (item.markers.length) {
       const visitAll = true;

tests/acceptance/editor-list-test.js

@@ -387,3 +387,29 @@ test('forward-delete end of li with markup section after', (assert) => {
   Helpers.dom.insertText(editor, 'X');
   assert.hasElement('#editor li:contains(abcXdef)', 'inserts text at right spot');
 });
+
+// see https://github.com/bustlelabs/content-kit-editor/issues/130
+test('selecting empty list items does not cause error', (assert) => {
+  const mobiledoc = Helpers.mobiledoc.build(builder => {
+    const {post, listSection, listItem, marker} = builder;
+    return post([
+      listSection('ul', [
+        listItem([marker('abc')]),
+        listItem(),
+        listItem([marker('def')])
+      ])
+    ]);
+  });
+
+  createEditorWithMobiledoc(mobiledoc);
+
+  assert.equal($('#editor li').length, 3, 'precond - 3 lis');
+  Helpers.dom.moveCursorTo($('#editor li:eq(1)')[0], 0,
+                           $('#editor li:eq(2)')[0], 0);
+  Helpers.dom.triggerEvent(editorElement, 'keyup');
+  assert.ok(true, 'no error');
+
+  Helpers.dom.insertText(editor, 'X');
+  assert.hasElement('#editor li:contains(Xdef)', 'insert text');
+  assert.equal($('#editor li').length, 2, 'inserting text deletes selected li');
+});