Use bind instead of domain.run plus minor runner fixes

Author: kamilogorek

packages/nextjs/src/utils/handlers.ts

@@ -19,8 +19,7 @@ export const withSentry = (handler: NextApiHandler): WrappedNextApiHandler => {
     const local = domain.create();
     local.add(req);
     local.add(res);
-
-    local.run(async () => {
+    return local.bind(async () => {
       try {
         const currentScope = getCurrentHub().getScope();
 
@@ -86,6 +85,6 @@ export const withSentry = (handler: NextApiHandler): WrappedNextApiHandler => {
           // no-empty
         }
       }
-    });
+    })();
   };
 };

packages/nextjs/test/integration/test/client/errorClick.js

@@ -2,7 +2,6 @@ const { waitForAll } = require('../utils/common');
 const { expectRequestCount, isEventRequest, expectEvent } = require('../utils/client');
 
 module.exports = async ({ page, url, requests }) => {
-  console.log(page, url, requests);
   await page.goto(`${url}/errorClick`);
 
   await waitForAll([page.click('button'), page.waitForRequest(isEventRequest)]);

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

@@ -66,7 +66,7 @@ module.exports.run = async ({
 
     let scenarios = await fs.readdir(scenariosDir);
     if (argv.filter) {
-      scenarios = scenarios.filter(file => file.toLowerCase().includes(argv.filter));
+      scenarios = scenarios.filter(file => file.toLowerCase().includes(argv.filter.toLowerCase()));
     }
     scenarios = scenarios.map(s => path.resolve(scenariosDir, s));
 

packages/nextjs/test/integration/test/utils/common.js

@@ -37,7 +37,7 @@ const logIf = (condition, message, input, depth = 4) => {
   if (condition) {
     console.log(message);
     if (input) {
-      console.log(inspect(input, { depth }));
+      console.log(JSON.stringify(input, null, 2));
     }
   }
 };