updated dependency versions and refactored logger class

Author: mfaisalkhatri

pom.xml

@@ -12,25 +12,25 @@
 
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-        <playwright-version>1.47.0</playwright-version>
+        <playwright-version>1.49.0</playwright-version>
         <testng-version>7.10.2</testng-version>
         <hamcrest-all-version>1.3</hamcrest-all-version>
         <json-simple-version>1.1.1</json-simple-version>
-        <lombok-version>1.18.34</lombok-version>
-        <datafaker-version>2.4.0</datafaker-version>
+        <lombok-version>1.18.36</lombok-version>
+        <datafaker-version>2.4.2</datafaker-version>
         <gson-version>2.11.0</gson-version>
         <org-json-verion>20240303</org-json-verion>
-        <jackson-databind-version>2.18.0</jackson-databind-version>
+        <jackson-databind-version>2.18.2</jackson-databind-version>
         <log4j-core-version>2.24.1</log4j-core-version>
         <log4j-api-version>2.24.1</log4j-api-version>
+        <commons-lang3-version>3.17.0</commons-lang3-version>
         <maven-compiler-plugin-version>3.13.0</maven-compiler-plugin-version>
         <maven-surefire-plugin-version>3.3.1</maven-surefire-plugin-version>
         <java-release-version>17</java-release-version>
-        <maven-enforcer-version>3.5.0</maven-enforcer-version>
+        <maven-enforcer-version>3.5.2</maven-enforcer-version>
         <suite-xml>test-suite/testng.xml</suite-xml>
     </properties>
     <dependencies>
-        <!-- https://mvnrepository.com/artifact/com.microsoft.playwright/playwright -->
         <dependency>
             <groupId>com.microsoft.playwright</groupId>
             <artifactId>playwright</artifactId>
@@ -46,7 +46,6 @@
             <artifactId>json</artifactId>
             <version>${org-json-verion}</version>
         </dependency>
-        <!-- https://mvnrepository.com/artifact/org.projectlombok/lombok -->
         <dependency>
             <groupId>org.projectlombok</groupId>
             <artifactId>lombok</artifactId>
@@ -78,6 +77,12 @@
                 <artifactId>log4j-core</artifactId>
                 <version>${log4j-core-version}</version>
             </dependency>
+        <dependency>
+            <groupId>org.apache.commons</groupId>
+            <artifactId>commons-lang3</artifactId>
+            <version>${commons-lang3-version}</version>
+        </dependency>
+
     </dependencies>
     <build>
         <pluginManagement>
@@ -133,4 +138,4 @@
             </plugins>
         </pluginManagement>
     </build>
-</project>
+</project>

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

@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.core.JsonProcessingException;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.microsoft.playwright.APIResponse;
+import org.apache.commons.lang3.StringUtils;
 import org.apache.logging.log4j.LogManager;
 
 
@@ -17,32 +18,40 @@ public Logger (final APIResponse response) {
     }
 
     public void logResponseDetails() {
-        this.log.info ("Logging Response Details.....");
-
-        this.log.info ("Response Headers: \n{}", this.response.headers ());
-        this.log.info ("Status Code: {}", this.response.status ());
-        if (this.response.text () != null && !this.response.text ()
-            .isEmpty () && !this.response.text ()
-            .isBlank ()) {
-            this.log.info ("Response Body: \n{}", prettyPrintJson (this.response.text ()));
+        String responseBody = this.response.text ();
+        if (StringUtils.isNotBlank (responseBody)) {
+            this.log.info ("Logging Response Details....\n responseHeaders: {}, \nstatusCode: {}, \nresponseBody: {}",
+                this.response.headers (), this.response.status (), prettyPrintJson (this.response.text ()));
         }
         this.log.info ("End of Logs!");
     }
 
     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 ());
-            }
+        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;
         }
-        this.log.info ("No response body generated!");
-        return null;
     }
+
+    //    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 ());
+    //            }
+    //        }
+    //        this.log.info ("No response body generated!");
+    //        return null;
+    //    }
 }