Merge pull request iden3#49 from krlosMata/feature/fix-smt-update

fix smt update
nearnadokeyser · Aug 31, 2020 · d422471 · d422471
2 parents 01da5f9 + 7e93b3f
commit d422471
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/smt.js b/src/smt.js
@@ -58,10 +58,10 @@ class SMT {
 
         res.newRoot = rtNew;
 
+        await this.db.multiDel(dels);
         await this.db.multiIns(ins);
         await this.db.setRoot(rtNew);
         this.root = rtNew;
-        await this.db.multiDel(dels);
 
         return res;
     }

diff --git a/test/smtjs.js b/test/smtjs.js
@@ -161,4 +161,13 @@ describe("SMT Javascript test", function () {
         assert(Fr.eq(tree1.root, tree2.root));
     });
 
+    it("Should test update with same key-value", async () => {
+        const tree1 = await smt.newMemEmptyTrie();
+
+        await tree1.insert(8,88);
+        await tree1.update(8,88);
+
+        const res = await tree1.db.get(tree1.root);
+        assert.notEqual(res, undefined);
+    });
 });