add integration test, add public export

Lms24 · Lms24 · commit 0426e920ef6e · 2025-07-16T10:55:37.000+02:00
diff --git a/dev-packages/browser-integration-tests/suites/integrations/webWorker/assets/worker.js b/dev-packages/browser-integration-tests/suites/integrations/webWorker/assets/worker.js
@@ -0,0 +1,14 @@
+self._sentryDebugIds = {
+  'Error at http://sentry-test.io/worker.js': 'worker-debug-id-789',
+};
+
+self.postMessage({
+  _sentryMessage: true,
+  _sentryDebugIds: self._sentryDebugIds,
+});
+
+self.addEventListener('message', event => {
+  if (event.data.type === 'throw-error') {
+    throw new Error('Worker error for testing');
+  }
+});
diff --git a/dev-packages/browser-integration-tests/suites/integrations/webWorker/init.js b/dev-packages/browser-integration-tests/suites/integrations/webWorker/init.js
@@ -0,0 +1,27 @@
+import * as Sentry from '@sentry/browser';
+
+// Initialize Sentry with webWorker integration
+Sentry.init({
+  dsn: 'https://public@dsn.ingest.sentry.io/1337',
+  debug: true,
+  beforeSend(event) {
+    console.log('xx beforeSend', JSON.stringify(event.exception.values[0].stacktrace.frames, null, 2));
+    return event;
+  },
+});
+
+const worker = new Worker('/worker.js');
+
+worker.addEventListener('message', event => {
+  console.log('xx message', event);
+});
+
+Sentry.addIntegration(Sentry.webWorkerIntegration({ worker }));
+
+const btn = document.getElementById('errWorker');
+
+btn.addEventListener('click', () => {
+  worker.postMessage({
+    type: 'throw-error',
+  });
+});
diff --git a/dev-packages/browser-integration-tests/suites/integrations/webWorker/subject.js b/dev-packages/browser-integration-tests/suites/integrations/webWorker/subject.js
diff --git a/dev-packages/browser-integration-tests/suites/integrations/webWorker/template.html b/dev-packages/browser-integration-tests/suites/integrations/webWorker/template.html
@@ -0,0 +1,9 @@
+<!doctype html>
+<html>
+  <head>
+    <meta charset="utf-8" />
+  </head>
+  <body>
+    <button id="errWorker">Throw error in worker</button>
+  </body>
+</html>
diff --git a/dev-packages/browser-integration-tests/suites/integrations/webWorker/test.ts b/dev-packages/browser-integration-tests/suites/integrations/webWorker/test.ts
@@ -0,0 +1,38 @@
+import { expect } from '@playwright/test';
+import type { Event } from '@sentry/core';
+import { sentryTest } from '../../../utils/fixtures';
+import { getFirstSentryEnvelopeRequest } from '../../../utils/helpers';
+
+sentryTest('Assigns web worker debug IDs when using webWorkerIntegration', async ({ getLocalTestUrl, page }) => {
+  const bundle = process.env.PW_BUNDLE as string | undefined;
+  if (bundle != null && !bundle.includes('esm') && !bundle.includes('cjs')) {
+    sentryTest.skip();
+  }
+
+  const url = await getLocalTestUrl({ testDir: __dirname });
+
+  const errorEventPromise = getFirstSentryEnvelopeRequest<Event>(page, url);
+
+  page.route('**/worker.js', route => {
+    route.fulfill({
+      path: `${__dirname}/assets/worker.js`,
+    });
+  });
+
+  const button = page.locator('#errWorker');
+  await button.click();
+
+  const errorEvent = await errorEventPromise;
+
+  expect(errorEvent.debug_meta?.images).toBeDefined();
+
+  const debugImages = errorEvent.debug_meta?.images || [];
+
+  expect(debugImages.length).toBe(1);
+
+  debugImages.forEach(image => {
+    expect(image.type).toBe('sourcemap');
+    expect(image.debug_id).toEqual('worker-debug-id-789');
+    expect(image.code_file).toEqual('http://sentry-test.io/worker.js');
+  });
+});
diff --git a/packages/browser/src/index.ts b/packages/browser/src/index.ts
@@ -73,3 +73,4 @@ export { openFeatureIntegration, OpenFeatureIntegrationHook } from './integratio
 export { unleashIntegration } from './integrations/featureFlags/unleash';
 export { statsigIntegration } from './integrations/featureFlags/statsig';
 export { diagnoseSdkConnectivity } from './diagnose-sdk';
+export { webWorkerIntegration } from './integrations/webWorker';
