test cases added

Author: prasath116

reactive-micro-service/student-service/src/main/java/com/prs/services/student/service/StudentServiceImpl.java

@@ -7,7 +7,7 @@
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
-import com.prs.services.exceptionHandler.DataException;
+import com.prs.services.exceptionHandler.NotFoundException;
 import com.prs.services.student.client.CollegeReactiveClient;
 import com.prs.services.student.client.DepartmentReactiveClient;
 import com.prs.services.student.entity.StudentEntity;
@@ -42,7 +42,7 @@ public Mono<Student> update(Long id, StudentEntity entity) {
 return findById(id).flatMap(s-> {
 entity.setId(id);
 return save(entity);
-}).switchIfEmpty(Mono.error(new DataException("Student not fount for id : "+id)));
+}).switchIfEmpty(Mono.error(new NotFoundException("Student not fount for id : "+id)));
 }
 
 @Override

reactive-micro-service/student-service/src/test/java/com/prs/services/student/controller/StudentControllerIntegrationTest.java

@@ -1,10 +1,14 @@
 package com.prs.services.student.controller;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNotEquals;
 
 import java.util.List;
 
+import org.junit.jupiter.api.MethodOrderer.OrderAnnotation;
+import org.junit.jupiter.api.Order;
 import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.TestMethodOrder;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.boot.test.autoconfigure.web.reactive.AutoConfigureWebTestClient;
 import org.springframework.boot.test.context.SpringBootTest;
@@ -18,6 +22,7 @@
 @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
 @ActiveProfiles("test")
 @AutoConfigureWebTestClient
+@TestMethodOrder(OrderAnnotation.class)
 public class StudentControllerIntegrationTest {
 
  @Autowired
@@ -26,6 +31,8 @@ public class StudentControllerIntegrationTest {
  @Autowired
  private StudentRepository studentRepository;
 
+ static Long testId;
+ 
 // @BeforeEach
  void setUp() {
  var students = List.of(new StudentEntity(null,"Prasath",31,1l,1l),
@@ -39,6 +46,7 @@ void setUp() {
  }
 
  @Test
+ @Order(1)
  void getAllStudents() {
  webTestClient
  .get()
@@ -56,32 +64,56 @@ void getAllStudents() {
  }
 
  @Test
+ @Order(2)
+ void addStudent() {
+ var student = new StudentEntity(null,"Prasath",31,1l,1l);
+ webTestClient
+ .post()
+ .uri("/add")
+ .bodyValue(student)
+ .exchange()
+ .expectStatus()
+ .isCreated()
+ .expectBody(Student.class)
+ .consumeWith(result -> {
+ var s = result.getResponseBody();
+ assert s != null;
+ assertNotEquals(null, s);
+ assertEquals("Prasath", s.getName());
+ testId = s.getId();
+ System.out.println("testId received : "+testId);
+ assertNotEquals(null, testId);
+	});
+ }
+ 
+ @Test
+ @Order(3)
 void getStudentById() {
-Long id = 1l;
+ System.out.println("testId in getStudentById: "+testId);
 webTestClient
 .get()
-.uri("/get-by/{id}", id)
+.uri("/get-by/{id}", testId)
 .exchange()
 .expectStatus()
 .is2xxSuccessful()
 .expectBody(Student.class)
 .consumeWith(result -> {
 var student = result.getResponseBody();
 assert student != null;
-assertEquals("aa", student.getName());
+assertEquals("Prasath", student.getName());
 });
 //.expectBody() .jsonPath("$.name").isEqualTo("Prasath");
 
 }
 
  @Test
+ @Order(4)
  void updateStudent() {
- var id = 2l;
- var updatedStudent = new StudentEntity(2l,"Nishanthi M",30,2l,2l);
+ var updatedStudent = new StudentEntity(testId,"Nishanthi M",30,2l,2l);
 
  webTestClient
  .put()
- .uri("/update/{id}", id)
+ .uri("/update/{id}", testId)
  .bodyValue(updatedStudent)
  .exchange()
  .expectStatus()
@@ -95,9 +127,10 @@ void updateStudent() {
  }
 
  @Test
+ @Order(5)
  void updateStudent_notFound() {
  var id = 5l;
- var updatedStudent = new StudentEntity(2l,"Nishanthi M",30,2l,2l);
+ var updatedStudent = new StudentEntity(testId,"Nishanthi M",30,2l,2l);
 
  webTestClient
  .put()
@@ -109,12 +142,11 @@ void updateStudent_notFound() {
  }
 
  @Test
+ @Order(6)
  void deleteStudentById() {
- var id = 2l;
-
  webTestClient
  .delete()
- .uri("/delete/{id}", id)
+ .uri("/delete/{id}", testId)
  .exchange()
  .expectStatus()
  .isNoContent();

reactive-micro-service/student-service/src/test/java/com/prs/services/student/cucumberConfig/CucumberSpringConfiguration.java

@@ -2,14 +2,10 @@
 package com.prs.services.student.cucumberConfig;
 
 import org.springframework.boot.test.context.SpringBootTest;
-import org.springframework.test.context.ContextConfiguration;
-
-import com.prs.services.StudentApplication;
 
 import io.cucumber.spring.CucumberContextConfiguration;
 
 @CucumberContextConfiguration
 @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
-//@SpringBootTest(classes = StudentApplication.class)
 public class CucumberSpringConfiguration {
 }

reactive-micro-service/student-service/src/test/java/com/prs/services/student/cucumberConfig/StudentCucumberSteps.java

@@ -2,8 +2,6 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.test.web.reactive.server.WebTestClient;
 
@@ -15,8 +13,6 @@
 import io.cucumber.java.en.When;
 
 public class StudentCucumberSteps {
-
-private final Logger log = LoggerFactory.getLogger(StudentCucumberSteps.class);
 StudentEntity studentTest = new StudentEntity(null, "testCucumber", 25, 2l, 2l);
 @Autowired
 private WebTestClient webTestClient;