search

ceramicnetwork / rust-ceramic

Implementation of the Ceramic protocol in Rust
Other
37 stars 10 forks source link

chore: Change EventInsertable to store the parsed Event object instead of its raw blocks #483

Closed stbrody closed 2 months ago

stbrody commented 2 months ago

builds on https://github.com/ceramicnetwork/rust-ceramic/pull/481

stbrody commented 2 months ago

This is passing all tests except the migration tests, I might need a hand to figure out what's up with the migration tests.

stbrody commented 2 months ago

Okay I figured out a part of what's wrong with the migration tests - there are some event CAR files being used in that test that don't round-trip through unvalidated::Event. Pushed a change that demonstrates that. Now the question is: is this a bug in our serialization/deserialization logic? Or is the migration test using invalid CAR files?

@nathanielc