test: test isolation scope

Author: logaretm

dev-packages/e2e-tests/test-applications/nitro-3/package.json

@@ -5,7 +5,7 @@
   "type": "module",
   "scripts": {
     "build": "nitro build",
-    "start": "PORT=3030 NODE_OPTIONS='--import ./instrument.mjs' npx srvx --prod .output/",
+    "start": "PORT=3030 NODE_OPTIONS='--import ./instrument.mjs' srvx --prod .output/",
     "clean": "npx rimraf node_modules pnpm-lock.yaml .output",
     "test": "playwright test",
     "test:build": "pnpm install && pnpm build",
@@ -17,8 +17,10 @@
   "devDependencies": {
     "@playwright/test": "~1.56.0",
     "@sentry-internal/test-utils": "link:../../../test-utils",
+    "@sentry/core": "latest || *",
     "nitro": "https://pkg.pr.new/nitrojs/nitro@4001",
-    "rolldown": "latest"
+    "rolldown": "latest",
+    "srvx": "^0.11.2"
   },
   "volta": {
     "extends": "../../package.json"

dev-packages/e2e-tests/test-applications/nitro-3/routes/test-isolation/[id].ts

@@ -0,0 +1,10 @@
+import { getDefaultIsolationScope, setTag } from '@sentry/core';
+import { defineHandler } from 'nitro/h3';
+
+export default defineHandler(() => {
+  setTag('my-isolated-tag', true);
+  // Check if the tag leaked into the default (global) isolation scope
+  setTag('my-global-scope-isolated-tag', getDefaultIsolationScope().getScopeData().tags['my-isolated-tag']);
+
+  throw new Error('Isolation test error');
+});

dev-packages/e2e-tests/test-applications/nitro-3/tests/isolation.test.ts

@@ -0,0 +1,25 @@
+import { expect, test } from '@playwright/test';
+import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
+
+test('Isolation scope prevents tag leaking between requests', async ({ request }) => {
+  const transactionEventPromise = waitForTransaction('nitro-3', event => {
+    return event?.transaction === 'GET /test-isolation/1';
+  });
+
+  const errorPromise = waitForError('nitro-3', event => {
+    return !event.type && event.exception?.values?.some(v => v.value === 'Isolation test error');
+  });
+
+  await request.get('/test-isolation/1').catch(() => {
+    // noop - route throws
+  });
+
+  const transactionEvent = await transactionEventPromise;
+  const error = await errorPromise;
+
+  // Assert that isolation scope works properly
+  expect(error.tags?.['my-isolated-tag']).toBe(true);
+  expect(error.tags?.['my-global-scope-isolated-tag']).not.toBeDefined();
+  expect(transactionEvent.tags?.['my-isolated-tag']).toBe(true);
+  expect(transactionEvent.tags?.['my-global-scope-isolated-tag']).not.toBeDefined();
+});