refactor: update ArtifactLocation enum to use an internally tagged representation

dlachaume · dlachaume · commit 92d63e33dcb0 · 2024-12-04T16:11:16.000+01:00
diff --git a/mithril-common/src/entities/cardano_database.rs b/mithril-common/src/entities/cardano_database.rs
@@ -52,9 +52,10 @@ impl CardanoDatabaseSnapshot {
 }
 
 #[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]
-#[serde(rename_all = "snake_case")]
+#[serde(rename_all = "snake_case", tag = "type")]
 pub enum ArtifactLocation {
-    Aggregator(String),
+    Aggregator { uri: String },
+    CloudStorage { uri: String },
 }
 
 /// Locations of the Cardano database related files.

Original file line number	Diff line number	Diff line change
`@@ -52,9 +52,10 @@ impl CardanoDatabaseSnapshot {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`#[derive(Clone, Debug, PartialEq, Eq, Serialize, Deserialize)]`
`55`		`-#[serde(rename_all = "snake_case")]`
	`55`	`+#[serde(rename_all = "snake_case", tag = "type")]`
`56`	`56`	`pub enum ArtifactLocation {`
`57`		`- Aggregator(String),`
	`57`	`+ Aggregator { uri: String },`
	`58`	`+ CloudStorage { uri: String },`
`58`	`59`	`}`
`59`	`60`
`60`	`61`	`/// Locations of the Cardano database related files.`