test run

Author: JonasBa

.github/workflows/build.yml

@@ -1205,7 +1205,8 @@ jobs:
       - name: Run E2E test
         working-directory: dev-packages/e2e-tests/test-applications/${{ matrix.test-application }}
         timeout-minutes: 10
-        run: xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- yarn test:assert
+        run:
+          xvfb-run --auto-servernum --server-args="-screen 0 1280x960x24" -- yarn test:assert
 
   job_required_jobs_passed:
     name: All required jobs passed or were skipped

dev-packages/e2e-tests/test-applications/node-profiling/index.mjs

@@ -1,17 +1,20 @@
+// This tests asserts that @sentr/profiling-node is not patching globalThis values, which
+// breaks our runtime detection and can break instrumentation
+// https://github.com/getsentry/sentry-javascript/issues/14525#issuecomment-2511208064
+
 function assertUnpatechedRequire(cycle) {
   if (globalThis.require !== undefined) {
     // Test that globalThis.require is not defined by any side effects of the profiling
     // https://github.com/getsentry/sentry-javascript/issues/13662
     throw new Error(
-      `globalThis.require should not be defined ${cycle}, check that profiling integration is not defining it, received: ` +
+      `globalThis.require should not be defined, check that profiling integration is not defining it, received: ` +
         typeof globalThis.require,
     );
   }
 }
 
 import * as Sentry from '@sentry/node';
 import { nodeProfilingIntegration } from '@sentry/profiling-node';
-assertUnpatechedRequire('after importing sentry/profiling-node');
 const wait = ms => new Promise(resolve => setTimeout(resolve, ms));
 
 Sentry.init({
@@ -25,4 +28,4 @@ Sentry.startSpan({ name: 'Precompile test' }, async () => {
   await wait(500);
 });
 
-assertUnpatechedRequire('after a span was created');
+assertUnpatechedRequire();