test(format): add unit tests for formatDuration

Author: incognitojam

src/utils/format.spec.ts

@@ -1,6 +1,6 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
-import { formatDate, formatDistance } from './format'
+import { formatDate, formatDistance, formatDuration } from './format'
 
 
 describe('formatDistance', () => {
@@ -14,6 +14,20 @@ describe('formatDistance', () => {
 })
 
 
+describe('formatDuration', () => {
+  it('should format duration', () => {
+    expect(formatDuration(0)).toBe('0m')
+    expect(formatDuration(12)).toBe('12m')
+    expect(formatDuration(12.34)).toBe('12m')
+    expect(formatDuration(90)).toBe('1h 30m')
+    expect(formatDuration(120)).toBe('2h 0m')
+  })
+  it('should be blank for undefined duration', () => {
+    expect(formatDuration(undefined)).toBe('')
+  })
+})
+
+
 describe('formatDate', () => {
   beforeEach(() => {
     vi.useFakeTimers()

src/utils/format.ts

@@ -17,7 +17,7 @@ export const formatDistance = (miles: number | undefined): string => {
 }
 
 const _formatDuration = (duration: Duration): string => {
-  if (duration.asHours() > 0) {
+  if (duration.hours() > 0) {
     return duration.format('H[h] m[m]')
   } else {
     return duration.format('m[m]')
@@ -28,7 +28,7 @@ export const formatDuration = (minutes: number | undefined): string => {
   if (minutes === undefined) return ''
   const duration = dayjs.duration({
     hours: Math.floor(minutes / 60),
-    minutes: minutes % 60,
+    minutes: Math.round(minutes % 60),
   })
   return _formatDuration(duration)
 }