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

Author: Sebastian Tello

src/dsl/interaction.spec.ts

@@ -87,6 +87,26 @@ 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().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().withRequest({
+          body: undefined,
+          method: HTTPMethod.GET,
+          path: "/path",
+        }).json();
+        expect(actual.request).not.to.have.any.keys("body");
+      });
+    });
   });
 
   describe("#willRespondWith", () => {
@@ -131,5 +151,27 @@ 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.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.willRespondWith({
+          body: undefined,
+          status: 204,
+        });
+        const actual = interaction.json();
+        expect(actual.response).not.to.have.any.keys("body");
+      });
+    });
   });
 });

src/dsl/interaction.ts

@@ -106,7 +106,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;