use path instead of pathbuf

powerd6 · Jan 5, 2024 · 6b3ff29 · 6b3ff29
1 parent e591dee
commit 6b3ff29
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/src/file_system.rs b/src/file_system.rs
@@ -1,11 +1,11 @@
-use std::path::PathBuf;
+use std::path::{PathBuf, Path};
 
 use mockall::automock;
 
 #[automock]
 pub(crate) trait FileSystem {
-    fn dir_exists(&self, path: &PathBuf) -> bool;
-    fn create_dir(&self, path: &PathBuf) -> Result<PathBuf, FileSystemError>;
+    fn dir_exists(&self, path: &Path) -> bool;
+    fn create_dir(&self, path: &Path) -> Result<PathBuf, FileSystemError>;
 }
 
 #[derive(Debug)]

diff --git a/src/file_system/real_file_system.rs b/src/file_system/real_file_system.rs
@@ -4,20 +4,21 @@ use std::fs::create_dir_all;
 
 use std::fs::metadata;
 
+use std::path::Path;
 use std::path::PathBuf;
 
 use super::FileSystem;
 
 pub(crate) struct RealFileSystem;
 
 impl FileSystem for RealFileSystem {
-    fn dir_exists(&self, path: &PathBuf) -> bool {
+    fn dir_exists(&self, path: &Path) -> bool {
         metadata(path).map_or(false, |metadata| metadata.is_dir())
     }
 
-    fn create_dir(&self, path: &PathBuf) -> Result<PathBuf, FileSystemError> {
+    fn create_dir(&self, path: &Path) -> Result<PathBuf, FileSystemError> {
         match create_dir_all(path) {
-            Ok(_) => Ok(path.clone()),
+            Ok(_) => Ok(PathBuf::from(path)),
             Err(_) => Err(FileSystemError::UnableToCreateDirectory),
         }
     }