fix bug that shows 1 h 60 min instead of 2 h

graphhopper · Aug 26, 2023 · f2c8ae6 · f2c8ae6
1 parent 624625a
commit f2c8ae6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/Converters.ts b/src/Converters.ts
@@ -6,6 +6,7 @@ export function milliSecondsToText(ms: number) {
     const hours = Math.floor(ms / 3600000)
     const minutes = Math.round((ms % 3600000) / 60000)
 
+    if (minutes == 60) return hours + 1 + ' h'
     const hourText = hours > 0 ? hours + ' h' : ''
     if (minutes == 0 && hourText.length > 0) return hourText
     return (hourText ? hourText + ' ' : '') + minutes + ' min'

diff --git a/test/Converters.test.ts b/test/Converters.test.ts
@@ -6,6 +6,8 @@ describe('Converters', function () {
             expect(milliSecondsToText(59 * 1000)).toEqual('1 min')
 
             expect(milliSecondsToText(63 * 60 * 1000)).toEqual('1 h 3 min')
+
+            expect(milliSecondsToText(7198000)).toEqual('2 h')
         })
     })