From 19f6db10381abd46566d8b044c90bbed6fc27a51 Mon Sep 17 00:00:00 2001
From: Colin Rofls <colin@cmyr.net>
Date: Tue, 29 Oct 2024 14:24:18 -0400
Subject: [PATCH] [crater] Show runtime in html

This is useful information, and will be even more useful if we want to
know if caching is useful.
---
 fontc_crater/resources/style.css |  6 +++++-
 fontc_crater/src/ci.rs           | 22 +++++++++++++++++++++-
 fontc_crater/src/ci/html.rs      |  3 ++-
 3 files changed, 28 insertions(+), 3 deletions(-)

diff --git a/fontc_crater/resources/style.css b/fontc_crater/resources/style.css
index 83f2e2e9..d32a1418 100644
--- a/fontc_crater/resources/style.css
+++ b/fontc_crater/resources/style.css
@@ -33,6 +33,11 @@ td {
   font-size: 0.8em;
 }
 
+.elapsed {
+  color: #888;
+  font-size: 0.8em;
+}
+
 td.rev {
   font-family: monospace;
 }
@@ -75,7 +80,6 @@ td.rev {
 }
 
 .changed_tag_list {
-  /*display: inline-block;*/
   font-family: monospace;
   font-size: 0.8em;
   color: #888;
diff --git a/fontc_crater/src/ci.rs b/fontc_crater/src/ci.rs
index 4c8dca91..d46de0eb 100644
--- a/fontc_crater/src/ci.rs
+++ b/fontc_crater/src/ci.rs
@@ -7,10 +7,11 @@
 
 use std::{
     collections::BTreeMap,
+    fmt::Write,
     path::{Path, PathBuf},
 };
 
-use chrono::{DateTime, Utc};
+use chrono::{DateTime, TimeZone, Utc};
 use google_fonts_sources::{Config, RepoInfo};
 use serde::de::DeserializeOwned;
 
@@ -99,6 +100,7 @@ fn run_crater_and_save_results(args: &CiArgs) -> Result<(), Error> {
     log::info!("using cache dir {}", cache_dir.display());
 
     let (targets, source_repos) = make_targets(&cache_dir, &inputs);
+    let n_targets = targets.len();
     let began = Utc::now();
     let results = super::run_all(targets, &cache_dir, super::ttx_diff_runner::run_ttx_diff)?
         .into_iter()
@@ -106,6 +108,9 @@ fn run_crater_and_save_results(args: &CiArgs) -> Result<(), Error> {
         .collect();
     let finished = Utc::now();
 
+    let elapsed = format_elapsed_time(&began, &finished);
+    log::info!("completed {n_targets} targets in {elapsed}");
+
     let summary = super::ttx_diff_runner::Summary::new(&results);
     if Some(&summary) == prev_runs.last().map(|run| &run.stats) {
         log::info!("output identical to last run, skipping");
@@ -221,3 +226,18 @@ fn should_build_in_gftools_mode(src_path: &Path, config: &Config) -> bool {
         .filter(|provider| *provider != "googlefonts")
         .is_none()
 }
+
+fn format_elapsed_time<Tmz: TimeZone>(start: &DateTime<Tmz>, end: &DateTime<Tmz>) -> String {
+    let delta = end.clone().signed_duration_since(start);
+    let mut out = String::new();
+    let hours = delta.num_hours();
+    let mins = delta.num_minutes() - hours * 60;
+    let secs = delta.num_seconds() - (hours * 60 + mins) * 60;
+    assert!(!hours.is_negative() | mins.is_negative() | secs.is_negative());
+    if delta.num_hours() > 0 {
+        write!(&mut out, "{hours}h").unwrap();
+    }
+    write!(&mut out, "{mins}m").unwrap();
+    write!(&mut out, "{secs}s").unwrap();
+    out
+}
diff --git a/fontc_crater/src/ci/html.rs b/fontc_crater/src/ci/html.rs
index c45eb044..59b32bea 100644
--- a/fontc_crater/src/ci/html.rs
+++ b/fontc_crater/src/ci/html.rs
@@ -188,9 +188,10 @@ fn make_table_body(runs: &[RunSummary]) -> Markup {
             td.other_err {  (run.stats.other_failure) " " (other_err_diff)  }
             }
         };
+        let elapsed = super::format_elapsed_time(&run.began, &run.finished);
         html! {
             tr.run {
-                td.date { (run.began.format("%Y-%m-%d %H:%M:%S")) }
+                td.date { (run.began.format("%Y-%m-%d %H%M")) span.elapsed { " (" (elapsed) ")"} }
                 td.rev { a href=(diff_url) { (short_rev) } }
                 td.total {  ( run.stats.total_targets) " " (total_diff) }
                 td.identical {  (run.stats.identical) " " (identical_diff)  }