ref: add issue comment

JonasBa · JonasBa · commit 2a49051d09ff · 2024-10-04T12:08:57.000-04:00
diff --git a/dev-packages/e2e-tests/test-applications/node-profiling/index.mjs b/dev-packages/e2e-tests/test-applications/node-profiling/index.mjs
@@ -14,7 +14,8 @@ Sentry.startSpan({ name: 'Precompile test' }, async () => {
   await wait(500);
 });
 
-// assert that we dont override require
+// Test that globalThis.require is not defined by any side effects of the profiling
+// https://github.com/getsentry/sentry-javascript/issues/13662
 if (globalThis.require !== undefined) {
   throw new Error('globalThis.require should not be defined, check that profiling integration is not defining it');
 }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,8 @@ Sentry.startSpan({ name: 'Precompile test' }, async () => {`
`14`	`14`	`await wait(500);`
`15`	`15`	`});`
`16`	`16`
`17`		`-// assert that we dont override require`
	`17`	`+// Test that globalThis.require is not defined by any side effects of the profiling`
	`18`	`+// https://github.com/getsentry/sentry-javascript/issues/13662`
`18`	`19`	`if (globalThis.require !== undefined) {`
`19`	`20`	`throw new Error('globalThis.require should not be defined, check that profiling integration is not defining it');`
`20`	`21`	`}`