Fix time rounding bug (fixes #154)

Author: rianadon

src/format-time.ts

@@ -46,8 +46,13 @@ const hmsTime = (d: number) => {
   return joinWithColons(h, m, s);
 }
 const hmTime = (d: number) => {
-  const h = Math.trunc(d / 3600);
-  const m = Math.ceil((d % 3600) / 60); // Round up the minutes (#86)
+  let h = Math.trunc(d / 3600);
+  let m = Math.ceil((d % 3600) / 60); // Round up the minutes (#86)
+  if (m == 60) {
+    // Edge case: minutes rounded up to 60, so I need to carry to the hours
+    h += 1;
+    m = 0;
+  }
   return joinWithColons(0, h, m);
 }
 

test/format-time.test.ts

@@ -62,4 +62,8 @@ it('can render negative times', () => {
     expect(formatTime(-30, 'hms')).toBe('-30')
     expect(formatTime(-60, 'hms')).toBe('-1:00')
     expect(formatTime(-90, 'hms')).toBe('-1:30')
-});
+});
+
+it('can render tricky times', () => {
+  expect(formatTime(59 + 59*60 + 1*60*60, 'hm')).toBe('2:00')
+});