old me just proved that it all works as expected

crates/re_sdk/src/recording_stream.rs

@@ -1144,21 +1144,7 @@ mod tests {
                 _ => panic!("expected SetStoreInfo"),
             }
 
-            // The underlying batcher is never flushing: there's nothing else.
-            assert!(msgs.pop().is_none());
-        }
-
-        // The underlying batcher is never flushing: there's nothing else.
-        assert!(storage.take().is_empty());
-
-        rec_stream.flush_blocking(); // flush the entire hierarchy
-
-        {
-            let mut msgs = {
-                let mut msgs = storage.take();
-                msgs.reverse();
-                msgs
-            };
+            // MemorySinkStorage transparently handles flushing during `take()`!
 
             // The batched table itself, which was sent as a result of the explicit flush above.
             match msgs.pop().unwrap() {