fix clippy and add unix absolute path test

PowerShell · Dec 16, 2024 · e5f6018 · e5f6018
1 parent 6a241ab
commit e5f6018
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/dsc_lib/src/functions/path.rs b/dsc_lib/src/functions/path.rs
@@ -39,9 +39,8 @@ impl Function for Path {
                     path.push(s.to_owned() + std::path::MAIN_SEPARATOR.to_string().as_str());
                     first = false;
                     continue;
-                } else {
-                    path.push(s);
                 }
+                path.push(s);
             } else {
                 return Err(DscError::Parser("Arguments must all be strings".to_string()));
             }
@@ -72,6 +71,14 @@ mod tests {
         assert_eq!(result, format!("a{separator}C:{separator}test"));
     }
 
+    #[test]
+    fn unix_absolute_path() {
+        let mut parser = Statement::new().unwrap();
+        let separator = std::path::MAIN_SEPARATOR;
+        let result = parser.parse_and_execute("[path('/','a','b')]", &Context::new()).unwrap();
+        assert_eq!(result, format!("/a{separator}b"));
+    }
+
     #[test]
     fn two_args() {
         let mut parser = Statement::new().unwrap();