add profiling integration in test

Author: s1gr1d

dev-packages/browser-integration-tests/suites/profiling/legacyMode/subject.js

@@ -1,10 +1,11 @@
 import * as Sentry from '@sentry/browser';
+import { browserProfilingIntegration } from '@sentry/browser';
 
 window.Sentry = Sentry;
 
 Sentry.init({
   dsn: 'https://[email protected]/1337',
-  integrations: [Sentry.browserProfilingIntegration()],
+  integrations: [browserProfilingIntegration()],
   tracesSampleRate: 1,
   profilesSampleRate: 1,
 });

dev-packages/browser-integration-tests/suites/profiling/legacyMode/test.ts

@@ -83,16 +83,22 @@ sentryTest('sends profile envelope in legacy mode', async ({ page, getLocalTestU
 
   const functionNames = profile.frames.map(frame => frame.function).filter(name => name !== '');
 
-  expect(functionNames).toEqual(
-    expect.arrayContaining([
-      '_startRootSpan',
-      'withScope',
-      'createChildOrRootSpan',
-      'startSpanManual',
-      'startProfileForSpan',
-      'startJSSelfProfile',
-    ]),
-  );
+  if (!(process.env.PW_BUNDLE || '').startsWith('bundle')) {
+    // In bundled mode, function names are minified
+    expect(functionNames.length).toBeGreaterThan(0);
+    expect((functionNames as string[]).every(name => name?.length > 0)).toBe(true); // Just make sure they're not empty strings
+  } else {
+    expect(functionNames).toEqual(
+      expect.arrayContaining([
+        '_startRootSpan',
+        'withScope',
+        'createChildOrRootSpan',
+        'startSpanManual',
+        'startProfileForSpan',
+        'startJSSelfProfile',
+      ]),
+    );
+  }
 
   expect(profile.thread_metadata).toHaveProperty('0');
   expect(profile.thread_metadata['0']).toHaveProperty('name');

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

@@ -36,6 +36,7 @@ const IMPORTED_INTEGRATION_CDN_BUNDLE_PATHS: Record<string, string> = {
   feedbackIntegration: 'feedback',
   moduleMetadataIntegration: 'modulemetadata',
   graphqlClientIntegration: 'graphqlclient',
+  browserProfilingIntegration: 'browserprofiling',
   // technically, this is not an integration, but let's add it anyway for simplicity
   makeMultiplexedTransport: 'multiplexedtransport',
 };