diff options
| author | Taras Madan <tarasmadan@google.com> | 2024-10-01 12:33:15 +0200 |
|---|---|---|
| committer | Taras Madan <tarasmadan@google.com> | 2024-10-02 09:40:02 +0000 |
| commit | fb95da31ee52924519e703c1d8aa32bffe1db3e0 (patch) | |
| tree | cc724ca89d94200bbb374c980984bb3d0bc87bd2 /pkg | |
| parent | cae5050a7d451664114fa502ed59fa100329141f (diff) | |
pkg/cover: embed correct period type into line coverage url
Diffstat (limited to 'pkg')
| -rw-r--r-- | pkg/cover/heatmap.go | 2 | ||||
| -rw-r--r-- | pkg/coveragedb/time_period.go | 3 |
2 files changed, 3 insertions, 2 deletions
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) } |