intervalToDuration seconds and minutes above 59

[ryancrunchi]

Version 2.29.3

https://codesandbox.io/s/date-fns-forked-2hwicl?file=/src/index.js

Expected result:
minutes and seconds are not above 59

Current result:
minutes = 1200
seconds = 144000

[fturmel]

Interesting edge case, thanks for reporting! PR incoming.

On the main branch we actually get {months:1, hours:-10} which makes more logical sense. The hotfix #3153 in v2 is breaking this issue even more because of Math.abs().

[rjhilgefort]

I was seeing this issue too with 2670827 * 1000 as my end time, which is 4 weeks 2 days 21 hours 53 min 47 s. I would see seconds that were in the thousands, same with minutes.

intervalToDuration({ start: 0, end: 2670827 * 1000 })

@fturmel I installed your PR and it fixes whatever was happening, thanks!