From aa144e68c6f16ffad2a5f58121a13fd76fbf6ae4 Mon Sep 17 00:00:00 2001 From: Dustin Lagoy Date: Thu, 3 Apr 2025 10:02:04 -0700 Subject: [PATCH] allow class lines to be optional in coverage xml --- helix-view/src/coverage.rs | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/helix-view/src/coverage.rs b/helix-view/src/coverage.rs index 43d87c755..fe0a0399f 100644 --- a/helix-view/src/coverage.rs +++ b/helix-view/src/coverage.rs @@ -60,7 +60,7 @@ struct Class { name: String, #[serde(rename = "@filename")] filename: String, - lines: Lines, + lines: Option, } #[derive(Deserialize, Debug)] @@ -148,8 +148,10 @@ impl From for Coverage { for package in coverage.packages.package { for class in package.classes.class { let mut lines = HashMap::new(); - for line in class.lines.line { - lines.insert(line.number - 1, line.hits > 0); + if let Some(class_lines) = class.lines { + for line in class_lines.line { + lines.insert(line.number - 1, line.hits > 0); + } } for source in &coverage.sources.source { // it is ambiguous to which source a coverage class might belong