test: change method of concurrently requests. Added type for response…

… type
WillianAgostini · Aug 4, 2023 · eb0ede7 · eb0ede7
1 parent 1d5cc81
commit eb0ede7
Show file tree

Hide file tree

Showing 5 changed files with 26 additions and 16 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/request.ts b/src/request.ts
@@ -134,13 +134,9 @@ export class NordusRequest {
   }
 
   private getBody(nordusConfig: NordusConfigApi) {
-    if (!nordusConfig?.body) return;
-
     switch (nordusConfig.responseType) {
       case "json":
         return JSON.stringify(nordusConfig.body);
-      case "text":
-        return nordusConfig.body.toString();
       default:
         return nordusConfig.body;
     }

diff --git a/tests/concurrency.spec.ts b/tests/concurrency.spec.ts
@@ -24,11 +24,14 @@ describe("concurrency", () => {
       responseType: "json",
     });
 
-    const instanceGet = await instance.get("todos/1");
-    const instancePost = await instance.post("todos/1", {});
-    const instancePut = await instance.put("todos/1", {});
-    const instancePatch = await instance.patch("todos/1", {});
-    const instanceDel = await instance.del("todos/1");
+    const [instanceGet, instancePost, instancePut, instancePatch, instanceDel] =
+      await Promise.all([
+        instance.get("todos/1"),
+        instance.post("todos/1", {}),
+        instance.put("todos/1", {}),
+        instance.patch("todos/1", {}),
+        instance.del("todos/1"),
+      ]);
 
     expect(instanceGet.data).toEqual({ method: "get" });
     expect(instancePost.data).toEqual({ method: "post" });

diff --git a/tests/get.spec.ts b/tests/get.spec.ts
@@ -67,7 +67,7 @@ describe("interceptors", () => {
         test: "test",
       })
     );
-    const response = await get("http://localhost:5000");
+    const response = await get<{ test: string }>("http://localhost:5000");
 
     expect(response.data).toEqual({
       test: "test",

diff --git a/tests/post.spec.ts b/tests/post.spec.ts
@@ -12,11 +12,12 @@ describe("interceptors", () => {
         test: "test",
       })
     );
-    const response = await post("http://localhost:5000", {
-      body: {
+    const response = await post<{ test: string }, { test: any }>(
+      "http://localhost:5000",
+      {
         test: "test",
-      },
-    });
+      }
+    );
 
     expect(response.data).toEqual({
       test: "test",
@@ -54,4 +55,14 @@ describe("interceptors", () => {
       test: "test",
     });
   });
+
+  it("success post with null body", async () => {
+    fetchMock.mockResponseOnce("plan text");
+
+    const response = await post("http://localhost:5000", null, {
+      responseType: "text",
+    });
+
+    expect(response.data).toEqual("plan text");
+  });
 });