RAC-254 test: 통합테스트 #91
Conversation
| } | ||
|
|
||
| @ParameterizedTest | ||
| @EnumSource(value = Status.class, names = {"WAITING", "DONE"}) |
There was a problem hiding this comment.
공부를 엄청 많이 하고 했네요! 잘못했다고 한 부분도 다시 잡혀있고요ㅎㅎ
.andExpect(jsonPath("$.data.mentoringInfos[0].nickName").value("선배"))
아래에는 대부분이 생성한 인스턴스에서 값을 가져와서 사용하는데, 몇 부분이 하드코딩을 해놓은 모습이 있어요!
그리고 응답코드나, 응답메시지도 하드 코딩이 아닌 그대로 값을 가져와서 사용하면 나중에 응답 코드 혹은 메시지가 수정되어도 그대로 테스트가 통과될 수 있지 않을까요?
되게 깔끔하게 잘 작성해서 좋네요! 추가로 예외가 발생하는 경우가 많이 존재할텐데 그에 대한 테스트도 같이 추가된다면 더욱 좋을 것 같아요!
There was a problem hiding this comment.
그리고 응답코드나, 응답메시지도 하드 코딩이 아닌 그대로 값을 가져와서 사용하면 나중에 응답 코드 혹은 메시지가 수정되어도 그대로 테스트가 통과될 수 있지 않을까요?
오 좋아요!! 감사합니다 :)
예외에 대한 부분도 추가해보도록 할게욧!
There was a problem hiding this comment.
제가 Mentoring부터 확인해서 이 부분에도 마찬가지로 이야기를 할 수 있을 것 같아요
마찬가지로 응답 코드 그리고 메시지를 하드코딩이 아닌 저장되어 있는 값을 가져다가 사용하면 수정을 해야하는 요인이 줄어들 수 있지 않을까요?
그리고 여기서는 Mockito를 사용하고 다른 테스트에서는 BDDMockito를 사용하고 있는데 이 부분은 사소하고 사소한 부분이라 상관은 없을 것 같아요ㅋㅋ
모르던 부분도 있어서 새롭게 많이 배워서 좋네요!
|
수정 내용입니다~!
|
🦝 PR 요약
✨ PR 상세 내용
🚨 주의 사항
✅ 체크 리스트