suimeng
/
helix
mirror of https://github.com/helix-editor/helix
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

better logging for coverage

pull/10758/head
Dustin Lagoy 2025-04-03 10:43:49 -07:00
parent 5fdd2e99b9
commit c142d85f71
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
helix-view/src/coverage.rs
View File

@ -89,7 +89,10 @@ pub fn get_coverage(document_path: &std::path::PathBuf) -> Option<FileCoverage>
log::debug!("document path: {:?}", document_path); log::debug!("document path: {:?}", document_path);
let file_coverage = coverage.files.get(document_path)?; let file_coverage = coverage.files.get(document_path).or_else(|| {
log::warn!("file: {:?} not found in coverage", document_path);
None
})?;
let coverage_time = file_coverage.modified_time?; let coverage_time = file_coverage.modified_time?;
let document_metadata = document_path.metadata().ok()?; let document_metadata = document_path.metadata().ok()?;
@ -159,6 +162,7 @@ impl From<RawCoverage> for Coverage {
let raw_path: std::path::PathBuf = let raw_path: std::path::PathBuf =
[&source.name, &class.filename].iter().collect(); [&source.name, &class.filename].iter().collect();
if let Ok(path) = std::fs::canonicalize(raw_path) { if let Ok(path) = std::fs::canonicalize(raw_path) {
log::debug!("add file {:?} to coverage", path);
files.insert( files.insert(
path, path,
FileCoverage { FileCoverage {
@ -168,6 +172,7 @@ impl From<RawCoverage> for Coverage {
); );
break; break;
} }
log::warn!("could not add file {:?} to coverage", path);
} }
} }
} }

4
helix-view/src/gutter.rs
View File

@ -154,7 +154,7 @@ pub fn coverage<'doc>(
if let Some(document_path) = &doc.path { if let Some(document_path) = &doc.path {
if let Some(file_coverage) = coverage::get_coverage(document_path) { if let Some(file_coverage) = coverage::get_coverage(document_path) {
log::debug!("return valid coverage gutter"); log::info!("return valid coverage gutter for {:?}", document_path);
return Box::new( return Box::new(
move |line: usize, _selected: bool, _first_visual_line: bool, out: &mut String| { move |line: usize, _selected: bool, _first_visual_line: bool, out: &mut String| {
if let Some(line_coverage) = file_coverage.lines.get(&(line as u32)) { if let Some(line_coverage) = file_coverage.lines.get(&(line as u32)) {
@ -172,7 +172,7 @@ pub fn coverage<'doc>(
); );
} }
} }
log::debug!("return empty coverage gutter"); log::info!("return empty coverage gutter for {:?}", document_path);
return Box::new(move |_, _, _, _| None); return Box::new(move |_, _, _, _| None);
} }