From a2ada0e73dabbcf555e77b26a3e4ee3455dfd3e7 Mon Sep 17 00:00:00 2001 From: Taras Madan Date: Mon, 7 Apr 2025 10:16:17 +0200 Subject: dashboard/app: allow quarter coverage analysis It was previously disabled because line coverage required data from BigQuery. It was too expensive. Coverage numbers are available in the Spanner now. --- dashboard/app/coverage.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'dashboard/app/coverage.go') diff --git a/dashboard/app/coverage.go b/dashboard/app/coverage.go index 4c83f299f..68f5459c4 100644 --- a/dashboard/app/coverage.go +++ b/dashboard/app/coverage.go @@ -87,8 +87,10 @@ func handleHeatmap(c context.Context, w http.ResponseWriter, r *http.Request, f if periodType == "" { periodType = coveragedb.DayPeriod } - if periodType != coveragedb.DayPeriod && periodType != coveragedb.MonthPeriod { - return fmt.Errorf("only day and month are allowed, but received %s instead, %w", + if periodType != coveragedb.DayPeriod && + periodType != coveragedb.MonthPeriod && + periodType != coveragedb.QuarterPeriod { + return fmt.Errorf("only 'day', 'month' and 'quarter' are allowed, but received %s instead, %w", periodType, ErrClientBadRequest) } -- cgit mrf-deployment