chore: Don't require futures to be Sync

wnfs-wg · Jan 1, 2024 · d0d29e3 · d0d29e3
1 parent 7310ed2
commit d0d29e3
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 8 deletions.
diff --git a/car-mirror/src/common.rs b/car-mirror/src/common.rs
@@ -472,26 +472,29 @@ impl Default for Config {
 
 #[cfg(test)]
 mod tests {
+    use wnfs_common::MemoryBlockStore;
+
     use super::*;
-    use crate::test_utils::assert_cond_send_sync;
+    use crate::{test_utils::assert_cond_send_sync, traits::NoCache};
 
-    fn send_sync_tests() {
+    #[allow(clippy::unreachable, unused)]
+    fn test_assert_send() {
         assert_cond_send_sync(|| {
             block_send(
                 unimplemented!(),
                 unimplemented!(),
                 unimplemented!(),
-                unimplemented!(),
-                unimplemented!(),
+                unimplemented!() as &MemoryBlockStore,
+                &NoCache,
             )
         });
         assert_cond_send_sync(|| {
             block_receive(
                 unimplemented!(),
                 unimplemented!(),
                 unimplemented!(),
-                unimplemented!(),
-                unimplemented!(),
+                unimplemented!() as &MemoryBlockStore,
+                &NoCache,
             )
         })
     }

diff --git a/car-mirror/src/test_utils/local_utils.rs b/car-mirror/src/test_utils/local_utils.rs
@@ -5,7 +5,7 @@ use anyhow::Result;
 use futures::TryStreamExt;
 use libipld::{Cid, Ipld};
 use proptest::strategy::Strategy;
-use wnfs_common::{utils::CondSync, BlockStore, MemoryBlockStore};
+use wnfs_common::{utils::CondSend, BlockStore, MemoryBlockStore};
 
 #[derive(Clone, Debug)]
 pub(crate) struct Metrics {
@@ -92,4 +92,4 @@ pub(crate) async fn total_dag_blocks(root: Cid, store: &impl BlockStore) -> Resu
         .len())
 }
 
-pub(crate) fn assert_cond_send_sync<T: CondSync>(fut: fn() -> T) {}
+pub(crate) fn assert_cond_send_sync<T: CondSend>(_fut: fn() -> T) {}