[Refactor] 2차 테스트 코드 스프린트

[taek2222]

📝 개요

테스트 코드 1차 스프린트를 마치고, 2차의 리팩토링 개선안을 검토해 적용해야합니다.

✅ 할 일

• [Refactor] DB Read/Write-intensive 로직 분리
• [Refactor] @BeforeAll을 활용한 TestHelper 객체 재사용 #55
• [Refactor] @tag를 이용한 느린 테스트 그룹 격리 #51
• [Refactor] @beforeeach 내 불필요한 중복 로직 제거 #50
• [Refactor] 테스트를 클래스 단위로 병렬 실행 #61
• [Refactor] Builder 패턴을 이용해 Fixture 메서드 리팩터링 #60
• [Refactor] RestAssured 헬퍼 메서드로 중복 제거

[taek2222]

RestAssured 헬퍼 메서드로 중복 제거 항목은

// 변경 전
RestAssured
     .given()
     .header(authorizationHeader)
     .contentType(ContentType.JSON)
     .body(request)
     .when()
     .post("/announcements")
     .then()
     .statusCode(HttpStatus.CREATED.value());
   
// 변경 후 
givenJson() 
     .header(authorizationHeader)
     .body(request)
     .when()
     .post("/announcements")
     .then()
     .statusCode(HttpStatus.CREATED.value());

84개의 인수 테스트 상황에서 2줄씩 제거하기 위해 이를 도입하면 168줄을 감소할 수 있지만, 유연성이 크게 감소한다는 느낌을 받았습니다. 항상 Json 형태로 전달된다는 보장도 없고 이는 추후 문제 발생 가능성이 존재해 적용하지 않기로 결정했습니다.

[taek2222]

DB Read/Write-intensive 로직 분리 고정에서 Festival 같은 변경이 없는 데이터로 시도하려 했으나, 격리성을 위해 테스트마다 다른 Festival 생성을 생성해 사용하고 있으므로, 이를 통합할 시 한 Festival 검증(ex. 핀 공지는 3개만 등록된다)이 걸린다면 서로 테스트의 영향을 줄것이라 판단했다. 그래서 Read/Wirte 분리할 부분이 없어서 이 과정은 제외했다.