Skip to content

Commit 5642b60

Browse files
committed
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.
1 parent 1c4febd commit 5642b60

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

dashboard/app/coverage.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -87,8 +87,10 @@ func handleHeatmap(c context.Context, w http.ResponseWriter, r *http.Request, f
8787
if periodType == "" {
8888
periodType = coveragedb.DayPeriod
8989
}
90-
if periodType != coveragedb.DayPeriod && periodType != coveragedb.MonthPeriod {
91-
return fmt.Errorf("only day and month are allowed, but received %s instead, %w",
90+
if periodType != coveragedb.DayPeriod &&
91+
periodType != coveragedb.MonthPeriod &&
92+
periodType != coveragedb.QuarterPeriod {
93+
return fmt.Errorf("only 'day', 'month' and 'quarter' are allowed, but received %s instead, %w",
9294
periodType, ErrClientBadRequest)
9395
}
9496

0 commit comments

Comments
 (0)