[Spring] 컨트롤러 테스트 작성하실 때 해당 컨트롤러 요청 처리 과정에서 발생하는 예외 케이스를 전부 작성하시나요?

Question

컨트롤러 테스트 작성하실 때 해당 컨트롤러의 핸들러 메서드에서 발생하는 예외 케이스를 전부 작성하시나요?현재 컨트롤러 테스트에서 성공 케이스만 테스트하고 Spirng REST Docs로 성공 케이스만 포함된 API 문서를 생성하고 있습니다.실패 상황도 컨트롤러 테스트에 추가해서 API 문서에 요청 처리 실패 상황도 포함 시키려는데 보통 어떻게 하나 궁금해서 여쭤봅니다.컨트롤러 클래스 테스트에서 예외 케이스들을 추가하면, 서비스 클래스 단위테스트에서도 예외 상황이 테스트 되니까 중복으로 테스트되는 것 같은데 이것도 보통 이렇게 하나요?아래는 우아한테크코스 플젝 API 명세인데, 실패 케이스도 전부 포함해서 컨트롤러 테스트 작성했던데 이렇게 하는게 일반적일까요?https://woowacourse-teams.github.io/2022-sokdak/backend/sokdak/src/main/resources/static/index.html#_%EB%A9%94%EC%9D%B8%ED%8E%98%EC%9D%B4%EC%A7%80

adduci · Answer

있으면 좋겠죠 근데 팀 상황에 따라 실패는 그냥 공통 에러 테이블 그려서 보여주는 케이스도 있습니다

나니야 · Answer

힘빠지는 답일수도 있지만 하기 나름입니다. 정하기 나름이예요.얼마나 꼼꼼하게 할건지 개발자 자유입니다.클린코드였던가 TDD관련 책이었던가 기억은 안나지만, 너무 타이트하게 테스트코드를 작성하지 말라고 하긴하더라구요. 물론 실패없는 프로그램을 만드는곳은 해야겠지만....

kiel · Answer

실패 상황도 테스트에 추가를 하는게 맞죠. 인증 실패했을때 401 을 반환할지 400 (로직의 오류로 취급해서) 할지 정해서 관리해야 하니까요. 그걸 API 문서에 넣는지는 부서 마음입니다. 인증 실패는 무조건 401에 공통 메시지로 반환한다 라고 정했다면 개별적 실패 케이스를 문서화 할 필요는 없고요.컨트롤러 테스트에서 실패 케이스를 추가한거와 서비스 클래스 단위 테스트는 별개입니다. 그건 서비스 클래스에서 오류를 반환하면 401로 보낸다는 케이스이지, 그게 서비스 클래스 테스트를 한건 아니니까요우테코 예시는 코스 참여자들이 공부하기 위한 예시이니 가능한 촘촘하게 만들어 놓은거지, 이대로 따라 만들어야 한다는건 아닙니다.

장독깨기 · Answer

테스트를 자동화 하든 수동으로 하든 당연히 예외 케이스를 시험해야 합니다.성공 케이스만 작성하는 건 시험이 아닙니다.그리고, 명세서에는 해당 API 에서 발생할 수 있는 예외를 기록하면 되는데요,굳이 저런 식으로 까지 할 필요는 없어 보이긴 합니다.

[Spring] 컨트롤러 테스트 작성하실 때 해당 컨트롤러 요청 처리 과정에서 발생하는 예외 케이스를 전부 작성하시나요?

4개의 답변

Q&A 목록