From 668bab3a61d0be395d7ed1c4dc3f4149688cbf34 Mon Sep 17 00:00:00 2001 From: dadgam3er Date: Fri, 18 Oct 2024 11:36:53 -0400 Subject: [PATCH] fix: adding Logs --- pom.xml | 7 ++ src/main/java/zacksolutions/App.java | 13 --- .../java/api/endpoints/UserEndpoints.java | 2 +- src/test/java/api/test/UserTest.java | 89 ++++++++++++++----- src/test/java/api/utilities/XLUtility.java | 4 + src/test/resources/log4j2.xml | 16 ++++ testng.xml | 3 - 7 files changed, 93 insertions(+), 41 deletions(-) delete mode 100644 src/main/java/zacksolutions/App.java create mode 100644 src/test/java/api/utilities/XLUtility.java create mode 100644 src/test/resources/log4j2.xml diff --git a/pom.xml b/pom.xml index 8fc4f8d..a48a98b 100644 --- a/pom.xml +++ b/pom.xml @@ -75,6 +75,13 @@ log4j-core 2.24.0 + + + org.apache.logging.log4j + log4j-api + 3.0.0-beta2 + + io.cucumber diff --git a/src/main/java/zacksolutions/App.java b/src/main/java/zacksolutions/App.java deleted file mode 100644 index a0566a9..0000000 --- a/src/main/java/zacksolutions/App.java +++ /dev/null @@ -1,13 +0,0 @@ -package zacksolutions; - -/** - * Hello world! - * - */ -public class App -{ - public static void main( String[] args ) - { - System.out.println( "Hello World!" ); - } -} diff --git a/src/test/java/api/endpoints/UserEndpoints.java b/src/test/java/api/endpoints/UserEndpoints.java index 8d5d999..e333460 100644 --- a/src/test/java/api/endpoints/UserEndpoints.java +++ b/src/test/java/api/endpoints/UserEndpoints.java @@ -29,7 +29,7 @@ public class UserEndpoints { Response response = given() .contentType(ContentType.JSON) .accept(ContentType.JSON) - .pathParam("usermame", userName) + .pathParam("username", userName) .body(payload) .when() .put(Routes.UPDATE_URL); diff --git a/src/test/java/api/test/UserTest.java b/src/test/java/api/test/UserTest.java index 6daa259..17a2c7c 100644 --- a/src/test/java/api/test/UserTest.java +++ b/src/test/java/api/test/UserTest.java @@ -3,37 +3,78 @@ package api.test; import api.endpoints.UserEndpoints; import api.payload.User; import io.restassured.response.Response; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.testng.Assert; import org.testng.annotations.BeforeClass; import org.testng.annotations.Test; public class UserTest { - User userPayload; - @BeforeClass - public void setupData(){ - userPayload = new User(); - userPayload.setId(10); - userPayload.setUsername("snowman"); - userPayload.setFirstname("Desmond"); - userPayload.setLastname("Hume"); - userPayload.setEmail("desmondh@gamil.com"); - userPayload.setPassword("4815162342"); - userPayload.setPhone("79460893"); - } + User userPayload; + public Logger logger; - @Test(priority = 1) - public void testPostUser(){ - Response response = UserEndpoints.createUser(userPayload); - response.then().log().all(); + @BeforeClass + public void setup() { + userPayload = new User(); + userPayload.setId(10); + userPayload.setUsername("snowman"); + userPayload.setFirstname("Desmond"); + userPayload.setLastname("Hume"); + userPayload.setEmail("desmondh@gamil.com"); + userPayload.setPassword("4815162342"); + userPayload.setPhone("79460893"); + //Logs + logger = LogManager.getLogger(UserTest.class.getName()); + } - Assert.assertEquals(response.getStatusCode(), 200); - } + @Test(priority = 1) + public void testPostUser() { + logger.info("Creating User"); + Response response = UserEndpoints.createUser(userPayload); + response.then().log().all(); - @Test(priority = 2) - public void testGetUser(){ - Response response = UserEndpoints.readUser(this.userPayload.getUsername()); - response.then() - .statusCode(200).log().all(); - } + Assert.assertEquals(response.getStatusCode(), 200); + logger.info("<><><><><>User Created<><><><><><"); + } + + @Test(priority = 2) + public void testGetUser() { + logger.info("<><><><>Display Info<><><><><>"); + Response response = UserEndpoints.readUser(this.userPayload.getUsername()); + response.then() + .statusCode(200).log().all(); + } + + @Test(priority = 3) + public void testUpdateUser() { + + logger.info("<><><><>Updating User creds<><><><><>"); + // Update data using payload + userPayload.setId(15); + userPayload.setFirstname("Ben"); + userPayload.setLastname("Linus"); + userPayload.setEmail("benlinus@gamil.com"); + + Response response = UserEndpoints.updateUser(this.userPayload.getUsername(), userPayload); + response.then().log().body(); + + Assert.assertEquals(response.getStatusCode(), 200); + + logger.info("<><><><>Creds Updated<><><><><>"); + + // check if the data was changed + + logger.info("<><><><>New User Update displayed<><><><><>"); + Response updatedRespose = UserEndpoints.readUser(this.userPayload.getUsername()); + updatedRespose.then().log().all(); + Assert.assertEquals(updatedRespose.getStatusCode(), 200); + } + + @Test(priority = 4) + public void testDeleteUser(){ + + Response response = UserEndpoints.deleteUser(this.userPayload.getUsername()); + response.then().log().all(); + } } diff --git a/src/test/java/api/utilities/XLUtility.java b/src/test/java/api/utilities/XLUtility.java new file mode 100644 index 0000000..da75783 --- /dev/null +++ b/src/test/java/api/utilities/XLUtility.java @@ -0,0 +1,4 @@ +package api.utilities; + +public class XLUtility { +} diff --git a/src/test/resources/log4j2.xml b/src/test/resources/log4j2.xml new file mode 100644 index 0000000..69f2f32 --- /dev/null +++ b/src/test/resources/log4j2.xml @@ -0,0 +1,16 @@ + + + + ./logs + + + + + + + + + + + + \ No newline at end of file diff --git a/testng.xml b/testng.xml index 6d57773..20594a1 100644 --- a/testng.xml +++ b/testng.xml @@ -4,9 +4,6 @@ - \ - -