fix for invalid time

Author: egor-zalenski

redisinsight/ui/src/utils/monitorUtils.ts

@@ -1,4 +1,7 @@
 import { format } from 'date-fns'
+import { isFinite } from 'lodash'
 
-export const getFormatTime = (time: string = '') =>
-  format(new Date(+time * 1_000), 'HH:mm:ss.SSS')
+export const getFormatTime = (time: string) =>
+  (typeof time === 'string' && isFinite(+time)
+    ? format(new Date(+time * 1_000), 'HH:mm:ss.SSS')
+    : 'Invalid time')

redisinsight/ui/src/utils/tests/monitorUtils.spec.ts

@@ -1,16 +1,23 @@
 import { getFormatTime } from '../monitorUtils'
 
 const getOutputForFormatTime: any[] = [
-  ['1641450853.668074', '09:34:13.668'],
-  ['1641450854.612083', '09:34:14.612'],
-  ['1641450856.616102', '09:34:16.616'],
-  ['1641450858.616121', '09:34:18.616'],
+  [undefined, 'Invalid time'],
+  [null, 'Invalid time'],
+  [{}, 'Invalid time'],
+  ['oeuoeu', 'Invalid time'],
+  [1, 'Invalid time'],
+  [11641450853, 'Invalid time'],
+  ['1641450853.668074[0', 'Invalid time'],
+  ['1641450853.668074', ':34:13.668'],
+  ['1641450854.612083', ':34:14.612'],
+  ['1641450856.616102', ':34:16.616'],
+  ['1641450858.616121', ':34:18.616'],
 ]
 
-describe.skip('formatToText', () => {
+describe('formatToText', () => {
   it.each(getOutputForFormatTime)('for input: %s (reply), should be output: %s',
     (reply, expected) => {
       const result = getFormatTime(reply)
-      expect(result).toBe(expected)
+      expect(result).toContain(expected)
     })
 })