Radish alpha
r
rad:zwTxygwuz5LDGBq255RA2CbNGrz8
Radicle CI broker
Radicle
Git
refactor(src/broker.rs): drop unused per-repository adapters
Merged liw opened 1 year ago

We don’t make any actual use of this so off it goes.

Signed-off-by: Lars Wirzenius liw@liw.fi

1 file changed +1 -37 558cc34c c89b471f
modified src/broker.rs
@@ -60,11 +60,7 @@ impl Broker {
        self.default_adapter.as_ref()
    }

-
    pub fn set_repository_adapter(&mut self, rid: &RepoId, adapter: &Adapter) {
-
        self.adapters.insert(*rid, adapter.clone());
-
    }
-

-
    pub fn adapter(&self, rid: &RepoId) -> Option<&Adapter> {
+
    fn adapter(&self, rid: &RepoId) -> Option<&Adapter> {
        self.adapters.get(rid).or(self.default_adapter.as_ref())
    }

@@ -222,11 +218,6 @@ mod test {
        Ok(RepoId::from_urn(RID)?)
    }

-
    fn rid2() -> anyhow::Result<RepoId> {
-
        const RID: &str = "rad:z3gqcJUoA1n9HaHKufZs5FCSGazv5";
-
        Ok(RepoId::from_urn(RID)?)
-
    }
-

    #[test]
    fn has_no_adapters_initially() -> TestResult<()> {
        let tmp = tempdir()?;
@@ -238,33 +229,6 @@ mod test {
    }

    #[test]
-
    fn adds_adapter() -> TestResult<()> {
-
        let tmp = tempdir()?;
-
        let db = tmp.path().join("db.db");
-
        let mut broker = broker(&db)?;
-

-
        let adapter = Adapter::default();
-
        let rid = rid()?;
-
        broker.set_repository_adapter(&rid, &adapter);
-
        assert_eq!(broker.adapter(&rid), Some(&adapter));
-
        Ok(())
-
    }
-

-
    #[test]
-
    fn does_not_find_unknown_repo() -> TestResult<()> {
-
        let tmp = tempdir()?;
-
        let db = tmp.path().join("db.db");
-
        let mut broker = broker(&db)?;
-

-
        let adapter = Adapter::default();
-
        let rid = rid()?;
-
        let rid2 = rid2()?;
-
        broker.set_repository_adapter(&rid, &adapter);
-
        assert_eq!(broker.adapter(&rid2), None);
-
        Ok(())
-
    }
-

-
    #[test]
    fn does_not_have_a_default_adapter_initially() -> TestResult<()> {
        let tmp = tempdir()?;
        let db = tmp.path().join("db.db");