fix bundle tests

Author: mydea

dev-packages/browser-integration-tests/suites/manual-client/skip-init-browser-extension/test.ts

@@ -1,5 +1,6 @@
 import { expect } from '@playwright/test';
 import { sentryTest } from '../../../utils/fixtures';
+import { hasDebugLogs } from '../../../utils/helpers';
 
 sentryTest(
   'should not initialize when inside a Firefox/Safari browser extension',
@@ -18,9 +19,14 @@ sentryTest(
     });
 
     expect(isInitialized).toEqual(false);
-    expect(errorLogs.length).toEqual(1);
-    expect(errorLogs[0]).toEqual(
-      '[Sentry] You cannot run Sentry this way in a browser extension, check: https://docs.sentry.io/platforms/javascript/best-practices/browser-extensions/',
-    );
+
+    if (hasDebugLogs()) {
+      expect(errorLogs.length).toEqual(1);
+      expect(errorLogs[0]).toEqual(
+        '[Sentry] You cannot run Sentry this way in a browser extension, check: https://docs.sentry.io/platforms/javascript/best-practices/browser-extensions/',
+      );
+    } else {
+      expect(errorLogs.length).toEqual(0);
+    }
   },
 );

dev-packages/browser-integration-tests/suites/manual-client/skip-init-chrome-extension/test.ts

@@ -1,5 +1,6 @@
 import { expect } from '@playwright/test';
 import { sentryTest } from '../../../utils/fixtures';
+import { hasDebugLogs } from '../../../utils/helpers';
 
 sentryTest('should not initialize when inside a Chrome browser extension', async ({ getLocalTestUrl, page }) => {
   const errorLogs: string[] = [];
@@ -16,8 +17,13 @@ sentryTest('should not initialize when inside a Chrome browser extension', async
   });
 
   expect(isInitialized).toEqual(false);
-  expect(errorLogs.length).toEqual(1);
-  expect(errorLogs[0]).toEqual(
-    '[Sentry] You cannot run Sentry this way in a browser extension, check: https://docs.sentry.io/platforms/javascript/best-practices/browser-extensions/',
-  );
+
+  if (hasDebugLogs()) {
+    expect(errorLogs.length).toEqual(1);
+    expect(errorLogs[0]).toEqual(
+      '[Sentry] You cannot run Sentry this way in a browser extension, check: https://docs.sentry.io/platforms/javascript/best-practices/browser-extensions/',
+    );
+  } else {
+    expect(errorLogs.length).toEqual(0);
+  }
 });

dev-packages/browser-integration-tests/utils/helpers.ts

@@ -302,6 +302,14 @@ export function shouldSkipFeatureFlagsTest(): boolean {
   return bundle != null && !bundle.includes('esm') && !bundle.includes('cjs');
 }
 
+/**
+ * Returns true if the current bundle has debug logs.
+ */
+export function hasDebugLogs(): boolean {
+  const bundle = process.env.PW_BUNDLE;
+  return !bundle?.includes('min');
+}
+
 /**
  * Waits until a number of requests matching urlRgx at the given URL arrive.
  * If the timeout option is configured, this function will abort waiting, even if it hasn't received the configured