fix aggressive compression of hours into a single day

github · Mar 23, 2023 · cacd9e6 · cacd9e6
1 parent a307efa
commit cacd9e6
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/src/duration.ts b/src/duration.ts
@@ -147,7 +147,8 @@ export function roundToSingleUnit(duration: Duration, {relativeTo = Date.now()}:
   if (minutes >= 55) hours += Math.round(minutes / 60)
   if (hours || days || weeks || months || years) minutes = 0
 
-  if (hours >= 21) days += Math.round(hours / 24)
+  if (days && hours >= 12) days += Math.round(hours / 24)
+  if (!days && hours >= 21) days += Math.round(hours / 24)
   if (days || weeks || months || years) hours = 0
 
   const currentYear = relativeTo.getFullYear()

diff --git a/test/duration.ts b/test/duration.ts
@@ -215,6 +215,11 @@ suite('duration', function () {
         input: '2021-10-29T14:46:00.000Z',
         expected: '-P1Y',
       },
+      {
+        now: '2023-03-23T12:03:00.000Z',
+        input: '2023-03-21T16:03:00.000Z',
+        expected: '-P1DT20H',
+      },
     ])
     for (const {input, now, precision = 'millisecond', expected} of elapsed) {
       test(`${input} is ${expected} elapsed from ${now} (precision ${precision})`, () => {
@@ -235,6 +240,8 @@ suite('duration', function () {
       ['PT1H55M', 'PT2H'],
       ['PT20H', 'PT20H'],
       ['PT21H', 'P1D'],
+      ['P1DT20H', 'P2D'],
+      ['P1DT18H', 'P2D'],
       ['P4D', 'P4D', {relativeTo: new Date('2023-07-01T00:00:00')}],
       ['-P4D', '-P4D', {relativeTo: new Date('2023-07-01T00:00:00')}],
       ['P6D', 'P1W', {relativeTo: new Date('2023-07-01T00:00:00')}],

diff --git a/test/relative-time.js b/test/relative-time.js
@@ -1752,6 +1752,13 @@ suite('relative-time', function () {
         tense: 'past',
         expected: '2 years, 10 days',
       },
+      {
+        reference: '2023-03-23T12:03:00.000Z',
+        datetime: '2023-03-21T16:03:00.000Z',
+        format: 'relative',
+        tense: 'past',
+        expected: '2 days ago',
+      },
     ])
 
     for (const {