more feature flags

Author: Frando

src/ranger.rs

@@ -288,6 +288,7 @@ pub trait Store<E: RangeEntry>: Sized {
     ///
     /// This will remove just the entry with the given key, but will not perform prefix deletion.
     #[cfg(test)]
+    #[allow(unused)]
     fn entry_remove(&mut self, key: &E::Key) -> Result<Option<E>, Self::Error>;
 
     /// Remove all entries whose key start with a prefix and for which the `predicate` callback

src/store/fs.rs

@@ -984,13 +984,12 @@ fn into_entry(key: RecordsId, value: RecordsValue) -> SignedEntry {
     SignedEntry::new(entry_signature, entry)
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "fs-store"))]
 mod tests {
-    use super::{tables::LATEST_PER_AUTHOR_TABLE, *};
+    use super::*;
     use crate::ranger::Store as _;
 
     #[tokio::test]
-    #[cfg(feature = "fs-store")]
     async fn test_ranges() -> Result<()> {
         let dbfile = tempfile::NamedTempFile::new()?;
         let mut store = Store::persistent(dbfile.path())?;
@@ -1018,7 +1017,6 @@ mod tests {
     }
 
     #[test]
-    #[cfg(feature = "fs-store")]
     fn test_basics() -> Result<()> {
         let dbfile = tempfile::NamedTempFile::new()?;
         let mut store = Store::persistent(dbfile.path())?;
@@ -1112,6 +1110,7 @@ mod tests {
     #[tokio::test]
     #[cfg(feature = "fs-store")]
     async fn test_migration_001_populate_latest_table() -> Result<()> {
+        use super::tables::LATEST_PER_AUTHOR_TABLE;
         let dbfile = tempfile::NamedTempFile::new()?;
         let namespace = NamespaceSecret::new(&mut rand::rng());
 

tests/gc.rs

@@ -1,3 +1,5 @@
+#![allow(unused)]
+
 use std::{path::PathBuf, time::Duration};
 
 use anyhow::Result;
@@ -18,6 +20,7 @@ pub fn create_test_data(size: usize) -> Bytes {
 }
 
 /// Wrap a bao store in a node that has gc enabled.
+#[cfg(feature = "fs-store")]
 async fn persistent_node(
     path: PathBuf,
     gc_period: Duration,
@@ -35,6 +38,7 @@ async fn persistent_node(
 }
 
 #[tokio::test]
+#[cfg(feature = "fs-store")]
 async fn redb_doc_import_stress() -> Result<()> {
     let _ = tracing_subscriber::fmt::try_init();
     let dir = testdir!();

tests/sync.rs

@@ -17,6 +17,7 @@ use iroh_docs::{
 };
 use n0_future::time::Instant;
 use rand::{CryptoRng, Rng, SeedableRng};
+#[cfg(feature = "fs-store")]
 use tempfile::tempdir;
 use tracing::{debug, error_span, info, Instrument};
 use tracing_test::traced_test;
@@ -584,6 +585,7 @@ async fn test_sync_via_relay() -> Result<()> {
 #[tokio::test]
 #[traced_test]
 #[ignore = "flaky"]
+#[cfg(feature = "fs-store")]
 async fn sync_restart_node() -> Result<()> {
     let mut rng = test_rng(b"sync_restart_node");
     let (relay_map, _relay_url, _guard) = iroh::test_utils::run_relay_server().await?;
@@ -1156,6 +1158,7 @@ impl PartialEq<ExpectedEntry> for (Entry, Bytes) {
 
 #[tokio::test]
 #[traced_test]
+#[cfg(feature = "fs-store")]
 async fn doc_delete() -> Result<()> {
     let tempdir = tempdir()?;
     // TODO(Frando): iroh-blobs has gc only for fs store atm, change test to test both

tests/util.rs

@@ -1,4 +1,5 @@
-#![allow(dead_code)]
+#![allow(unused)]
+
 use std::{
     net::{Ipv4Addr, Ipv6Addr, SocketAddrV4, SocketAddrV6},
     ops::Deref,