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

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

[질문 내용]
안녕하세요, 영한님.

아래에 상황 - 의문 - 추측 - 결론 순으로 질문드리겠습니다!

상황:

이번 시간에 AppConfig를 사용하도록 수정한 OrderServiceTest와 MemberServiceTest 코드에서 새롭게 다음 메서드를 만들었습니다.

@BeforeEach
public void beforeEach() {
//    ...
}

의문1:

그런데 이전에 만들었던 다른 메서드들의 경우 package-private로 접근을 제어하고 있습니다. 그래서 왜 두 경우에 다른 접근 제어자가 사용되었는지 의문이 들었습니다.

@Test
void createOrder() {
//    ...        
}

추측1:

스택오버플로우 1을 보니 과거에는 리플렉션 기술을 통해 테스트 메서드를 식별해야 했기 때문에 public으로 열어두는 것이 필요했다.

또한 이에 대한 JUnit5 탑 컨트리뷰터 Sam Brannen의 답변에서는 "less is more라는 원칙에 따라 더이상 public이 필요하지 않기 때문에 JUnit5부터는 작성하지 않아도 된다."고 하더군요. JUnit5 공식 문서에서도 "public 접근 제어자로 선언될 필요는 없다. 다만, private이면 안 된다."는 안내를 확인했습니다(아래 사진 참고).

결과적으로,

위 내용들을 근거로 "과거 관례로 인해 영한님이 public을 사용하셨겠구나."라는 추측을 했습니다.

결론:

결국 'JUnit5 이후부터는 테스트 메서드에 대해 public을 붙이지 않는 것을 권장한다(붙여도 문제는 없다).'는 것인데, (물론 개발팀 컨벤션에 따라 다르겠지만) 현업에서는 다음 두 경우 중 어떤 방식이 채택되는 경우가 더 빈번한지 여쭤봅니다!

1. 과거 관례에 이어 public을 여전히 작성한다.

2. 이제 public을 안 붙여도 되고, 이 방식을 JUnit5에서도 권장하기 때문에 더 이상 붙이지 않는다.

감사합니다.