actually introduce random numbers

Author: AbhiPrasad

packages/remix/src/utils/serverAdapters/express.ts

@@ -1,3 +1,5 @@
+import { getCurrentHub } from '@sentry/hub';
+import { hasTracingEnabled } from '@sentry/tracing';
 import { extractRequestData, loadModule } from '@sentry/utils';
 import * as domain from 'domain';
 
@@ -17,8 +19,6 @@ import {
   ReactRouterDomPkg,
   ServerBuild,
 } from '../types';
-import { getCurrentHub } from '@sentry/hub';
-import { hasTracingEnabled } from '@sentry/tracing';
 
 function wrapExpressRequestHandler(
   origRequestHandler: ExpressRequestHandler,
@@ -67,7 +67,9 @@ function wrapExpressRequestHandler(
  */
 export function wrapExpressCreateRequestHandler(
   origCreateRequestHandler: ExpressCreateRequestHandler,
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
 ): (options: any) => ExpressRequestHandler {
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   return function (this: unknown, options: any): ExpressRequestHandler {
     const newBuild = instrumentBuild((options as ExpressCreateRequestHandlerOptions).build);
     const requestHandler = origCreateRequestHandler.call(this, { ...options, build: newBuild });

packages/remix/test/integration/app/routes/scope-bleed/$id.tsx

@@ -4,7 +4,8 @@ import * as Sentry from '@sentry/remix';
 
 export const loader: LoaderFunction = async ({ params: { id } }) => {
   // Set delay to simulate requests at the same time
-  await new Promise(resolve => setTimeout(resolve, 5000 - (parseInt(id || '', 10) * 1000 - 1000 - 14)));
+  const randomNum = Math.floor(Math.random() * 15) + 1;
+  await new Promise(resolve => setTimeout(resolve, 4000 - (parseInt(id || '', 10) * 1000 - randomNum)));
   Sentry.setTag(`tag${id}`, id);
   return json({ test: 'test' });
 };

packages/remix/test/integration/test/server/loader.test.ts

@@ -145,8 +145,9 @@ describe.each(['builtin', 'express'])('Remix API Loaders with adapter = %s', ada
         const url = `${baseURL}/scope-bleed/${id}`;
         // Send requests with 1 sec delays, but they should all resolve at the same time
         // See packages/remix/test/integration/app/routes/scope-bleed/$id.tsx
-        // for server-side set up. Add 13 for a bit of randomness.
-        await new Promise(resolve => setTimeout(resolve, id * 1000 - 1000 + 13));
+        // for server-side set up.
+        const randomNum = Math.floor(Math.random() * 15) + 1;
+        await new Promise(resolve => setTimeout(resolve, id * 1000 - 1000 + randomNum));
         const envelope = await getEnvelopeRequest(url);
         const transaction = envelope[2];