Interpret relative paths relative to <ROOT>

Author: smoelius

README.md

@@ -172,7 +172,7 @@ To be clear, Necessist does not apply (`*`) formally, e.g., Necessist does not a
 Usage: necessist [OPTIONS] [TEST_FILES_OR_DIRS]... [-- <ARGS>...]
 
 Arguments:
-  [TEST_FILES_OR_DIRS]...  Test files or directories to mutilate (optional)
+  [TEST_FILES_OR_DIRS]...  Test files or directories to mutilate; if `--root <ROOT>` is passed, relative paths are interpreted relative to <ROOT>
   [ARGS]...                Additional arguments to pass to each test command
 
 Options:

core/src/cli.rs

@@ -59,7 +59,8 @@ pub struct Opts<Identifier: Clone + Send + Sync + ValueEnum + 'static> {
     verbose: bool,
     #[clap(
         value_name = "TEST_FILES_OR_DIRS",
-        help = "Test files or directories to mutilate (optional)"
+        help = "Test files or directories to mutilate; if `--root <ROOT>` is passed, relative \
+                paths are interpreted relative to <ROOT>"
     )]
     zsource_files: Vec<String>,
     #[clap(

core/src/core.rs

@@ -491,7 +491,12 @@ fn canonicalize_source_files(context: &LightContext) -> Result<Vec<PathBuf>> {
         .source_files
         .iter()
         .map(|path| {
-            let path_buf = dunce::canonicalize(path)
+            let path_to_canonicalize = if path.is_absolute() {
+                path.clone()
+            } else {
+                context.root.join(path)
+            };
+            let path_buf = dunce::canonicalize(&path_to_canonicalize)
                 .with_context(|| format!("Failed to canonicalize `{}`", path.display()))?;
             ensure!(
                 path_buf.starts_with(context.root.as_path()),

necessist/tests/necessist_db_absent/cli_input_mixed.toml

@@ -2,8 +2,8 @@ args = [
     "--no-sqlite",
     "--dump-candidates",
     "--root=fixtures/cli_input",
-    "fixtures/cli_input/src/some_dir",
-    "fixtures/cli_input/src/other_dir/some_file.rs",
+    "src/some_dir",
+    "src/other_dir/some_file.rs",
 ]
 
 stderr = ""

necessist/tests/necessist_db_absent/dry_run_failure_test_file.toml

@@ -1,7 +1,7 @@
 args = [
     "--no-sqlite",
     "--root=fixtures/dry_run_failure",
-    "fixtures/dry_run_failure/tests/b.rs",
+    "tests/b.rs",
 ]
 
 stderr = ""