Conversation
|
🚅 Deployed to the rivet-pr-4314 environment in rivet-frontend
|
PR Review: chore: fix testsOverall this is a clean set of test fixes adapting to updated KV and workflow APIs. Changes are well-scoped and the intent is clear. Issues1. Two // Current (kv_operations.rs:259-262)
let size = db
.run(|tx| async move { kv::estimate_kv_size(&tx, actor_id).await })
.await
.unwrap(); // should be ?
// Better
let size = db
.run(|tx| async move { kv::estimate_kv_size(&tx, actor_id).await })
.await?;2.
Minor Notes
|
460c180 to
1f8ab8e
Compare
51558a0 to
165fa47
Compare
1f8ab8e to
ea391b1
Compare
165fa47 to
d33b11b
Compare
d33b11b to
a1e9af4
Compare
ea391b1 to
4aa5dca
Compare
4aa5dca to
0a28359
Compare

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Checklist: