feat(#9): anyhow, no clean, option

src/main.rs

@@ -28,7 +28,7 @@ use crate::xml::storage::touch_storage;
 
 #[tokio::main]
 async fn main() {
-    touch_storage().unwrap();
+    touch_storage(Some("fakehub.xml")).unwrap();
     let app = Router::new().route("/", get(home::home));
     let listener = tokio::net::TcpListener::bind("0.0.0.0:3000").await.unwrap();
     axum::serve(listener, app).await.unwrap();

src/xml/storage.rs

@@ -25,7 +25,7 @@ use std::fs::File;
 use log::info;
 
 pub fn touch_storage(name: Option<&str>) -> Result<File, Error> {
-    let path = "fakehub.xml";
+    let path = name.unwrap_or("fakehub.xml");
     info!("Initializing XML storage: {path}");
     let creating = File::create(path);
     match creating {
@@ -41,24 +41,31 @@ pub fn touch_storage(name: Option<&str>) -> Result<File, Error> {
 
 #[cfg(test)]
 mod tests {
-    use std::fs;
     use std::path::Path;
 
     use crate::xml::storage::touch_storage;
 
-    fn clean() {
-        fs::remove_file("fakehub.xml").unwrap();
-    }
-
     #[test]
-    fn creates_xml_storage() {
-        touch_storage().unwrap();
+    fn creates_xml_storage() -> anyhow::Result<()> {
+        touch_storage(None).unwrap();
         let storage = "fakehub.xml";
         let exists = Path::new(storage).exists();
         assert!(
             exists,
-            "storage file '{storage}' was not created, but should"
+            "storage file '{storage}' was not created, but should be"
+        );
+        Ok(())
+    }
+
+    #[test]
+    fn creates_xml_storage_with_different_name() -> anyhow::Result<()> {
+        let path = "test.xml";
+        touch_storage(Some(path)).unwrap();
+        let exists = Path::new(path).exists();
+        assert!(
+            exists,
+            "storage file '{path}' was not created, but should be"
         );
-        clean();
+        Ok(())
     }
 }