handled no response body generation case in logger

Author: mfaisalkhatri

src/test/java/io/github/mfaisalkhatri/api/restfulecommerce/Logger.java

@@ -17,7 +17,7 @@ public Logger (final APIResponse response) {
         this.log = LogManager.getLogger (getClass ());
     }
 
-    public void logResponseDetails() {
+    public void logResponseDetails () {
         String responseBody = this.response.text ();
         if (StringUtils.isNotBlank (responseBody)) {
             this.log.info ("Logging Response Details....\n responseHeaders: {}, \nstatusCode: {}, \nresponseBody: {}",
@@ -27,31 +27,35 @@ public void logResponseDetails() {
     }
 
     private String prettyPrintJson (final String text) {
-        try {
-            final ObjectMapper objectMapper = new ObjectMapper ();
-            final Object jsonObject = objectMapper.readValue (text, Object.class);
-            return objectMapper.writerWithDefaultPrettyPrinter ()
-                .writeValueAsString (jsonObject);
-        } catch (final JsonProcessingException e) {
-            this.log.error ("Failed to pretty print JSON: {}", e.getMessage (), e);
-            return text;
+        if (StringUtils.isNotBlank (text)) {
+            try {
+                final ObjectMapper objectMapper = new ObjectMapper ();
+                final Object jsonObject = objectMapper.readValue (text, Object.class);
+                return objectMapper.writerWithDefaultPrettyPrinter ()
+                    .writeValueAsString (jsonObject);
+            } catch (final JsonProcessingException e) {
+                this.log.error ("Failed to pretty print JSON: {}", e.getMessage (), e);
+                return text;
+            }
         }
+        return "No response body generated!";
     }
 
-    //    private String prettyPrintJson (final String text) {
-    //        String prettyPrintJson = "";
-    //        if (text != null && !text.isBlank() && !text.isEmpty()) {
-    //            try {
+    //        private String prettyPrintJson (final String text) {
+    //            String prettyPrintJson = "";
+    //            if (text != null && !text.isBlank() && !text.isEmpty()) {
+    //                try {
     //
-    //                final ObjectMapper objectMapper = new ObjectMapper ();
-    //                final Object jsonObject = objectMapper.readValue (text, Object.class);
-    //                prettyPrintJson = objectMapper.writerWithDefaultPrettyPrinter().writeValueAsString(jsonObject);
-    //                return prettyPrintJson;
-    //            } catch (final JsonProcessingException e) {
-    //                this.log.error ("Error Printing Pretty Json : {}", e.getMessage ());
+    //                    final ObjectMapper objectMapper = new ObjectMapper ();
+    //                    final Object jsonObject = objectMapper.readValue (text, Object.class);
+    //                    prettyPrintJson = objectMapper.writerWithDefaultPrettyPrinter().writeValueAsString(jsonObject);
+    //                    return prettyPrintJson;
+    //                } catch (final JsonProcessingException e) {
+    //                    this.log.error ("Error Printing Pretty Json : {}", e.getMessage ());
+    //                }
     //            }
+    //            this.log.info ("No response body generated!");
+    //            return null;
     //        }
-    //        this.log.info ("No response body generated!");
-    //        return null;
-    //    }
 }
+