인프런 커뮤니티 질문&답변

stopzero님의 프로필 이미지
stopzero

작성한 질문수

스프링 입문 - 코드로 배우는 스프링 부트, 웹 MVC, DB 접근 기술

회원 서비스 테스트

try, catch 문이 애매한 이유가 있나요?

해결된 질문

작성

·

545

·

수정됨

0

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예)

[질문 내용]
@Test

public void 중복_회원_예약(){

//given

Member member1 = new Member();

member1.setName("spring");

Member member2 = new Member();

member2.setName("spring");

//when

try{

memberService.join(member2);

fail();

}catch (IllegalStateException e){

assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다. ");

}

//then

}

try, catch 문을 사용하기에 애매한 이유가 정확하게 뭔지 궁금합니다. 사용하면 안되는 이유가 있나요?

답변 1

1

안녕하세요. stopzero님, 공식 서포터즈 y2gcoder입니다.

개인적으로는 2가지 이유로 인해 테스트 코드에서 try catch 문을 사용하지 않는 것 같습니다.

1) 테스트 코드의 의도가 불명확해질 수 있습니다. 예를 들어 위의 테스트 코드는 같은 이름을 가진 회원이 이미 존재할 때 회원가입을 시도하면 회원서비스에서 예외를 반환하는 것을 검증하기 위한 의도를 가지고 있습니다. 그리고 try-catch 문은 보통 try 문의 예외를 잡아 정상처리해주기 위해서 사용합니다. 그러면 위의 코드에서 해당 예외를 잡아 처리해주는 것은 테스트 코드의 의도와는 다소 맞지 않아보일 수 있습니다. 예외가 정상적으로 터지는 것을 보고 싶은데, try-catch 문으로 예외를 잡아서 처리해주는 것은 협업하는 다른 개발자로 하여금 혼란을 줄 수 있다 생각합니다 :)

2) 두번째로는 try-catch문을 사용하는 것보다 테스트 프레임워크의 기능을 사용하는 것이 훨씬 가독성이 좋습니다. JUnit에 내장되어있는 assertThrows() 나 Assertions 모듈의 assertThatThrownBy()와 같은 메서드를 활용하면 이 테스트가 예외를 반환하고 이를 확인하기 위한 코드라는 것을 훨씬 더 짧은 코드로 가독성 있게 보여줄 수 있는 것 같습니다.

 

감사합니다.

stopzero님의 프로필 이미지
stopzero

작성한 질문수

질문하기