refactor: use specific types for each ArtifactsLocations field

Author: dlachaume

mithril-aggregator/src/artifact_builder/cardano_database.rs

@@ -199,11 +199,7 @@ mod tests {
             "merkleroot".to_string(),
             beacon,
             expected_total_size,
-            ArtifactsLocations {
-                digest: vec![],
-                immutables: vec![],
-                ancillary: vec![],
-            },
+            ArtifactsLocations::default(),
             CompressionAlgorithm::Zstandard,
             &Version::parse("1.0.0").unwrap(),
         );

mithril-common/src/entities/cardano_database.rs

@@ -53,20 +53,32 @@ impl CardanoDatabaseSnapshot {
 
 #[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
 #[serde(rename_all = "snake_case", tag = "type")]
-pub enum ArtifactLocation {
+enum DigestLocation {
     Aggregator { uri: String },
     CloudStorage { uri: String },
 }
 
+#[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(rename_all = "snake_case", tag = "type")]
+enum ImmutablesLocation {
+    CloudStorage { uri: String },
+}
+
+#[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
+#[serde(rename_all = "snake_case", tag = "type")]
+enum AncillaryLocation {
+    CloudStorage { uri: String },
+}
+
 /// Locations of the Cardano database related files.
 #[derive(Clone, Debug, Default, PartialEq, Eq, Serialize, Deserialize)]
 pub struct ArtifactsLocations {
     /// Locations of the the immutable file digests.
-    pub digest: Vec<ArtifactLocation>,
+    digest: Vec<DigestLocation>,
     /// Locations of the immutable files.
-    pub immutables: Vec<ArtifactLocation>,
-    /// Locations of the ancillary files (ledger and volatile).
-    pub ancillary: Vec<ArtifactLocation>,
+    immutables: Vec<ImmutablesLocation>,
+    /// Locations of the ancillary files.
+    ancillary: Vec<AncillaryLocation>,
 }
 
 #[typetag::serde]