Fix problem with missing reminders making the reminder window wider (#…

…170)
mattermost · Jun 15, 2020 · c18c1c1 · c18c1c1
1 parent 7546d8b
commit c18c1c1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/server/mscalendar/availability.go b/server/mscalendar/availability.go
@@ -19,7 +19,7 @@ const (
 	calendarViewTimeWindowSize      = 10 * time.Minute
 	StatusSyncJobInterval           = 5 * time.Minute
 	upcomingEventNotificationTime   = 10 * time.Minute
-	upcomingEventNotificationWindow = (StatusSyncJobInterval * 9) / 10 //90% of the interval
+	upcomingEventNotificationWindow = (StatusSyncJobInterval * 11) / 10 //110% of the interval
 )
 
 type Availability interface {

diff --git a/server/mscalendar/availability_test.go b/server/mscalendar/availability_test.go
@@ -219,7 +219,7 @@ func TestReminders(t *testing.T) {
 		},
 		"One remote event, but it is too far in the future.": {
 			remoteEvents: []*remote.Event{
-				{ICalUID: "event_id", Start: remote.NewDateTime(time.Now().Add(15*time.Minute).UTC(), "UTC"), End: remote.NewDateTime(time.Now().Add(45*time.Minute).UTC(), "UTC")},
+				{ICalUID: "event_id", Start: remote.NewDateTime(time.Now().Add(20*time.Minute).UTC(), "UTC"), End: remote.NewDateTime(time.Now().Add(45*time.Minute).UTC(), "UTC")},
 			},
 			numReminders:   0,
 			shouldLogError: false,