oxidecomputer / crucible

A storage service.
Mozilla Public License 2.0
166 stars 17 forks source link

Remove optionality for `BlockRes` in `DeferredWrite` #1441

Closed mkeeter closed 3 weeks ago

mkeeter commented 3 weeks ago

The Option<BlockRes> is always Some(..), except in unit testing, so it has too many degrees of freedom during normal operation. This PR turns it into a plain BlockRes, and adds a #[cfg(test)] helper function to make an empty BlockRes during unit tests.