test(#zimic): extend path parameter tests (#360)

### Tests - [#zimic] Added more test cases for interceptor workers, considering paths starting and not starting with `/`.
zimicjs · Aug 25, 2024 · a29f656 · a29f656
1 parent f24d8e4
commit a29f656
Showing 1 changed file with 55 additions and 7 deletions.
diff --git a/packages/zimic/src/interceptor/http/interceptorWorker/__tests__/shared/methods.ts b/packages/zimic/src/interceptor/http/interceptorWorker/__tests__/shared/methods.ts
@@ -131,6 +131,38 @@ export function declareMethodHttpInterceptorWorkerTests(options: SharedHttpInter
       const param3 = '3';
 
       it.each([
+        {
+          use: ':param',
+          fetch: `${param1}`,
+          params: { param: param1 },
+        },
+        {
+          use: ':param',
+          fetch: `${param2}`,
+          params: { param: param2 },
+        },
+        {
+          use: `${param1}`,
+          fetch: `${param1}`,
+          params: {},
+        },
+
+        {
+          use: ':param',
+          fetch: `/${param1}`,
+          params: { param: param1 },
+        },
+        {
+          use: ':param',
+          fetch: `/${param2}`,
+          params: { param: param2 },
+        },
+        {
+          use: `${param1}`,
+          fetch: `/${param1}`,
+          params: {},
+        },
+
         {
           use: '/:param',
           fetch: `/${param1}`,
@@ -147,6 +179,22 @@ export function declareMethodHttpInterceptorWorkerTests(options: SharedHttpInter
           params: {},
         },
 
+        {
+          use: '/:param',
+          fetch: `${param1}`,
+          params: { param: param1 },
+        },
+        {
+          use: '/:param',
+          fetch: `${param2}`,
+          params: { param: param2 },
+        },
+        {
+          use: `/${param1}`,
+          fetch: `${param1}`,
+          params: {},
+        },
+
         {
           use: '/other/path/:param',
           fetch: `/other/path/${param1}`,
@@ -242,16 +290,16 @@ export function declareMethodHttpInterceptorWorkerTests(options: SharedHttpInter
           fetch: `/${param1}/path/${param2}/${param3}/path`,
           params: {},
         },
-      ])(`should intercept ${method} requests with matching dynamic paths (use $use; fetch $fetch)`, async (paths) => {
-        const url = joinURL(baseURL, paths.use);
+      ])(`should intercept ${method} requests with matching dynamic paths (use $use; fetch $fetch)`, async (path) => {
+        const url = joinURL(baseURL, path.use);
 
         await usingHttpInterceptorWorker(workerOptions, async (worker) => {
           const interceptor = createDefaultHttpInterceptor();
           await promiseIfRemote(worker.use(interceptor.client(), method, url, spiedRequestHandler), worker);
 
           expect(spiedRequestHandler).not.toHaveBeenCalled();
 
-          const urlExpectedToSucceed = joinURL(baseURL, paths.fetch);
+          const urlExpectedToSucceed = joinURL(baseURL, path.fetch);
           const response = await fetch(urlExpectedToSucceed, { method });
 
           expect(spiedRequestHandler).toHaveBeenCalledTimes(numberOfRequestsIncludingPreflight);
@@ -262,7 +310,7 @@ export function declareMethodHttpInterceptorWorkerTests(options: SharedHttpInter
 
           const urlRegex = createRegexFromURL(url);
           const parsedRequest = await HttpInterceptorWorker.parseRawRequest(handlerContext.request, { urlRegex });
-          expect(parsedRequest.pathParams).toEqual(paths.params);
+          expect(parsedRequest.pathParams).toEqual(path.params);
 
           expect(response.status).toBe(200);
           await expectMatchedBodyIfNotHead(response);
@@ -418,16 +466,16 @@ export function declareMethodHttpInterceptorWorkerTests(options: SharedHttpInter
         },
       ])(
         `should not intercept ${method} requests with non-matching dynamic paths (use $use; fetch $fetch)`,
-        async (paths) => {
-          const url = joinURL(baseURL, paths.use);
+        async (path) => {
+          const url = joinURL(baseURL, path.use);
 
           await usingHttpInterceptorWorker(workerOptions, async (worker) => {
             const interceptor = createDefaultHttpInterceptor();
             await promiseIfRemote(worker.use(interceptor.client(), method, url, spiedRequestHandler), worker);
 
             expect(spiedRequestHandler).not.toHaveBeenCalled();
 
-            const urlExpectedToFail = joinURL(baseURL, paths.fetch);
+            const urlExpectedToFail = joinURL(baseURL, path.fetch);
 
             const fetchPromise = fetchWithTimeout(urlExpectedToFail, { method, timeout: 200 });
             await expectFetchErrorOrPreflightResponse(fetchPromise, {