[ 5주차 ] 부족한 테스트 코드 추가 및 Jacoco 적용, 보고서 작성#106
[ 5주차 ] 부족한 테스트 코드 추가 및 Jacoco 적용, 보고서 작성#106futuremaker019 wants to merge 17 commits intohanghae-skillup:futuremaker019from
Conversation
futuremaker019
changed the title
| @DisplayName("스케줄과 여러 좌석을 선택하여, 예약을 등록한다.") | ||
| @Test | ||
| void given_when_then() throws Exception { | ||
| Long scheduleId = 1L; |
There was a problem hiding this comment.
DisplayName 을 사용하는 대신 함수 이름을 한글로 작성하는게 더 좋을 것 같습니다 :)
현재의 함수명은 수정이 필요해보이네요 !
| Reservation.isAlreadyReserved(reservations, seats); | ||
| }); | ||
| assertEquals(ErrorCode.UNABLE_TO_RESERVE, thrown.getErrorCode()); | ||
| assertTrue(thrown.getMessage().contains("reservation contains already reserved seats:")); |
There was a problem hiding this comment.
도메인에 대한 단위 테스트도 잘 작성해주셨네요 👍🏿
| assertThat(reservations) | ||
| .isNotEmpty() | ||
| .hasSize(3) | ||
| .extracting("scheduleId", "seatId") |
There was a problem hiding this comment.
seatIds 의 size 를 가져오면 하드코딩을 없앨 수 있을 것 같습니다.
|
@futuremaker019 구현님 안녕하세요 ! 마지막까지 고생많으셨습니다. 전반적으로 다 깔끔하게 테스트를 진행해주셨습니다. 리뷰 포인트 및 추가 질문에 대한 답변
지금 하신 것 처럼 캐싱해서 사용하는 방식으로 처리합니다. 실무에서도 body 를 사전에 읽어서 사용하려면 캐싱해두어야 합니다.
이 정도가 중요하다고 생각됩니다 !
cloud engineer 라면 위 기술들에 대해서 깊이 있게 알아야 합니다. 반면 service engineer 라면 위 기술이 무엇인지 아는 정도면 충분하다고 생각됩니다. 엔지니어 직군마다 본질적으로 요구되는 능력이 다르다고 생각됩니다. 또한 JD 에 위 기술이 명시되어있으면 면접을 위해서 개념적으로 더 많이 찾아봐야겠죠. 하지만 일반적으로 service engineer 직군으로 가는 경우 위 기술에 대해서 깊이감 있는 질문을 하진 않습니다. 5주간 수고 많으셨습니다 !! 💯 새해복 많이 받으세요 ~ |
2 participants
제목(title)
[ 5주차 ] 부족한 테스트 코드 추가 및 Jacoco 적용, 보고서 작성
작업 내용
발생했던 문제와 해결 과정을 남겨 주세요.
request body missing에러를 수정하느라 고생했습니다. interceptor 보다 먼저 실행되는 filter에서 RequestWrapper를 사용하여 body를 미리 캐시하도록 수정하여 해결했습니다.이번 주차에서 고민되었던 지점이나, 어려웠던 점을 알려 주세요.
리뷰 포인트
README 링크
기타 질문
코드 리뷰 너무 감사했습니다. 5주동안 공부하며 많은 인사이트 얻어갑니다. 새해복 많이 받으세요! 👏👏👏👏