Event recovery does not record last_version correctly

[jrudolph]

Instead it always sets the last_version field to None in

evented/src/entity.rs

Line 147 in 9d62331

last_version: None,

so that when an entity is reinstantiated it will fail after handling the next command in

evented/src/entity.rs

Line 318 in 9d62331

return Err(Error::UnexpectedVersion(version, last_version));

[jrudolph]

Here's a diff for the tests that makes it fail:

diff --git a/src/entity.rs b/src/entity.rs
index 6f6a367..7728e73 100644
--- a/src/entity.rs
+++ b/src/entity.rs
@@ -625,7 +625,7 @@ mod tests {
 
         let id = Uuid::from_u128(0);
 
-        let mut counter = Counter::default().entity().build(id, pool).await?;
+        let mut counter = Counter::default().entity().build(id, pool.clone()).await?;
         assert_eq!(counter.entity, Counter(0));
 
         let result = counter.handle_command(Decrease(1)).await?;
@@ -633,8 +633,12 @@ mod tests {
 
         let result = counter.handle_command(Increase(40)).await?;
         assert_eq!(result, Ok(&Counter(40)));
+
         let result = counter.handle_command(Decrease(20)).await?;
         assert_eq!(result, Ok(&Counter(20)));
+
+        // reinstantiate the counter (to test that event recovery works)
+        let mut counter = Counter::default().entity().build(id, pool).await?;
         let result = counter.handle_command(Increase(22)).await?;
         assert_eq!(result, Ok(&Counter(42)));

[jrudolph]

Btw. what's the difference between version and seq_no?

[jrudolph]

Ok, seq_no seems to be the global sequence number across all entities of all types.