LandvibeDev / web-chat-backend

Web Chat Application 🛩
1 stars 3 forks source link

메시지 조회 기능에 대한 tc 추가 #27

Closed raccoonback closed 4 years ago

raccoonback commented 4 years ago

⚽ 기능 설명

🏷 연관 이슈

26

🧢 체크 리스트

🍄 ps

raccoonback commented 4 years ago

다른 tc 에서도 assertj 를 사용하고 있었으니까 이것도 assertj 로 통일하면 좋을 것 같아 (개인적으로는 junit 에서 제공하는 테스트 메서드보다 가독성이 더 좋은 것 같아)

// 메서드 체이닝 가능
// 방법1
assertThat(messages)
  .hasSize(3)
  .containsExactlyElementsOf(givenMessages);

// 방법2
assertThat(messages).hasSize(3);
assertThat(messages).containsExactlyElementsOf(givenMessages);

ps. 근데 assertIterableEquals(expected, actual) 처럼 containsExactlyElementsOf() 도 deeply euqal 여부를 검사하는지는 좀더 찾아봐야 할듯. 일단 size 검증 만이라도 assertj 사용해도 상관없음

ref. https://pjh3749.tistory.com/241 ref. https://joel-costigliola.github.io/assertj/

usingRecursiveComparison() 사용하면 오브젝트 필드를 재귀적으로 비교하는 거 같아

        assertThat(messages)
            .hasSameSizeAs(givenMessages)
            .usingRecursiveComparison()
            .isEqualTo(givenMessages);

ref. https://joel-costigliola.github.io/assertj/core-8/api/org/assertj/core/api/RecursiveComparisonAssert.html