fix(touch_storage): remove overhead

src/main.rs

@@ -28,7 +28,7 @@ use crate::xml::storage::touch_storage;
 
 #[tokio::main]
 async fn main() {
-    touch_storage(Some("fakehub.xml")).unwrap();
+    touch_storage(Some("fakehub.xml"));
     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

@@ -19,18 +19,17 @@
 // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 // SOFTWARE.
-use std::fmt::Error;
 use std::fs::File;
 
 use log::info;
 
-pub fn touch_storage(path: Option<&str>) -> Result<File, Error> {
+pub fn touch_storage(path: Option<&str>) -> File {
     let location = path.unwrap_or("fakehub.xml");
     info!("Initializing XML storage: {location}");
     match File::create(location) {
         Ok(file) => {
             info!("'{location}' initialized");
-            Ok(file)
+            file
         }
         Err(err) => {
             panic!("fakehub storage failed to initialize in '{location}': {err}")
@@ -49,7 +48,7 @@ mod tests {
         let temp = TempDir::new("temp")?;
         let path = temp.path().join("fakehub.xml");
         let storage = path.to_str();
-        touch_storage(storage).unwrap();
+        touch_storage(storage);
         assert!(
             path.exists(),
             "storage file {:?} was not created, but should be",
@@ -63,7 +62,7 @@ mod tests {
         let temp = TempDir::new("temp")?;
         let path = temp.path().join("test.xml");
         let storage = path.to_str();
-        touch_storage(storage).unwrap();
+        touch_storage(storage);
         assert!(
             path.exists(),
             "storage file {:?} was not created, but should be",