chore: appease clippy and cargo check

axodotdev · Apr 26, 2023 · f026266 · f026266
1 parent 8d8dc5b
commit f026266
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/src/local.rs b/src/local.rs
@@ -147,13 +147,11 @@ impl LocalAsset {
                     details,
                 });
             }
-        } else {
-            if let Err(details) = fs::remove_file(&dest_path) {
-                return Err(AxoassetError::LocalAssetRemoveFailed {
-                    dest_path: dest_path.display().to_string(),
-                    details,
-                });
-            }
+        } else if let Err(details) = fs::remove_file(&dest_path) {
+            return Err(AxoassetError::LocalAssetRemoveFailed {
+                dest_path: dest_path.display().to_string(),
+                details,
+            });
         }
 
         Ok(())

diff --git a/tests/local_remove.rs b/tests/local_remove.rs
@@ -7,12 +7,12 @@ fn it_removes_both_file_and_directory() {
     let file_path = Path::new(&dest.as_os_str()).join("subdir").join("test.md");
     let dir_path = Path::new(&dest.as_os_str()).join("subdir");
 
-    fs::create_dir_all(&file_path.parent().unwrap());
+    fs::create_dir_all(&file_path.parent().unwrap()).unwrap();
     fs::write(&file_path, "hello").unwrap();
 
-    axoasset::LocalAsset::remove(&file_path.display().to_string());
+    axoasset::LocalAsset::remove(&file_path.display().to_string()).unwrap();
     assert!(!file_path.exists());
 
-    axoasset::LocalAsset::remove(&dir_path.display().to_string());
+    axoasset::LocalAsset::remove(&dir_path.display().to_string()).unwrap();
     assert!(!dir_path.exists());
 }