Check for ongoing recurring events when making date-based event calcu…

…lations. (#795)
department-of-veterans-affairs · Oct 25, 2024 · 59729e8 · 59729e8
1 parent 8217ee7
commit 59729e8
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 10 deletions.
diff --git a/src/lib/utils/date.test.ts b/src/lib/utils/date.test.ts
@@ -246,14 +246,27 @@ describe('formatDateObject', () => {
 })
 
 describe('deriveMostRecentDate', () => {
-  it('should return the closest future event', () => {
-    const mockCurrentTime = new Date('2023-08-01T12:00:00Z').getTime()
+  it('should return the closest future or ongoing event', () => {
+    const mockCurrentTime = new Date('2023-08-01T18:00:00Z').getTime()
     jest.spyOn(Date, 'now').mockImplementation(() => mockCurrentTime)
 
     const datetimeRange = [
-      { value: new Date('2023-07-01T14:00:00Z').getTime() / 1000 }, // Past event
-      { value: new Date('2023-09-01T14:00:00Z').getTime() / 1000 }, // Closest future event
-      { value: new Date('2023-10-01T14:00:00Z').getTime() / 1000 }, // Later future event
+      {
+        value: new Date('2023-07-01T14:00:00Z').getTime() / 1000,
+        endValue: new Date('2023-07-01T16:00:00Z').getTime() / 1000,
+      }, // Past event
+      {
+        value: new Date('2023-08-01T14:00:00Z').getTime() / 1000,
+        endValue: new Date('2023-08-01T19:00:00Z').getTime() / 1000,
+      },
+      {
+        value: new Date('2023-09-01T14:00:00Z').getTime() / 1000,
+        endValue: new Date('2023-09-01T16:00:00Z').getTime() / 1000,
+      }, // Closest future event
+      {
+        value: new Date('2023-10-01T14:00:00Z').getTime() / 1000,
+        endValue: new Date('2023-10-01T16:00:00Z').getTime() / 1000,
+      }, // Later future event
     ]
 
     const closestEvent = deriveMostRecentDate(datetimeRange)

diff --git a/src/lib/utils/date.ts b/src/lib/utils/date.ts
@@ -193,14 +193,14 @@ export const deriveMostRecentDate = (
 ) => {
   const currentTime = Math.floor(Date.now() / 1000)
 
-  // Filter for future events
-  const futureEvents = datetimeRange.filter(
-    (event) => event.value > currentTime
+  // Filter for ongoing and future events
+  const ongoingAndFutureEvents = datetimeRange.filter(
+    (event) => event.endValue > currentTime
   )
 
   // If there are future events, return closest event
-  if (futureEvents.length > 0) {
-    return futureEvents.reduce((closest, current) => {
+  if (ongoingAndFutureEvents.length > 0) {
+    return ongoingAndFutureEvents.reduce((closest, current) => {
       return closest.value < current.value ? closest : current
     })
   }