Merge pull request #383 from wuerthcs/bugfix/NODEJS-633-cannot-read-property-executor

Fix for NODEJS-633

Author: matchish

lib/mapping/tree.js

@@ -109,6 +109,9 @@ class Tree extends Node {
       this._onItemAdded();
       return value;
     }
+    if (node.value === null && node.edges.length > 0) {
+      node.value = valueHandler();
+    }
     return node.value;
   }
 

test/unit/mapping/tree-tests.js

@@ -67,6 +67,14 @@ describe('Tree', function () {
       const k = assertNode(j.edges[0], [ 'k' ], 8, 1);
       assertNode(k.edges[0], [ 'l' ], 6, 0);
     });
+
+    it('should set value if a node not a leaf and it is a match', () => {
+      const tree = new Tree();
+      tree.getOrCreate(['a', 'b', 'c'], () => true);
+      tree.getOrCreate(['a', 'b', 'd'], () => true);
+      assert.strictEqual(tree.getOrCreate(['a', 'b'], () => true), true);
+    });
+
   });
 });