fix(interaction): include response body if set to empty string

Author: Sebastian Tello

src/dsl/interaction.spec.ts

@@ -92,6 +92,30 @@ describe("Interaction", () => {
         expect(actual.request).to.have.keys("method", "path", "query", "headers", "body");
       });
     });
+
+    describe("request body", () => {
+      it("is included when an empty string is specified", () => {
+        const actual = new Interaction()
+          .uponReceiving("request")
+          .withRequest({
+            body: "",
+            method: HTTPMethod.GET,
+            path: "/path",
+          }).json();
+        expect(actual.request).to.have.any.keys("body");
+      });
+
+      it("is not included when explicitly set to undefined", () => {
+        const actual = new Interaction()
+          .uponReceiving("request")
+          .withRequest({
+            body: undefined,
+            method: HTTPMethod.GET,
+            path: "/path",
+          }).json();
+        expect(actual.request).not.to.have.any.keys("body");
+      });
+    });
   });
 
   describe("#willRespondWith", () => {
@@ -138,5 +162,31 @@ describe("Interaction", () => {
         expect(actual.response).to.have.keys("status", "headers", "body");
       });
     });
+
+    describe("response body", () => {
+      it("is included when an empty string is specified", () => {
+        interaction = new Interaction();
+        interaction
+          .uponReceiving("request")
+          .willRespondWith({
+            body: "",
+            status: 204,
+          });
+        const actual = interaction.json();
+        expect(actual.response).to.have.any.keys("body");
+      });
+
+      it("is not included when explicitly set to undefined", () => {
+        interaction = new Interaction();
+        interaction
+          .uponReceiving("request")
+          .willRespondWith({
+            body: undefined,
+            status: 204,
+          });
+        const actual = interaction.json();
+        expect(actual.response).not.to.have.any.keys("body");
+      });
+    });
   });
 });

src/dsl/interaction.ts

@@ -108,7 +108,7 @@ export class Interaction {
     }
 
     this.state.response = omitBy({
-      body: responseOpts.body || undefined,
+      body: responseOpts.body,
       headers: responseOpts.headers || undefined,
       status: responseOpts.status,
     }, isNil) as ResponseOptions;