fixed bug; wrote tests (#5329)

Kira-Pilot · web-flow · commit 85945af55e86 · 2022-12-06T16:58:32.000-05:00
diff --git a/site/src/components/Timeline/Timeline.test.tsx b/site/src/components/Timeline/Timeline.test.tsx
@@ -0,0 +1,8 @@
+import { createDisplayDate } from "./TimelineDateRow"
+
+describe("createDisplayDate", () => {
+  it("returns correctly for Saturdays", () => {
+    const date = new Date("Sat Dec 03 2022 00:00:00 GMT-0500")
+    expect(createDisplayDate(date)).toEqual("last Saturday")
+  })
+})
diff --git a/site/src/components/Timeline/TimelineDateRow.tsx b/site/src/components/Timeline/TimelineDateRow.tsx
@@ -8,16 +8,18 @@ export interface TimelineDateRow {
   date: Date
 }
 
+// We only want the message related to the date since the time is displayed
+// inside of the build row
+export const createDisplayDate = (date: Date): string =>
+  formatRelative(date, new Date()).split(" at ")[0]
+
 export const TimelineDateRow: FC<TimelineDateRow> = ({ date }) => {
   const styles = useStyles()
-  // We only want the message related to the date since the time is displayed
-  // inside of the build row
-  const displayDate = formatRelative(date, new Date()).split("at")[0]
 
   return (
     <TableRow className={styles.dateRow}>
       <TableCell className={styles.dateCell} title={date.toLocaleDateString()}>
-        {displayDate}
+        {createDisplayDate(date)}
       </TableCell>
     </TableRow>
   )
