add test demonstrating correct isolationScope behavior

Author: Lms24

dev-packages/node-integration-tests/suites/express/handle-error-withScope/server.ts renamed to dev-packages/node-integration-tests/suites/express/handle-error-scope-data-loss/server.ts

@@ -21,6 +21,11 @@ app.get('/test/withScope', () => {
   });
 });
 
+app.get('/test/isolationScope', () => {
+  Sentry.getIsolationScope().setTag('isolation-scope', 'tag');
+  throw new Error('isolation_test_error');
+});
+
 app.use(Sentry.Handlers.errorHandler());
 
 startExpressServerAndSendPortToRunner(app);

dev-packages/node-integration-tests/suites/express/handle-error-withScope/test.ts renamed to dev-packages/node-integration-tests/suites/express/handle-error-scope-data-loss/test.ts

@@ -13,7 +13,7 @@ afterAll(() => {
  *
  * This test nevertheless covers the behavior so that we're aware.
  */
-test('applies withScope scope to thrown error', done => {
+test('withScope scope is NOT applied to thrown error caught by global handler', done => {
   const runner = createRunner(__dirname, 'server.ts')
     .ignore('session', 'sessions')
     .expect({
@@ -49,3 +49,40 @@ test('applies withScope scope to thrown error', done => {
 
   expect(() => runner.makeRequest('get', '/test/withScope')).rejects.toThrow();
 });
+
+test('isolation scope is applied to thrown error caught by global handler', done => {
+  const runner = createRunner(__dirname, 'server.ts')
+    .ignore('session', 'sessions')
+    .expect({
+      event: {
+        exception: {
+          values: [
+            {
+              mechanism: {
+                type: 'middleware',
+                handled: false,
+              },
+              type: 'Error',
+              value: 'isolation_test_error',
+              stacktrace: {
+                frames: expect.arrayContaining([
+                  expect.objectContaining({
+                    function: expect.any(String),
+                    lineno: expect.any(Number),
+                    colno: expect.any(Number),
+                  }),
+                ]),
+              },
+            },
+          ],
+        },
+        tags: {
+          global: 'tag',
+          'isolation-scope': 'tag',
+        },
+      },
+    })
+    .start(done);
+
+  expect(() => runner.makeRequest('get', '/test/isolationScope')).rejects.toThrow();
+});