test: write an e2e test

Author: logaretm

dev-packages/browser-integration-tests/suites/integrations/cultureContext/init.js

@@ -0,0 +1,20 @@
+import * as Sentry from '@sentry/browser';
+
+window.Sentry = Sentry;
+
+const integrations = Sentry.getDefaultIntegrations({}).filter(
+  defaultIntegration => defaultIntegration.name === 'CultureContext',
+);
+
+const client = new Sentry.BrowserClient({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  transport: Sentry.makeFetchTransport,
+  stackParser: Sentry.defaultStackParser,
+  integrations: integrations,
+});
+
+const scope = new Sentry.Scope();
+scope.setClient(client);
+client.init();
+
+window._sentryScope = scope;

dev-packages/browser-integration-tests/suites/integrations/cultureContext/subject.js

@@ -0,0 +1 @@
+window._sentryScope.captureException(new Error('test error'));

dev-packages/browser-integration-tests/suites/integrations/cultureContext/test.ts

@@ -0,0 +1,18 @@
+import { expect } from '@playwright/test';
+import type { Event } from '@sentry/core';
+import { sentryTest } from '../../../utils/fixtures';
+import { getFirstSentryEnvelopeRequest } from '../../../utils/helpers';
+
+sentryTest('cultureContextIntegration captures locale, timezone, and calendar', async ({ getLocalTestUrl, page }) => {
+  const url = await getLocalTestUrl({ testDir: __dirname });
+
+  const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
+
+  expect(eventData.exception?.values).toHaveLength(1);
+
+  expect(eventData.contexts?.culture).toEqual({
+    locale: expect.any(String),
+    timezone: expect.any(String),
+    calendar: expect.any(String),
+  });
+});

packages/browser/src/integrations/culturecontext.ts

@@ -1,5 +1,6 @@
 import type { CultureContext, IntegrationFn } from '@sentry/core';
-import { defineIntegration, GLOBAL_OBJ } from '@sentry/core';
+import { defineIntegration } from '@sentry/core';
+import { WINDOW } from '../helpers';
 
 const INTEGRATION_NAME = 'CultureContext';
 
@@ -40,11 +41,12 @@ export const cultureContextIntegration = defineIntegration(_cultureContextIntegr
  */
 function getCultureContext(): CultureContext | undefined {
   try {
-    if (typeof (GLOBAL_OBJ as { Intl?: typeof Intl }).Intl === 'undefined') {
+    const intl = (WINDOW as { Intl?: typeof Intl }).Intl;
+    if (!intl) {
       return undefined;
     }
 
-    const options = Intl.DateTimeFormat().resolvedOptions();
+    const options = intl.DateTimeFormat().resolvedOptions();
 
     return {
       locale: options.locale,