Preserve parent nodes when removing data

Fix #108
goldibex · Dec 23, 2016 · 4f06d3c · 4f06d3c
1 parent 52a6e97
commit 4f06d3c
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 8 deletions.
diff --git a/lib/database/store.js b/lib/database/store.js
@@ -355,14 +355,6 @@ class DataNode {
 
     paths.split(path).every(key => {
       const siblings = Object.keys(currSrc).filter(k => k !== key);
-
-      // if the only item is the one to remove,
-      // abort iteration.
-      if (siblings.length === 0) {
-        return false;
-      }
-
-      // Or copy other items
       const currDest = dest();
 
       siblings.forEach(k => {

diff --git a/test/spec/lib/database/store.js b/test/spec/lib/database/store.js
@@ -313,6 +313,22 @@ describe('store', function() {
       expect(newRoot.$value()).to.deep.equal(data.$value());
     });
 
+    it('should copy parent nodes', function() {
+      data = store.create({
+        a: {
+          b: {
+            c: true,
+            d: true
+          },
+          e: true
+        }
+      });
+
+      const newRoot = data.$remove('a/b/c');
+
+      expect(newRoot.$value()).to.eql({a: {b: {d: true}, e: true}});
+    });
+
   });
 
   describe('#$child', function() {