Merge pull request #239 from teloon/fix-time-range

Fix time range

Author: cbusbey

internal/time_range.go

@@ -89,6 +89,10 @@ func (r *TimeRange) isInWeekRange(t time.Time) bool {
 			return r.isInTimeRange(t)
 		}
 
+		if r.startTime.d < r.endTime.d {
+			return false
+		}
+
 		return true
 	}
 

internal/time_range_test.go

@@ -204,6 +204,20 @@ func TestTimeRangeIsInRangeWithDay(t *testing.T) {
 
 	now = time.Date(2006, time.December, 4, 9, 0, 0, 0, time.UTC)
 	assert.True(t, NewUTCWeekRange(startTime, endTime, startDay, endDay).IsInRange(now))
+
+	startTime = NewTimeOfDay(8, 59, 0)
+	endTime = NewTimeOfDay(9, 1, 0)
+	startDay = time.Sunday
+	endDay = time.Sunday
+
+	now = time.Date(2006, time.December, 3, 8, 59, 0, 0, time.UTC)
+	assert.True(t, NewUTCWeekRange(startTime, endTime, startDay, endDay).IsInRange(now))
+
+	now = time.Date(2006, time.December, 3, 9, 1, 0, 0, time.UTC)
+	assert.True(t, NewUTCWeekRange(startTime, endTime, startDay, endDay).IsInRange(now))
+
+	now = time.Date(2006, time.December, 4, 8, 59, 0, 0, time.UTC)
+	assert.False(t, NewUTCWeekRange(startTime, endTime, startDay, endDay).IsInRange(now))
 }
 
 func TestTimeRangeIsInSameRange(t *testing.T) {