Merge #1520: Remove integration tests from torrent-repository package

1472c8e refactor: [#1504] remove unneded trait implementationis for Swarm (Jose Celano)
c2dabb2 chore: [#1504] remove uneeded fn attribute (Jose Celano)
8d3b948 tests: [#1504] remove integration tests from torrent-repository pacakge (Jose Celano)

Pull request description:

  All features are now covered by unit tests.

  I have also removed unused trait implementations for the `Swarm` type. If we need them in the future, we should extract a private type to use derived implementations:

  From:

  ```rust
  #[derive(Clone)]
  pub struct Swarm {
      info_hash: InfoHash,
      peers: BTreeMap<SocketAddr, Arc<PeerAnnouncement>>,
      metadata: SwarmMetadata,
      event_sender: Sender,
  }
  ```

  To:

  ```rust
  #[derive(Clone)]
  pub struct Swarm {
      data: SwarmData,
      event_sender: Sender,
  }

  #[derive(Clone, Debug, Display, Hash, PartialEq, Eq)]
  pub struct SwarmData {
      info_hash: InfoHash,
      peers: BTreeMap<SocketAddr, Arc<PeerAnnouncement>>,
      metadata: SwarmMetadata,
  }
  ```

ACKs for top commit:
  josecelano:
    ACK 1472c8e

Tree-SHA512: f350957ed47f234e41762e523bef66e337176e91011c3fbed4219a10bae44cca11a29e7736392f759d1d67f298553ec0c4a2960bed6817f3035cea8acc0beccf

Author: josecelano

packages/torrent-repository/src/swarm.rs

@@ -1,8 +1,6 @@
 //! A swarm is a collection of peers that are all trying to download the same
 //! torrent.
 use std::collections::BTreeMap;
-use std::fmt::Debug;
-use std::hash::{Hash, Hasher};
 use std::net::SocketAddr;
 use std::sync::Arc;
 
@@ -24,31 +22,6 @@ pub struct Swarm {
     event_sender: Sender,
 }
 
-#[allow(clippy::missing_fields_in_debug)]
-impl Debug for Swarm {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.debug_struct("Swarm")
-            .field("peers", &self.peers)
-            .field("metadata", &self.metadata)
-            .finish()
-    }
-}
-
-impl Hash for Swarm {
-    fn hash<H: Hasher>(&self, state: &mut H) {
-        self.peers.hash(state);
-        self.metadata.hash(state);
-    }
-}
-
-impl PartialEq for Swarm {
-    fn eq(&self, other: &Self) -> bool {
-        self.peers == other.peers && self.metadata == other.metadata
-    }
-}
-
-impl Eq for Swarm {}
-
 impl Swarm {
     #[must_use]
     pub fn new(info_hash: &InfoHash, downloaded: u32, event_sender: Sender) -> Self {
@@ -329,16 +302,6 @@ mod tests {
     use crate::swarm::Swarm;
     use crate::tests::sample_info_hash;
 
-    #[test]
-    fn it_should_allow_debugging() {
-        let swarm = Swarm::new(&sample_info_hash(), 0, None);
-
-        assert_eq!(
-            format!("{swarm:?}"),
-            "Swarm { peers: {}, metadata: SwarmMetadata { downloaded: 0, complete: 0, incomplete: 0 } }"
-        );
-    }
-
     #[test]
     fn it_should_be_empty_when_no_peers_have_been_inserted() {
         let swarm = Swarm::new(&sample_info_hash(), 0, None);

packages/torrent-repository/src/swarms.rs

@@ -49,7 +49,6 @@ impl Swarms {
     /// # Errors
     ///
     /// This function panics if the lock for the swarm handle cannot be acquired.
-    #[allow(clippy::await_holding_lock)]
     pub async fn handle_announcement(
         &self,
         info_hash: &InfoHash,