Remove redirect test from nexctjs-14, remove redirect events from remix

integration tests

Author: andreiborza

dev-packages/e2e-tests/test-applications/nextjs-14/app/generation-functions/with-redirect/page.tsx

@@ -108,20 +108,3 @@ test('Should send a transaction and an error event for a faulty generateViewport
 
   expect(errorEvent.transaction).toBe('Page.generateViewport (/generation-functions)');
 });
-
-test('Should send a transaction event with correct status for a generateMetadata() function invocation with redirect()', async ({
-  page,
-}) => {
-  const testTitle = 'redirect-foobar';
-
-  const transactionPromise = waitForTransaction('nextjs-14', async transactionEvent => {
-    return (
-      transactionEvent.contexts?.trace?.data?.['http.target'] ===
-      `/generation-functions/with-redirect?metadataTitle=${testTitle}`
-    );
-  });
-
-  await page.goto(`/generation-functions/with-redirect?metadataTitle=${testTitle}`);
-
-  expect((await transactionPromise).contexts?.trace?.status).toBe('ok');
-});

dev-packages/e2e-tests/test-applications/nextjs-14/tests/generation-functions.test.ts

@@ -152,28 +152,15 @@ describe('Remix API Actions', () => {
 
     const envelopes = await env.getMultipleEnvelopeRequest({
       url,
-      count: 3,
+      count: 2,
       method: 'post',
       envelopeType: ['transaction', 'event'],
     });
 
-    const [transaction_1, transaction_2] = envelopes.filter(envelope => envelope[1].type === 'transaction');
+    const [transaction] = envelopes.filter(envelope => envelope[1].type === 'transaction');
     const [event] = envelopes.filter(envelope => envelope[1].type === 'event');
 
-    assertSentryTransaction(transaction_1[2], {
-      contexts: {
-        trace: {
-          op: 'http.server',
-          status: 'ok',
-          data: {
-            'http.response.status_code': 302,
-          },
-        },
-      },
-      transaction: `POST action-json-response/:id`,
-    });
-
-    assertSentryTransaction(transaction_2[2], {
+    assertSentryTransaction(transaction[2], {
       contexts: {
         trace: {
           op: 'http.server',

packages/remix/test/integration/test/server/instrumentation/action.test.ts

@@ -123,27 +123,14 @@ describe('Remix API Loaders', () => {
 
     const envelopes = await env.getMultipleEnvelopeRequest({
       url,
-      count: 3,
+      count: 2,
       envelopeType: ['transaction', 'event'],
     });
 
-    const [transaction_1, transaction_2] = envelopes.filter(envelope => envelope[1].type === 'transaction');
+    const [transaction] = envelopes.filter(envelope => envelope[1].type === 'transaction');
     const [event] = envelopes.filter(envelope => envelope[1].type === 'event');
 
-    assertSentryTransaction(transaction_1[2], {
-      contexts: {
-        trace: {
-          op: 'http.server',
-          status: 'ok',
-          data: {
-            'http.response.status_code': 302,
-          },
-        },
-      },
-      transaction: `GET loader-json-response/:id`,
-    });
-
-    assertSentryTransaction(transaction_2[2], {
+    assertSentryTransaction(transaction[2], {
       contexts: {
         trace: {
           op: 'http.server',