test: remove test with 100 status

nodejs/undici#197
upfrontjs · Apr 12, 2024 · ca26449 · ca26449
1 parent 117d23c
commit ca26449
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/Services/ApiResponseHandler.ts b/src/Services/ApiResponseHandler.ts
@@ -55,7 +55,7 @@ export default class ApiResponseHandler implements HandlesApiResponse {
 
         if (method && ['OPTIONS', 'HEAD', 'TRACE', 'CONNECT'].includes(method)) {
             // the user might just want the headers or debug info
-            // so just return the whole response
+            // so return the whole response
             return response;
         }
 

diff --git a/tests/Calliope/Concerns/CallsApi.test.ts b/tests/Calliope/Concerns/CallsApi.test.ts
@@ -264,7 +264,7 @@ describe('CallsApi', () => {
         });
 
         it('should return undefined if the response from the handler is undefined', async () => {
-            fetchMock.mockResponseOnce(undefined, { status: 100 });
+            fetchMock.mockResponseOnce(undefined, { status: 204 });
             await expect(caller.call('GET')).resolves.toBeUndefined();
         });
 

diff --git a/tests/Services/ApiResponseHandler.test.ts b/tests/Services/ApiResponseHandler.test.ts
@@ -87,7 +87,8 @@ describe('ApiResponseHandler', () => {
 
     it.each([
         [204, 'has no content'],
-        [101, 'is an informational response'],
+        // waiting for https://github.com/nodejs/undici/issues/197
+        // [100, 'is an informational response'],
         [302, 'as a redirect response']
     ])('should return undefined if the response (%s) %s', async (status) => {
         fetchMock.mockResponseOnce(undefined, { status });