Map deprecated timezone in shopify/dates

.changeset/rude-countries-tickle.md → .changeset/light-students-learn.md

@@ -1,5 +1,5 @@
 ---
-'@shopify/react-i18n': patch
+'@shopify/dates': patch
 ---
 
 Map deprecated timezone

packages/dates/src/utilities/formatDate.ts

@@ -1,3 +1,5 @@
+import {mapDeprecatedTimezones} from '../map-deprecated-timezones';
+
 const intl = new Map<string, Intl.DateTimeFormat>();
 export function memoizedGetDateTimeFormat(
   locales?: string | string[],
@@ -39,6 +41,7 @@ export function formatDate(
   locales: string | string[],
   options: FormatDateOptions = {},
 ) {
+  console.log('FormatDate', options);
   const hourCycleRequired =
     resolvedOptions != null &&
     options.hour12 === false &&
@@ -58,7 +61,14 @@ export function formatDate(
     }).format(adjustedDate);
   }
 
-  return memoizedGetDateTimeFormat(locales, options).format(date);
+  console.log(options.timeZone);
+
+  return memoizedGetDateTimeFormat(locales, {
+    ...options,
+    timeZone: options.timeZone
+      ? mapDeprecatedTimezones(options.timeZone)
+      : undefined,
+  }).format(date);
 }
 
 export function dateTimeFormatCacheKey(

packages/dates/src/utilities/tests/formatDate.test.ts

@@ -0,0 +1,22 @@
+import {formatDate} from '../formatDate';
+
+jest.mock('../../map-deprecated-timezones', () => ({
+  mapDeprecatedTimezones: jest.fn(),
+}));
+
+const mapDeprecatedTimezones: jest.Mock = jest.requireMock(
+  '../../map-deprecated-timezones',
+).mapDeprecatedTimezones;
+
+describe('formatDate', () => {
+  it('maps the deprecated timezone', () => {
+    const date = new Date();
+    const locale = 'en-UA';
+    const options: Intl.DateTimeFormatOptions = {
+      timeZone: 'Europe/Kyiv',
+    };
+
+    formatDate(date, locale, options);
+    expect(mapDeprecatedTimezones).toHaveBeenCalledWith('Europe/Kyiv');
+  });
+});

packages/react-i18n/src/i18n.ts

@@ -11,7 +11,6 @@ import {
   TimeUnit,
   isLessThanOneWeekAway,
   isLessThanOneYearAway,
-  mapDeprecatedTimezones,
 } from '@shopify/dates';
 import {
   formatName as importedFormatName,
@@ -133,7 +132,7 @@ export class I18n {
     this.locale = locale;
     this.defaultCountry = country;
     this.defaultCurrency = currency;
-    this.defaultTimezone = this.normalizeTimezone(timezone);
+    this.defaultTimezone = timezone;
     this.pseudolocalize = pseudolocalize;
     this.defaultInterpolate = interpolate;
     this.onError = onError || this.defaultOnError;
@@ -331,32 +330,25 @@ export class I18n {
   ): string {
     const {locale, defaultTimezone} = this;
     const {timeZone = defaultTimezone} = options;
-    const normalizedTimezone = this.normalizeTimezone(timeZone);
 
     const {style = undefined, ...formatOptions} = options || {};
 
     if (style) {
       switch (style) {
         case DateStyle.Humanize:
-          return this.humanizeDate(date, {
-            ...formatOptions,
-            timeZone: normalizedTimezone,
-          });
+          return this.humanizeDate(date, {...formatOptions, timeZone});
         case DateStyle.DateTime:
           return this.formatDateTime(date, {
             ...formatOptions,
-            timeZone: normalizedTimezone,
+            timeZone,
             ...dateStyle[style],
           });
         default:
           return this.formatDate(date, {...formatOptions, ...dateStyle[style]});
       }
     }
 
-    return formatDate(date, locale, {
-      ...formatOptions,
-      timeZone: normalizedTimezone,
-    });
+    return formatDate(date, locale, {...formatOptions, timeZone});
   }
 
   ordinal(amount: number) {
@@ -833,8 +825,4 @@ export class I18n {
   private defaultOnError(error: I18nError) {
     throw error;
   }
-
-  private normalizeTimezone(timeZone?: string) {
-    return timeZone ? mapDeprecatedTimezones(timeZone) : undefined;
-  }
 }