2023-07-21 18:00:19 +00:00
|
|
|
package coderd
|
|
|
|
|
|
|
|
import (
|
|
|
|
"context"
|
|
|
|
"net/http/httptest"
|
|
|
|
"testing"
|
|
|
|
"time"
|
|
|
|
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/stretchr/testify/require"
|
2023-09-19 11:06:19 +00:00
|
|
|
|
|
|
|
"github.com/coder/coder/v2/codersdk"
|
2023-07-21 18:00:19 +00:00
|
|
|
)
|
|
|
|
|
|
|
|
func Test_parseInsightsStartAndEndTime(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Logf("machine location: %s", time.Now().Location())
|
2023-07-21 18:00:19 +00:00
|
|
|
layout := insightsTimeLayout
|
|
|
|
now := time.Now().UTC()
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Logf("now: %s", now)
|
|
|
|
t.Logf("now location: %s", now.Location())
|
2023-07-21 18:00:19 +00:00
|
|
|
y, m, d := now.Date()
|
|
|
|
today := time.Date(y, m, d, 0, 0, 0, 0, time.UTC)
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Logf("today: %s", today)
|
2023-07-21 18:00:19 +00:00
|
|
|
thisHour := time.Date(y, m, d, now.Hour(), 0, 0, 0, time.UTC)
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Logf("thisHour: %s", thisHour)
|
2023-07-21 18:00:19 +00:00
|
|
|
thisHourRoundUp := thisHour.Add(time.Hour)
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Logf("thisHourRoundUp: %s", thisHourRoundUp)
|
2023-07-21 18:00:19 +00:00
|
|
|
|
|
|
|
helsinki, err := time.LoadLocation("Europe/Helsinki")
|
|
|
|
require.NoError(t, err)
|
|
|
|
|
|
|
|
type args struct {
|
|
|
|
startTime string
|
|
|
|
endTime string
|
|
|
|
}
|
|
|
|
tests := []struct {
|
|
|
|
name string
|
|
|
|
args args
|
|
|
|
wantStartTime time.Time
|
|
|
|
wantEndTime time.Time
|
|
|
|
wantOk bool
|
|
|
|
}{
|
2023-12-04 17:20:22 +00:00
|
|
|
{
|
|
|
|
name: "Same",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-07-10T00:00:00Z",
|
|
|
|
endTime: "2023-07-10T00:00:00Z",
|
|
|
|
},
|
|
|
|
wantStartTime: time.Date(2023, 7, 10, 0, 0, 0, 0, time.UTC),
|
|
|
|
wantEndTime: time.Date(2023, 7, 10, 0, 0, 0, 0, time.UTC),
|
|
|
|
wantOk: true,
|
|
|
|
},
|
2023-07-21 18:00:19 +00:00
|
|
|
{
|
|
|
|
name: "Week",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-07-10T00:00:00Z",
|
|
|
|
endTime: "2023-07-17T00:00:00Z",
|
|
|
|
},
|
|
|
|
wantStartTime: time.Date(2023, 7, 10, 0, 0, 0, 0, time.UTC),
|
|
|
|
wantEndTime: time.Date(2023, 7, 17, 0, 0, 0, 0, time.UTC),
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Today",
|
|
|
|
args: args{
|
|
|
|
startTime: today.Format(layout),
|
|
|
|
endTime: thisHour.Format(layout),
|
|
|
|
},
|
2023-08-01 08:43:49 +00:00
|
|
|
wantStartTime: time.Date(today.Year(), today.Month(), today.Day(), 0, 0, 0, 0, time.UTC),
|
|
|
|
wantEndTime: time.Date(today.Year(), today.Month(), today.Day(), thisHour.Hour(), 0, 0, 0, time.UTC),
|
2023-07-21 18:00:19 +00:00
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Today with minutes and seconds",
|
|
|
|
args: args{
|
|
|
|
startTime: today.Format(layout),
|
|
|
|
endTime: thisHour.Add(time.Minute + time.Second).Format(layout),
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
{
|
2024-03-01 00:20:25 +00:00
|
|
|
name: "Today hour round up",
|
2023-07-21 18:00:19 +00:00
|
|
|
args: args{
|
|
|
|
startTime: today.Format(layout),
|
|
|
|
endTime: thisHourRoundUp.Format(layout),
|
|
|
|
},
|
2023-08-01 08:43:49 +00:00
|
|
|
wantStartTime: time.Date(today.Year(), today.Month(), today.Day(), 0, 0, 0, 0, time.UTC),
|
2024-03-01 00:20:25 +00:00
|
|
|
wantEndTime: time.Date(thisHourRoundUp.Year(), thisHourRoundUp.Month(), thisHourRoundUp.Day(), thisHourRoundUp.Hour(), 0, 0, 0, time.UTC),
|
2023-07-21 18:00:19 +00:00
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Other timezone week",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-07-10T00:00:00+03:00",
|
|
|
|
endTime: "2023-07-17T00:00:00+03:00",
|
|
|
|
},
|
|
|
|
wantStartTime: time.Date(2023, 7, 10, 0, 0, 0, 0, helsinki),
|
|
|
|
wantEndTime: time.Date(2023, 7, 17, 0, 0, 0, 0, helsinki),
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Daylight savings time",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-03-26T00:00:00+02:00",
|
|
|
|
endTime: "2023-03-27T00:00:00+03:00",
|
|
|
|
},
|
|
|
|
wantStartTime: time.Date(2023, 3, 26, 0, 0, 0, 0, helsinki),
|
|
|
|
wantEndTime: time.Date(2023, 3, 27, 0, 0, 0, 0, helsinki),
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Bad format",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-07-10",
|
|
|
|
endTime: "2023-07-17",
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Zero time",
|
|
|
|
args: args{
|
|
|
|
startTime: (time.Time{}).Format(layout),
|
|
|
|
endTime: (time.Time{}).Format(layout),
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Time in future",
|
|
|
|
args: args{
|
|
|
|
startTime: today.AddDate(0, 0, 1).Format(layout),
|
|
|
|
endTime: today.AddDate(0, 0, 2).Format(layout),
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "End before start",
|
|
|
|
args: args{
|
|
|
|
startTime: today.Format(layout),
|
|
|
|
endTime: today.AddDate(0, 0, -1).Format(layout),
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
|
|
tt := tt
|
|
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Log("startTime: ", tt.args.startTime)
|
|
|
|
t.Log("endTime: ", tt.args.endTime)
|
|
|
|
if tt.wantOk {
|
|
|
|
t.Log("wantStartTime: ", tt.wantStartTime)
|
|
|
|
t.Log("wantEndTime: ", tt.wantEndTime)
|
|
|
|
}
|
|
|
|
|
2023-07-21 18:00:19 +00:00
|
|
|
rw := httptest.NewRecorder()
|
2023-12-04 17:20:22 +00:00
|
|
|
gotStartTime, gotEndTime, gotOk := parseInsightsStartAndEndTime(context.Background(), rw, now, tt.args.startTime, tt.args.endTime)
|
2023-07-21 18:00:19 +00:00
|
|
|
|
|
|
|
if !assert.Equal(t, tt.wantOk, gotOk) {
|
|
|
|
//nolint:bodyclose
|
|
|
|
t.Log("Status: ", rw.Result().StatusCode)
|
|
|
|
t.Log("Body: ", rw.Body.String())
|
2023-12-04 17:20:22 +00:00
|
|
|
return
|
2023-07-21 18:00:19 +00:00
|
|
|
}
|
|
|
|
// assert.Equal is unable to test time equality with different
|
|
|
|
// (but same) locations because the *time.Location names differ
|
|
|
|
// between LoadLocation and Parse, so we use assert.WithinDuration.
|
|
|
|
assert.WithinDuration(t, tt.wantStartTime, gotStartTime, 0)
|
|
|
|
assert.True(t, tt.wantStartTime.Equal(gotStartTime))
|
|
|
|
assert.WithinDuration(t, tt.wantEndTime, gotEndTime, 0)
|
|
|
|
assert.True(t, tt.wantEndTime.Equal(gotEndTime))
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
2023-09-19 11:06:19 +00:00
|
|
|
|
|
|
|
func Test_parseInsightsInterval_week(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
|
|
|
layout := insightsTimeLayout
|
|
|
|
sydneyLoc, err := time.LoadLocation("Australia/Sydney") // Random location
|
|
|
|
require.NoError(t, err)
|
|
|
|
|
2023-12-04 17:20:22 +00:00
|
|
|
now := time.Now().In(sydneyLoc)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("now: %s", now)
|
|
|
|
|
2023-09-19 11:06:19 +00:00
|
|
|
y, m, d := now.Date()
|
|
|
|
today := time.Date(y, m, d, 0, 0, 0, 0, sydneyLoc)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("today: %s", today)
|
2023-09-19 11:06:19 +00:00
|
|
|
|
|
|
|
thisHour := time.Date(y, m, d, now.Hour(), 0, 0, 0, sydneyLoc)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("thisHour: %s", thisHour)
|
2023-09-19 11:06:19 +00:00
|
|
|
twoHoursAgo := thisHour.Add(-2 * time.Hour)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("twoHoursAgo: %s", twoHoursAgo)
|
2023-09-19 11:06:19 +00:00
|
|
|
thirteenDaysAgo := today.AddDate(0, 0, -13)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("thirteenDaysAgo: %s", thirteenDaysAgo)
|
2023-09-19 11:06:19 +00:00
|
|
|
|
|
|
|
sixDaysAgo := today.AddDate(0, 0, -6)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("sixDaysAgo: %s", sixDaysAgo)
|
2023-09-19 11:06:19 +00:00
|
|
|
nineDaysAgo := today.AddDate(0, 0, -9)
|
2023-09-26 10:33:21 +00:00
|
|
|
t.Logf("nineDaysAgo: %s", nineDaysAgo)
|
2023-09-19 11:06:19 +00:00
|
|
|
|
|
|
|
type args struct {
|
|
|
|
startTime string
|
|
|
|
endTime string
|
|
|
|
}
|
|
|
|
tests := []struct {
|
|
|
|
name string
|
|
|
|
args args
|
|
|
|
wantOk bool
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
name: "Two full weeks",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-08-10T00:00:00+02:00",
|
|
|
|
endTime: "2023-08-24T00:00:00+02:00",
|
|
|
|
},
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "One full week",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-09-06T00:00:00+02:00",
|
|
|
|
endTime: "2023-09-13T00:00:00+02:00",
|
|
|
|
},
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "6 days are acceptable",
|
|
|
|
args: args{
|
|
|
|
startTime: sixDaysAgo.Format(layout),
|
2023-12-04 17:20:22 +00:00
|
|
|
endTime: stripTime(thisHour).Format(layout),
|
2023-09-19 11:06:19 +00:00
|
|
|
},
|
|
|
|
wantOk: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "Shorter than a full week",
|
|
|
|
args: args{
|
|
|
|
startTime: "2023-09-08T00:00:00+02:00",
|
|
|
|
endTime: "2023-09-13T00:00:00+02:00",
|
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "9 days (7 + 2) are not acceptable",
|
|
|
|
args: args{
|
|
|
|
startTime: nineDaysAgo.Format(layout),
|
2023-12-04 17:20:22 +00:00
|
|
|
endTime: stripTime(thisHour).Format(layout),
|
2023-09-19 11:06:19 +00:00
|
|
|
},
|
|
|
|
wantOk: false,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
for _, tt := range tests {
|
|
|
|
tt := tt
|
2023-12-04 17:20:22 +00:00
|
|
|
|
2023-09-19 11:06:19 +00:00
|
|
|
t.Run(tt.name, func(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
2023-12-04 17:20:22 +00:00
|
|
|
t.Log("startTime: ", tt.args.startTime)
|
|
|
|
t.Log("endTime: ", tt.args.endTime)
|
|
|
|
|
2023-09-19 11:06:19 +00:00
|
|
|
rw := httptest.NewRecorder()
|
2023-12-04 17:20:22 +00:00
|
|
|
startTime, endTime, ok := parseInsightsStartAndEndTime(context.Background(), rw, now, tt.args.startTime, tt.args.endTime)
|
2023-09-19 11:06:19 +00:00
|
|
|
if !ok {
|
|
|
|
//nolint:bodyclose
|
|
|
|
t.Log("Status: ", rw.Result().StatusCode)
|
|
|
|
t.Log("Body: ", rw.Body.String())
|
|
|
|
}
|
|
|
|
require.True(t, ok, "start_time and end_time must be valid")
|
|
|
|
|
|
|
|
parsedInterval, gotOk := parseInsightsInterval(context.Background(), rw, "week", startTime, endTime)
|
|
|
|
if !assert.Equal(t, tt.wantOk, gotOk) {
|
|
|
|
//nolint:bodyclose
|
|
|
|
t.Log("Status: ", rw.Result().StatusCode)
|
|
|
|
t.Log("Body: ", rw.Body.String())
|
2023-12-04 17:20:22 +00:00
|
|
|
return
|
2023-09-19 11:06:19 +00:00
|
|
|
}
|
|
|
|
if tt.wantOk {
|
|
|
|
assert.Equal(t, codersdk.InsightsReportIntervalWeek, parsedInterval)
|
|
|
|
}
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
func TestLastReportIntervalHasAtLeastSixDays(t *testing.T) {
|
|
|
|
t.Parallel()
|
|
|
|
|
|
|
|
loc, err := time.LoadLocation("Europe/Warsaw")
|
|
|
|
require.NoError(t, err)
|
|
|
|
|
|
|
|
testCases := []struct {
|
|
|
|
name string
|
|
|
|
startTime time.Time
|
|
|
|
endTime time.Time
|
|
|
|
expected bool
|
|
|
|
}{
|
|
|
|
{
|
|
|
|
name: "perfectly full week",
|
|
|
|
startTime: time.Date(2023, time.September, 11, 12, 0, 0, 0, loc),
|
|
|
|
endTime: time.Date(2023, time.September, 18, 12, 0, 0, 0, loc),
|
|
|
|
expected: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "exactly 6 days apart",
|
|
|
|
startTime: time.Date(2023, time.September, 11, 12, 0, 0, 0, loc),
|
|
|
|
endTime: time.Date(2023, time.September, 17, 12, 0, 0, 0, loc),
|
|
|
|
expected: true,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "less than 6 days apart",
|
|
|
|
startTime: time.Date(2023, time.September, 11, 12, 0, 0, 0, time.UTC),
|
|
|
|
endTime: time.Date(2023, time.September, 17, 11, 0, 0, 0, time.UTC),
|
|
|
|
expected: false,
|
|
|
|
},
|
|
|
|
{
|
|
|
|
name: "forward DST change, 5 days and 23 hours apart",
|
|
|
|
startTime: time.Date(2023, time.March, 22, 12, 0, 0, 0, loc), // A day before DST starts
|
|
|
|
endTime: time.Date(2023, time.March, 28, 12, 0, 0, 0, loc), // Exactly 6 "days" apart
|
|
|
|
expected: true,
|
|
|
|
},
|
|
|
|
}
|
|
|
|
|
|
|
|
for _, tc := range testCases {
|
|
|
|
tc := tc
|
|
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
|
|
t.Parallel()
|
2023-12-04 17:20:22 +00:00
|
|
|
|
|
|
|
t.Log("startTime: ", tc.startTime)
|
|
|
|
t.Log("endTime: ", tc.endTime)
|
|
|
|
|
2023-09-19 11:06:19 +00:00
|
|
|
result := lastReportIntervalHasAtLeastSixDays(tc.startTime, tc.endTime)
|
|
|
|
if result != tc.expected {
|
|
|
|
t.Errorf("Expected %v, but got %v for start time %v and end time %v", tc.expected, result, tc.startTime, tc.endTime)
|
|
|
|
}
|
|
|
|
})
|
|
|
|
}
|
|
|
|
}
|
2023-12-04 17:20:22 +00:00
|
|
|
|
|
|
|
// stripTime strips the time from a time.Time value, but keeps the date and TZ.
|
|
|
|
func stripTime(t time.Time) time.Time {
|
|
|
|
y, m, d := t.Date()
|
|
|
|
return time.Date(y, m, d, 0, 0, 0, 0, t.Location())
|
|
|
|
}
|