test(nextjs): New server-side integration tests.

Author: onurtemizkan

packages/nextjs/jest.config.js

@@ -4,5 +4,5 @@ module.exports = {
   ...baseConfig,
   // This prevents the build tests from running when unit tests run. (If they do, they fail, because the build being
   // tested hasn't necessarily run yet.)
-  testPathIgnorePatterns: ['<rootDir>/test/buildProcess/', '<rootDir>/test/integration/'],
+  testPathIgnorePatterns: ['<rootDir>/test/buildProcess/'],
 };

packages/nextjs/test/integration/jest.config.js

@@ -0,0 +1,11 @@
+const baseConfig = require('../../jest.config.js');
+
+module.exports = {
+  ...baseConfig,
+  testMatch: [`${__dirname}/test/server/**/*.test.ts`],
+  testPathIgnorePatterns: [`${__dirname}/test/client`],
+  detectOpenHandles: true,
+  forceExit: true,
+  testTimeout: 30000,
+  setupFilesAfterEnv: [`${__dirname}/jest.setup.js`],
+};

packages/nextjs/test/integration/jest.setup.js

@@ -0,0 +1,8 @@
+global.console = {
+  ...console,
+  log: jest.fn(),
+  info: jest.fn(),
+  warn: jest.fn(),
+  error: jest.fn(),
+  // console.debug is available
+};

packages/nextjs/test/integration/test/runner.js

@@ -0,0 +1,69 @@
+import { NextTestEnv } from '../utils/server';
+
+describe('CommonJS API Endpoints', () => {
+  it('should not intercept unwrapped request', async () => {
+    const env = await NextTestEnv.init();
+    const unwrappedRoute = '/api/wrapApiHandlerWithSentry/unwrapped/cjsExport';
+    const url = `${env.url}${unwrappedRoute}`;
+
+    const unwrappedEnvelope = await env.getEnvelopeRequest({
+      url,
+      envelopeType: 'transaction',
+      endServer: false,
+    });
+
+    expect(unwrappedEnvelope[2]).toMatchObject({
+      contexts: {
+        trace: {
+          op: 'http.server',
+          status: 'ok',
+          tags: { 'http.status_code': '200' },
+        },
+      },
+      transaction: `GET ${unwrappedRoute}`,
+      type: 'transaction',
+      request: {
+        url,
+      },
+    });
+
+    const response = await env.getAPIResponse(url);
+
+    expect(response).toMatchObject({
+      success: true,
+    });
+  });
+
+  it('should intercept wrapped request', async () => {
+    const env = await NextTestEnv.init();
+    const wrappedRoute = '/api/wrapApiHandlerWithSentry/wrapped/cjsExport';
+    const url = `${env.url}${wrappedRoute}`;
+
+    const wrappedEnvelope = await env.getEnvelopeRequest({
+      url,
+      envelopeType: 'transaction',
+      endServer: false,
+    });
+
+    expect(wrappedEnvelope[2]).toMatchObject({
+      contexts: {
+        trace: {
+          op: 'http.server',
+          status: 'ok',
+          tags: { 'http.status_code': '200' },
+        },
+      },
+      transaction: `GET ${wrappedRoute}`,
+      type: 'transaction',
+      request: {
+        url,
+      },
+    });
+
+    const response = await env.getAPIResponse(url);
+
+    expect(response).toMatchObject({
+      success: true,
+    });
+  });
+});

packages/nextjs/test/integration/test/server.js

@@ -0,0 +1,18 @@
+import { NextTestEnv } from '../utils/server';
+
+// This test asserts that our wrapping of `res.end` doesn't break API routes on Vercel if people call `res.json` or
+// `res.send` multiple times in one request handler.
+//  https://github.com/getsentry/sentry-javascript/issues/6670
+it.skip('should not break API routes on Vercel if people call res.json or res.send multiple times in one request handler', async () => {
+  if (process.env.NODE_MAJOR === '10') {
+    console.log('not running doubleEndMethodOnVercel test on Node 10');
+    return;
+  }
+  const env = await NextTestEnv.init();
+  const url = `${env.url}/api/doubleEndMethodOnVercel`;
+  const response = await env.getAPIResponse(url);
+
+  expect(response).toMatchObject({
+    success: true,
+  });
+});