Fixed bug in sdf mv that occurred in addressing issue #14.

 - New `test_mv()` function to test mv and catch future bugs.
 - Fixed the bug, in which the wrong variable was used.

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "scidataflow"
-version = "0.8.8"
+version = "0.8.9"
 edition = "2021"
 exclude = ["logo.png", "tests/test_data/**"]
 license = "MIT"

src/lib/project.rs

@@ -482,10 +482,10 @@ impl Project {
             }
 
             // move the actual file
-            rename(source, destination_path).context("Error encountered when moving file.")?;
+            rename(source, &destination_path).context("Error encountered when moving file.")?;
 
             // update the relative path
-            let relative_destination = self.relative_path_string(Path::new(destination))?;
+            let relative_destination = self.relative_path_string(destination_path.as_path())?;
 
             // modify the DataFile
             let mut new_file = file.clone();

tests/test_project.rs

@@ -13,6 +13,7 @@ mod tests {
     use super::get_statuses;
     use super::setup;
     use scidataflow::lib::data::LocalStatusCode;
+    use std::fs;
     use std::path::PathBuf;
 
     #[tokio::test]
@@ -160,4 +161,51 @@ mod tests {
             }
         }
     }
+
+    #[tokio::test]
+    async fn test_mv() {
+        let mut fixture = setup(false).await;
+        let path_context = fixture.project.path_context();
+        let statuses = get_statuses(&mut fixture, &path_context).await;
+
+        // at this point the status should be empty
+        assert!(statuses.is_empty());
+
+        // get the files to add
+        let files = &fixture.env.files.as_ref().unwrap();
+        let add_files: Vec<String> = files
+            .into_iter()
+            .filter(|f| f.add)
+            .map(|f| f.path.clone())
+            .collect();
+
+        // add those files
+        let _ = fixture.project.add(&add_files).await;
+
+        let new_name = "data/data_alt.tsv";
+        let target_path = PathBuf::from(new_name);
+
+        let statuses = get_statuses(&mut fixture, &path_context).await;
+        let exists = statuses.iter().any(|(path, _status)| path == &target_path);
+        assert!(!exists); // not there before move
+
+        // try moving a file (renaming)
+        fixture.project.mv("data/data.tsv", new_name).await.unwrap();
+
+        let exists = statuses.iter().any(|(path, _status)| path == &target_path);
+        assert!(!exists); // now it should be there
+
+        // now let's try moving to a directory
+        fs::create_dir_all("new_data/").unwrap();
+        fixture
+            .project
+            .mv("data/supplement/big_1.tsv.gz", "new_data/")
+            .await
+            .unwrap();
+
+        let statuses = get_statuses(&mut fixture, &path_context).await;
+        let target_path = PathBuf::from("data/supplement/big_1.tsv.gz");
+        let exists = statuses.iter().any(|(path, _status)| path == &target_path);
+        assert!(!exists); // now it should be there
+    }
 }