From a05aba26a8ba1c3e5572e9b7bd2ae4241f10f580 Mon Sep 17 00:00:00 2001 From: Dmitry Krasilnikov Date: Fri, 1 Sep 2017 17:00:01 +0300 Subject: [PATCH] Allure rest-assured filter doesn't requires body for a request --- .../allure/restassured/AllureRestAssured.java | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/allure-rest-assured/src/main/java/io/qameta/allure/restassured/AllureRestAssured.java b/allure-rest-assured/src/main/java/io/qameta/allure/restassured/AllureRestAssured.java index d407e93be..fb0d2626f 100644 --- a/allure-rest-assured/src/main/java/io/qameta/allure/restassured/AllureRestAssured.java +++ b/allure-rest-assured/src/main/java/io/qameta/allure/restassured/AllureRestAssured.java @@ -14,6 +14,7 @@ import java.util.HashMap; import java.util.Map; +import java.util.Objects; import static io.qameta.allure.attachment.http.HttpRequestAttachment.Builder.create; import static io.qameta.allure.attachment.http.HttpResponseAttachment.Builder.create; @@ -29,11 +30,15 @@ public Response filter(final FilterableRequestSpecification requestSpec, final FilterContext filterContext) { final Prettifier prettifier = new Prettifier(); - final HttpRequestAttachment requestAttachment = create("Request", requestSpec.getURI()) - .withBody(prettifier.getPrettifiedBodyIfPossible(requestSpec)) + final HttpRequestAttachment.Builder requestAttachmentBuilder = create("Request", requestSpec.getURI()) .withHeaders(toMapConverter(requestSpec.getHeaders())) - .withCookies(toMapConverter(requestSpec.getCookies())) - .build(); + .withCookies(toMapConverter(requestSpec.getCookies())); + + if (Objects.nonNull(requestSpec.getBody())) { + requestAttachmentBuilder.withBody(prettifier.getPrettifiedBodyIfPossible(requestSpec)); + } + + final HttpRequestAttachment requestAttachment = requestAttachmentBuilder.build(); new DefaultAttachmentProcessor().addAttachment( requestAttachment,