Node with loadOnDemand which has 0 children doesn't update the state …

…with openNodes (#25) * Fixed the regression in addChildNodes * wip * rebuilt dist & examples * applied feedback * fix for addChildNodes on empty parent node * merged upstream/master * fix * updated docs * added more assertions
cheton · Jan 31, 2018 · 2bf9482 · 2bf9482
1 parent 7bca7c6
commit 2bf9482
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 2 deletions.
diff --git a/dist/infinite-tree.js b/dist/infinite-tree.js
@@ -1983,6 +1983,7 @@ var InfiniteTree = function (_events$EventEmitter) {
 
                     if (nodes.length === 0 && currentNodeIndex >= 0) {
                         node.state.open = true;
+                        _this6.state.openNodes = [node].concat(_this6.state.openNodes);
                     }
 
                     if (err || nodes.length === 0) {

diff --git a/dist/infinite-tree.min.js b/dist/infinite-tree.min.js
diff --git a/docs/examples.js b/docs/examples.js
diff --git a/src/infinite-tree.js b/src/infinite-tree.js
@@ -1069,6 +1069,7 @@ class InfiniteTree extends events.EventEmitter {
 
                     if (nodes.length === 0 && currentNodeIndex >= 0) {
                         node.state.open = true;
+                        this.state.openNodes = [node].concat(this.state.openNodes);
                     }
 
                     if (err || nodes.length === 0) {

diff --git a/test/index.js b/test/index.js
@@ -143,7 +143,9 @@ test('loadOnDemand', (t) => {
             setTimeout(() => {
                 next(null, [], () => {
                     t.equal(tree.getNodeById('<root>').getChildren().length, 0);
+                    t.equal(tree.state.openNodes.length, 1);
                     t.equal(node.state.open, true);
+                    t.end();
                 });
             }, 250);
 
@@ -152,6 +154,7 @@ test('loadOnDemand', (t) => {
                 const data = getTreeData();
                 next(null, data.children, () => {
                     t.equal(tree.getNodeById('<root>').getChildren().length, 2);
+                    t.equal(tree.state.openNodes.length, 1);
                     t.end();
                 });
             }, 500);