Add test for duration with both months and minutes (P1M2DT3M30S)

Copilot · francinelucca · Copilot · commit 7bf72252feaa · 2025-08-26T02:52:38.000Z
Co-authored-by: francinelucca &lt;40550942+francinelucca@users.noreply.github.com&gt;
diff --git a/src/duration-format-ponyfill.ts b/src/duration-format-ponyfill.ts
@@ -114,7 +114,7 @@ export default class DurationFormat {
       let formattedValue = new Intl.NumberFormat(locale, nfOpts).format(value)
 
       // Custom handling for narrow month formatting to use "mo" instead of "m"
-      if (unit === 'months' && unitStyle === 'narrow') {
+      if (unit === 'months' && (unitStyle === 'narrow' || (style === 'narrow' && formattedValue.endsWith('m')))) {
         formattedValue = formattedValue.replace(/(\d+)m$/, '$1mo')
       }
 
diff --git a/test/duration-format-ponyfill.ts b/test/duration-format-ponyfill.ts
@@ -92,6 +92,20 @@ suite('duration format ponyfill', function () {
         {type: 'element', value: '8s'},
       ],
     },
+    {
+      duration: 'P1M2DT3M30S',
+      locale: 'en',
+      style: 'narrow',
+      parts: [
+        {type: 'element', value: '1mo'},
+        {type: 'literal', value: ' '},
+        {type: 'element', value: '2d'},
+        {type: 'literal', value: ' '},
+        {type: 'element', value: '3m'},
+        {type: 'literal', value: ' '},
+        {type: 'element', value: '30s'},
+      ],
+    },
   ])
 
   for (const {duration, locale, parts, ...opts} of tests) {

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ export default class DurationFormat {`
`114`	`114`	`let formattedValue = new Intl.NumberFormat(locale, nfOpts).format(value)`
`115`	`115`
`116`	`116`	`// Custom handling for narrow month formatting to use "mo" instead of "m"`
`117`		`- if (unit === 'months' && unitStyle === 'narrow') {`
	`117`	`+ if (unit === 'months' && (unitStyle === 'narrow' \|\| (style === 'narrow' && formattedValue.endsWith('m')))) {`
`118`	`118`	`formattedValue = formattedValue.replace(/(\d+)m$/, '$1mo')`
`119`	`119`	`}`
`120`	`120`