Fixing time duration display (#1950)

* Fixing time module and tests.

* Better tests and constants.

Co-authored-by: phix <peter.hicks@astronomer.io>
Co-authored-by: Willy Lulciuc <willy@datakin.com>

Author: 3 people

web/src/__tests__/reducers/jobs.test.ts

@@ -2,6 +2,7 @@
 
 import * as actionTypes from '../../store/actionCreators/actionTypes'
 import jobsReducer, { initialState } from '../../store/reducers/jobs'
+import { stopWatchDuration } from "../../helpers/time";
 
 const jobs = require('../../../docker/db/data/jobs.json')
 
@@ -17,3 +18,45 @@ describe('jobs reducer', () => {
     expect(jobsReducer(initialState, action)).toStrictEqual({ isLoading: false, result: jobs, init: true })
   })
 })
+
+describe('stopWatchDuration', () => {
+  const oneMinute = 60 * 1000;
+  const oneHour = 60 * oneMinute;
+  const oneDay = 24 * oneHour;
+
+  it('more than one week', () => {
+    const value = stopWatchDuration(oneDay * 9)
+    expect("9d 0h 0m 0s").toBe(value);
+  })
+
+  it('more than one day', () => {
+    const value = stopWatchDuration(oneDay + oneHour)
+    expect("1d 1h 0m 0s").toBe(value);
+  })
+
+  it('less than one day', () => {
+    const value = stopWatchDuration(oneDay - 1000);
+    expect("23h 59m 59s").toBe(value);
+  })
+
+  it('less than one hour', () => {
+    const value = stopWatchDuration(oneHour - 1000);
+    expect("59m 59s").toBe(value);
+  })
+
+  it('less than one minute', () => {
+    const value = stopWatchDuration(oneMinute - 1000);
+    expect("0m 59s").toBe(value);
+  })
+
+  it('less than one second', () => {
+    const value = stopWatchDuration(999);
+    expect("999 ms").toBe(value);
+  })
+
+  it('no time', () => {
+    const value = stopWatchDuration(0);
+    expect("0").toBe(value);
+  })
+
+})

web/src/helpers/time.ts

@@ -14,13 +14,15 @@ export function stopWatchDuration(durationMs: number) {
   if (duration.asMilliseconds() === 0) {
     return '0'
   }
-
-  if (duration.asHours() > 1) {
-    return `${duration.hours()}h ${addLeadingZero(duration.seconds())}s`
+  if (duration.asHours() > 24) {
+    return `${duration.days()}d ${duration.hours()}h ${duration.minutes()}m ${duration.seconds()}s`
+  }
+  if (duration.asMinutes() > 60) {
+    return `${duration.hours()}h ${duration.minutes()}m ${duration.seconds()}s`
   }
   if (duration.asSeconds() > 1) {
     return `${duration.minutes()}m ${addLeadingZero(duration.seconds())}s`
   } else {
-    return `${duration.milliseconds()} ms`
+    return `${duration.asMilliseconds()} ms`
   }
 }