From fb95da31ee52924519e703c1d8aa32bffe1db3e0 Mon Sep 17 00:00:00 2001
From: Taras Madan <tarasmadan@google.com>
Date: Tue, 1 Oct 2024 12:33:15 +0200
Subject: pkg/cover: embed correct period type into line coverage url

---
 pkg/cover/heatmap.go          | 2 +-
 pkg/coveragedb/time_period.go | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

(limited to 'pkg')

diff --git a/pkg/cover/heatmap.go b/pkg/cover/heatmap.go
index 685518bd5..90737c885 100644
--- a/pkg/cover/heatmap.go
+++ b/pkg/cover/heatmap.go
@@ -88,7 +88,7 @@ func (thm *templateHeatmapRow) prepareDataFor(timePeriods []coveragedb.TimePerio
 			thm.FileCoverageLink = append(thm.FileCoverageLink,
 				fmt.Sprintf("/upstream/graph/coverage/file?dateto=%s&period=%s&commit=%s&filepath=%s",
 					tp.DateTo.String(),
-					"day",
+					tp.Type,
 					"commit",
 					thm.filePath))
 		}
diff --git a/pkg/coveragedb/time_period.go b/pkg/coveragedb/time_period.go
index 4e17e4273..9503cf3e1 100644
--- a/pkg/coveragedb/time_period.go
+++ b/pkg/coveragedb/time_period.go
@@ -15,6 +15,7 @@ import (
 type TimePeriod struct {
 	DateTo civil.Date
 	Days   int
+	Type   string // DayPeriod, MonthPeriod, QuarterPeriod.
 }
 
 // DatesFromTo returns the closed range [fromDate, toDate].
@@ -27,7 +28,7 @@ func MakeTimePeriod(targetDate civil.Date, periodType string) (TimePeriod, error
 	if err != nil {
 		return TimePeriod{}, err
 	}
-	tp := TimePeriod{DateTo: targetDate, Days: pOps.PointedPeriodDays(targetDate)}
+	tp := TimePeriod{DateTo: targetDate, Days: pOps.PointedPeriodDays(targetDate), Type: periodType}
 	if !pOps.IsValidPeriod(tp) {
 		return TimePeriod{}, fmt.Errorf("date %s doesn't point the period(%s) end", targetDate.String(), periodType)
 	}
-- 
cgit mrf-deployment