liftedinit / many-rs

Rust libraries for the Many Protocol
Apache License 2.0
5 stars 11 forks source link

fix!: `web.update` storage batch apply #416

Closed fmorency closed 11 months ago

fmorency commented 11 months ago

merk doesn't support applying b1 and b2 where

E.g.

let b1 = vec![(b"key".to_vec(), Op::Delete)];
storage.apply(&b1)?;
let b2 = vec![(b"key".to_vec(), Op::Put(b"value".to_vec()))];
storage.apply(&b2)?;
storage.commit(&[])?;

The above doesn't work. The Put operation in b2 is ignored.

Relates https://github.com/liftedinit/ghostcloud/issues/40

codecov[bot] commented 11 months ago

Codecov Report

Merging #416 (c3b9ec7) into main (60eefa9) will decrease coverage by 0.03%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #416      +/-   ##
==========================================
- Coverage   81.83%   81.80%   -0.03%     
==========================================
  Files         149      149              
  Lines       16811    16785      -26     
==========================================
- Hits        13757    13731      -26     
  Misses       3054     3054              
Files Coverage Δ
src/many-web/src/storage.rs 74.57% <100.00%> (-1.76%) :arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more