feat(update-expected): accept and warn on empty exec. reports

erichdongubler-mozilla · Jun 11, 2024 · 45843f1 · 45843f1
1 parent 7b64a0e
commit 45843f1
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/moz-webgpu-cts/src/main.rs b/moz-webgpu-cts/src/main.rs
@@ -391,7 +391,7 @@ fn run(cli: Cli) -> ExitCode {
                         .map_err(Report::msg)
                         .wrap_err("failed to open file")
                         .and_then(|reader| {
-                            serde_json::from_reader::<_, ExecutionReport>(reader)
+                            serde_json::from_reader::<_, Option<ExecutionReport>>(reader)
                                 .into_diagnostic()
                                 .wrap_err("failed to parse JSON")
                         })
@@ -410,11 +410,19 @@ fn run(cli: Cli) -> ExitCode {
                 });
 
             for res in exec_reports_receiver {
-                let (_path, exec_report) = match res {
+                let (path, exec_report) = match res {
                     Ok(ok) => ok,
                     Err(AlreadyReportedToCommandline) => return ExitCode::FAILURE,
                 };
 
+                let exec_report = match exec_report {
+                    Some(some) => some,
+                    None => {
+                        log::warn!("empty report found: {}", path.display());
+                        continue;
+                    }
+                };
+
                 let ExecutionReport {
                     run_info:
                         RunInfo {