chore: use path without query strings

package.json

@@ -186,6 +186,7 @@
   },
   "devDependencies": {
     "@babel/preset-react": "^7.12.13",
+    "@fastify/url-data": "^5.3.1",
     "@netlify/eslint-config-node": "^7.0.0",
     "@netlify/functions": "^1.6.0",
     "@types/fs-extra": "^11.0.1",

tests/integration/utils/mock-api-vitest.mjs

@@ -1,13 +1,14 @@
 import { isDeepStrictEqual } from 'util'
 
+import fastifyUrlData from '@fastify/url-data'
 import fastify from 'fastify'
 import { afterAll, beforeAll, beforeEach } from 'vitest'
 
 // Replace mock-api.cjs with this once everything migrated
 
 const addRequest = (requests, request) => {
   requests.push({
-    path: request.url,
+    path: request.urlData().path,
     body: request.body,
     method: request.method,
     headers: request.headers,
@@ -22,6 +23,7 @@ const clearRequests = (requests) => {
 const startMockApi = async ({ routes, silent }) => {
   const requests = []
   const app = fastify()
+  app.register(fastifyUrlData)
 
   routes.forEach(({ method = 'get', path, requestBody, response = {}, status = 200 }) => {
     app.route({
@@ -74,7 +76,7 @@ export const withMockApi = async (routes, factory, silent = false) => {
   })
 
   afterAll(() => {
-    mockApi.close()
+    if (mockApi) mockApi.close()
   })
 
   factory()

tests/integration/utils/mock-api.cjs

@@ -1,5 +1,6 @@
 const { isDeepStrictEqual } = require('util')
 
+const fastifyUrlData = require('@fastify/url-data')
 const fastify = require('fastify')
 
 /**
@@ -8,7 +9,7 @@ const fastify = require('fastify')
  */
 const addRequest = (requests, request) => {
   requests.push({
-    path: request.url,
+    path: request.urlData().path,
     body: request.body,
     method: request.method,
     headers: request.headers,
@@ -23,6 +24,7 @@ const addRequest = (requests, request) => {
 const startMockApi = async ({ routes, silent }) => {
   const requests = []
   const app = fastify()
+  app.register(fastifyUrlData)
 
   routes.forEach(({ method = 'get', path, requestBody, response = {}, status = 200 }) => {
     app.route({
@@ -68,7 +70,7 @@ const withMockApi = async (routes, testHandler, silent = false) => {
     const apiUrl = `http://localhost:${mockApi.server.address().port}/api/v1`
     return await testHandler({ apiUrl, requests: mockApi.requests })
   } finally {
-    mockApi.server.close()
+    if (mockApi) mockApi.close()
   }
 }