diff --git a/cli/tools/coverage/reporter.rs b/cli/tools/coverage/reporter.rs index f6f8144a4c..6547f2036d 100644 --- a/cli/tools/coverage/reporter.rs +++ b/cli/tools/coverage/reporter.rs @@ -52,7 +52,12 @@ pub trait CoverageReporter { file_reports: &'a Vec<(CoverageReport, String)>, ) -> CoverageSummary { let urls = file_reports.iter().map(|rep| &rep.0.url).collect(); - let root = util::find_root(urls).unwrap().to_file_path().unwrap(); + let root = match util::find_root(urls) + .and_then(|root_path| root_path.to_file_path().ok()) + { + Some(path) => path, + None => return HashMap::new(), + }; // summary by file or directory // tuple of (line hit, line miss, branch hit, branch miss, parent) let mut summary = HashMap::new(); diff --git a/tests/integration/coverage_tests.rs b/tests/integration/coverage_tests.rs index a8a7ca0232..6e9a1454ca 100644 --- a/tests/integration/coverage_tests.rs +++ b/tests/integration/coverage_tests.rs @@ -555,3 +555,57 @@ File | Branch % | Line % | ", ); } + +#[test] +fn test_collect_summary_with_no_matches() { + let context: TestContext = TestContext::default(); + let temp_dir: &TempDir = context.temp_dir(); + let temp_dir_path: PathRef = PathRef::new(temp_dir.path().join("cov")); + + let empty_test_dir: PathRef = temp_dir_path.join("empty_dir"); + empty_test_dir.create_dir_all(); + + let output: util::TestCommandOutput = context + .new_command() + .args_vec(vec![ + "test".to_string(), + "--quiet".to_string(), + "--allow-read".to_string(), + format!("--coverage={}", temp_dir_path.as_path().display()), + empty_test_dir.as_path().to_str().unwrap().to_string(), + ]) + .run(); + + output.assert_exit_code(1); + + let actual: &str = output.combined_output(); + let expected_message: &str = "error: No test modules found"; + assert_contains!(actual, expected_message); + + // Check the contents of the coverage directory, ignoring 'empty_dir' + let mut unexpected_contents: Vec = Vec::new(); + for entry in std::fs::read_dir(temp_dir_path.as_path()) + .unwrap() + .flatten() + { + if entry.file_name() != "empty_dir" { + // Ignore the 'empty_dir' + unexpected_contents.push(entry.path()); + } + } + + // Report unexpected contents + if !unexpected_contents.is_empty() { + eprintln!("Unexpected files or directories in the coverage directory:"); + for path in &unexpected_contents { + eprintln!("{:?}", path); + } + } + + // Assert that the coverage directory is otherwise empty + assert!( + unexpected_contents.is_empty(), + "Expected the coverage directory to be empty except for 'empty_dir', but found: {:?}", + unexpected_contents + ); +}