chore: use form body parser in fastify app

Author: rustworthy

dev-packages/node-integration-tests/utils/index.ts

@@ -118,8 +118,12 @@ async function makeRequest(
       await axios.post(url, axiosConfig);
     }
   } catch (e) {
-    // We sometimes expect the request to fail, but not the test.
-    // So, we do nothing.
+    // We sometimes expect the request to fail, but not the test, so we are just logging the error.
+    // Beware though! There are also framework specific errors, e.g. fastify can give a 415 with
+    // FST_ERR_CTP_INVALID_MEDIA_TYPE if handling of application/x-www-form-urlencoded is not configure
+    // properly when building a test application / server. Errors of these kind can lead to requests
+    // not actually hitting the handler, and so no tracing will be happenning. This in its turn will
+    // most like make the test time out.
     logger.warn(e);
   }
 }

packages/remix/test/integration/package.json

@@ -7,6 +7,7 @@
     "start": "remix-serve build"
   },
   "dependencies": {
+    "@fastify/formbody": "^7.4.0",
     "@mcansh/remix-fastify": "^3.2.2",
     "@remix-run/express": "1.17.0",
     "@remix-run/node": "1.17.0",

packages/remix/test/integration/test/server/utils/helpers.ts

@@ -5,6 +5,7 @@ import { createRequestHandler as createExpressRequestHandler } from '@remix-run/
 import { wrapExpressCreateRequestHandler, wrapFastifyCreateRequestHandler } from '@sentry/remix';
 import express from 'express';
 import fastify from 'fastify';
+import formBody from '@fastify/formbody'
 
 import { TestEnv } from '../../../../../../../dev-packages/node-integration-tests/utils';
 
@@ -32,6 +33,7 @@ const runExpressApp = (adapter: Adapter.Builtin | Adapter.Express): Promise<http
 
 const runFastifyApp = (): Promise<http.Server> => new Promise(res => {
   const app = fastify();
+  app.register(formBody);
   // @ts-ignore
   app.all('*', adapters[Adapter.Fastify]({ build: require('../../../build') }));
   app.listen({port: 0}, (_err, _addr) => {