학습 테스트 실습 리뷰 부탁드립니다 #1843
학습 테스트 실습 리뷰 부탁드립니다 #1843
Conversation
| @DisplayName("문자열 1,2 나누기") | ||
| public void stringTest01() throws Exception { | ||
| String[] result = "1,2".split(","); | ||
| // assertThat(result).contains("1", "2"); |
| public void stringTest03() throws Exception { | ||
| String data = "(1,2)"; | ||
| String result = data.substring(1, data.length()-1); | ||
| assertThat(result).isEqualTo("1,2"); |
There was a problem hiding this comment.
잘 작성하셨네요 💯
추가로 아래처럼 given, when, then 순으로 작성하시면
좀 더 가독성 및 Test 코드를 보고 의도를 파악하기 쉽습니다.😄
// given
String data = "(1,2)";
// when
String result = data.substring(1, data.length()-1);
// then
assertThat(result).isEqualTo("1,2");
There was a problem hiding this comment.
given, when, then 템플릿으로 진행하겠습니다
| assertThatThrownBy(() -> { | ||
| data.charAt(3); | ||
| }).isInstanceOf(IndexOutOfBoundsException.class) | ||
| .hasMessageContaining("String index out of range: 3"); |
There was a problem hiding this comment.
예외테스트 경우엔 실패할 에러메시지까지 일치시켜야 되는건가요?
아니면 예외처리안에서 에러가 발생하면 통과인지 궁금합니다.
| } | ||
|
|
||
| @Test | ||
| @DisplayName("Set size 확인") |
There was a problem hiding this comment.
@DisplayName 으로 어떠한 테스트 인지 파악할 수 있어 좋네요 💯
메소드 이름이나 @DisplayName이 어느 정도까지 구체적이어야 하는지에 대해서 정답은 없다고 생각합니다.
다만, 이름만으로도 언제 이 메서드를 호출해야 하는지 또 어떤 테스트를 하는 것인지 의미를 파악할 수 있도록 작성하시면 다른 사람이 볼때도 쉽게 의도를 파악할 수 있을거 같네요. 아래 자료도 참고해 보시면 좋을거 같아요. 👍
https://woowacourse.github.io/javable/2020-04-26/Method-Naming
There was a problem hiding this comment.
|
|
||
| @Test | ||
| @DisplayName("Set size 확인") | ||
| public void setTest01() throws Exception { |
There was a problem hiding this comment.
물론 DisplayName으로 어떤 테스트인지 파악할 수 있지만 함수명도 어떤 의미의 테스트인지 담을 수록 좋습니다.
다음 단계부터는 함수명도 의미있는 이름으로 바꿔보도록 해요!
There was a problem hiding this comment.
함수명... 변수명...
이름짓기가 너무 힘드네요😂
https://www.curioustore.com/#!/
사이트의 도움을 받아야겠습니다.
| public void setTest03(int inputNumber, boolean isFree) throws Exception { | ||
| assertThat(numbers.contains(inputNumber)).isEqualTo(isFree); | ||
| } | ||
| } |
There was a problem hiding this comment.
커밋 로그를 의미있는 단위로 나눠주시면 좋습니다.
또한 메시지에 로그 타입도 같이 작성해주시면 좋을거 같아요 👍
feat (feature)
fix (bug fix)
docs (documentation)
style (formatting, missing semi colons, …)
refactor
test (when adding missing tests)
chore (maintain)
아래 링크 참고하셔서 미션 수행시 기능 단위로 Commit Log 남길때 참고해보시면 좋을거 같아요 😄
https://gist.github.com/stephenparish/9941e89d80e2bc58a153
There was a problem hiding this comment.
커밋 컨벤션 처음 알았네요!
커밋로그에 맞게 남기곘습니다👍
안녕하세요 박현철입니다
테스트코드 미숙해서 제대로 했는지 모르겠습니다
수정사항 부탁드립니다
감사합니다