refactor: [torrust#1495] make TrackedTorrent fields private

Author: josecelano

packages/torrent-repository/src/entry/torrent.rs

@@ -19,15 +19,20 @@ use super::swarm::Swarm;
 #[derive(Clone, Debug, Default, PartialEq, Eq, PartialOrd, Ord, Hash)]
 pub struct TrackedTorrent {
     /// A network of peers that are all trying to download the torrent.
-    pub(crate) swarm: Swarm,
+    swarm: Swarm,
 
     /// The number of peers that have ever completed downloading the torrent.
     /// This value is can be persistent so it's loaded from the database when
     /// the tracker starts.
-    pub(crate) downloaded: u32,
+    downloaded: u32,
 }
 
 impl TrackedTorrent {
+    #[must_use]
+    pub fn new(swarm: Swarm, downloaded: u32) -> Self {
+        Self { swarm, downloaded }
+    }
+
     #[must_use]
     pub fn get_swarm_metadata(&self) -> SwarmMetadata {
         let metadata = self.swarm.metadata();

packages/torrent-repository/src/repository.rs

@@ -51,13 +51,7 @@ impl TorrentRepository {
             tracing::debug!("Inserting new torrent: {:?}", info_hash);
 
             let new_entry = if let Some(number_of_downloads) = opt_persistent_torrent {
-                TrackedTorrentHandle::new(
-                    TrackedTorrent {
-                        swarm: Swarm::default(),
-                        downloaded: number_of_downloads,
-                    }
-                    .into(),
-                )
+                TrackedTorrentHandle::new(TrackedTorrent::new(Swarm::default(), number_of_downloads).into())
             } else {
                 TrackedTorrentHandle::default()
             };
@@ -235,13 +229,7 @@ impl TorrentRepository {
                 continue;
             }
 
-            let entry = TrackedTorrentHandle::new(
-                TrackedTorrent {
-                    swarm: Swarm::default(),
-                    downloaded: *completed,
-                }
-                .into(),
-            );
+            let entry = TrackedTorrentHandle::new(TrackedTorrent::new(Swarm::default(), *completed).into());
 
             // Since SkipMap is lock-free the torrent could have been inserted
             // after checking if it exists.