fix and add tests

Author: Lms24

dev-packages/e2e-tests/test-applications/nextjs-13/tests/client/click-error.test.ts

@@ -13,8 +13,18 @@ test('should send error for faulty click handlers', async ({ page }) => {
 
   expect(errorEvent).toBeDefined();
 
-  const frames = errorEvent?.exception?.values?.[0]?.stacktrace?.frames;
+  const exception = errorEvent?.exception?.values?.[0];
+
+  expect(exception?.mechanism).toEqual({
+    type: 'auto.browser.browserapierrors.addEventListener',
+    handled: false,
+    data: {
+      handler: 'bound ed',
+      target: 'EventTarget',
+    },
+  });
 
+  const frames = exception?.stacktrace?.frames;
   await test.step('error should have a non-url-encoded top frame in route with parameter', () => {
     if (process.env.TEST_ENV === 'development') {
       // In dev mode we want to check local source mapping

dev-packages/e2e-tests/test-applications/nextjs-13/tests/client/sessions.test.ts

@@ -5,6 +5,7 @@ test('should report healthy sessions', async ({ page }) => {
   test.skip(process.env.TEST_ENV === 'development', 'test is flakey in dev mode');
 
   const sessionPromise = waitForSession('nextjs-13', session => {
+    console.log('session', session);
     return session.init === true && session.status === 'ok' && session.errors === 0;
   });
 

dev-packages/e2e-tests/test-applications/nextjs-13/tests/server/getServerSideProps.test.ts

@@ -23,7 +23,7 @@ test('Should report an error event for errors thrown in getServerSideProps', asy
     exception: {
       values: [
         {
-          mechanism: { handled: false, type: 'generic' },
+          mechanism: { handled: false, type: 'auto.function.nextjs.wrapped' },
           type: 'Error',
           value: 'getServerSideProps Error',
           stacktrace: {
@@ -110,7 +110,7 @@ test('Should report an error event for errors thrown in getServerSideProps in pa
     exception: {
       values: [
         {
-          mechanism: { handled: false, type: 'generic' },
+          mechanism: { handled: false, type: 'auto.function.nextjs.wrapped' },
           type: 'Error',
           value: 'custom page extension error',
           stacktrace: {

dev-packages/e2e-tests/test-applications/nextjs-13/tests/server/pages-router-api-endpoints.test.ts

@@ -30,7 +30,7 @@ test('Should report an error event for errors thrown in pages router api routes'
               function: 'withSentry',
             },
             handled: false,
-            type: 'instrument',
+            type: 'auto.http.nextjs.apiHandler',
           },
           stacktrace: { frames: expect.arrayContaining([]) },
           type: 'Error',

dev-packages/e2e-tests/test-applications/nextjs-13/tests/server/server-component-error.test.ts

@@ -20,7 +20,7 @@ test('Should capture an error thrown in a server component', async ({ page }) =>
     exception: {
       values: [
         {
-          mechanism: { handled: false, type: 'generic' },
+          mechanism: { handled: false, type: 'auto.function.nextjs.serverComponent' },
           type: 'Error',
           value: 'RSC error',
         },

dev-packages/e2e-tests/test-applications/nextjs-15/tests/nested-rsc-error.test.ts

@@ -41,4 +41,9 @@ test('Should capture errors from nested server components when `Sentry.captureRe
     router_path: '/nested-rsc-error/[param]',
     request_path: '/nested-rsc-error/123',
   });
+
+  expect(errorEvent.exception?.values?.[0]?.mechanism).toEqual({
+    handled: false,
+    type: 'auto.function.nextjs.onRequestError',
+  });
 });

dev-packages/e2e-tests/test-applications/nextjs-15/tests/streaming-rsc-error.test.ts

@@ -41,4 +41,9 @@ test('Should capture errors for crashing streaming promises in server components
     router_path: '/streaming-rsc-error/[param]',
     request_path: '/streaming-rsc-error/123',
   });
+
+  expect(errorEvent.exception?.values?.[0]?.mechanism).toEqual({
+    handled: false,
+    type: 'auto.function.nextjs.onRequestError',
+  });
 });

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/client-errors.test.ts

@@ -35,4 +35,13 @@ test('Sends a client-side exception to Sentry', async ({ page }) => {
     trace_id: expect.stringMatching(/[a-f0-9]{32}/),
     span_id: expect.stringMatching(/[a-f0-9]{16}/),
   });
+
+  expect(errorEvent.exception?.values?.[0]?.mechanism).toEqual({
+    handled: false,
+    type: 'auto.browser.browserapierrors.addEventListener',
+    data: {
+      handler: expect.stringContaining('bound uM'),
+      target: 'EventTarget',
+    },
+  });
 });

dev-packages/e2e-tests/test-applications/nextjs-app-dir/tests/edge.test.ts

@@ -19,6 +19,11 @@ test('Should record exceptions for faulty edge server components', async ({ page
   expect(errorEvent.tags?.['my-global-scope-isolated-tag']).not.toBeDefined();
 
   expect(errorEvent.transaction).toBe(`Page Server Component (/edge-server-components/error)`);
+
+  expect(errorEvent.exception?.values?.[0]?.mechanism).toEqual({
+    handled: false,
+    type: 'auto.function.nextjs.serverComponent',
+  });
 });
 
 test('Should record transaction for edge server components', async ({ page }) => {

dev-packages/e2e-tests/test-applications/nextjs-orpc/tests/orpc-error.test.ts

@@ -19,4 +19,10 @@ test('should capture orpc error', async ({ page }) => {
       }),
     ],
   });
+
+  // orpc errors are captured manually by the orpc middleware (user-land)
+  expect(orpcError.exception?.values?.[0]?.mechanism).toEqual({
+    handled: true,
+    type: 'generic',
+  });
 });