Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#16] 동네 인증 테스트 코드 리펙토링 #19

Merged
merged 1 commit into from
Sep 22, 2024
Merged

Conversation

Srltas
Copy link
Collaborator

@Srltas Srltas commented Sep 21, 2024

📌 Summary

동네 인증 테스트 코드 리펙토링을 통해 Hamcrest 메소드를 적용하고, BDD(Behavior Driven Development) 스타일로 변경 했습니다.

📝 Description

  • Hamcrest 메소드 적용
    • 기존 JUnit 메소드(assertEquals, assertTrue 등)를 Hamcrest의 assertThat으로 변경하여 가독성을 높였습니다.
    • 테스트 실패 시 더 명확한 오류 메시지를 제공 받을 수 있습니다.
  • BDD 스타일로 변경
    • 기존의 when, verify 구문을 BDD 스타일 given,then으로 변경하여, 행동 기반 개발 패턴을 따라 리펙토링 하고자 합니다.

📚 References

✅ Checklist

  • 새로운 기능이나 수정된 기능에 대해 충분한 테스트를 작성했습니다.
  • 코딩 스타일 가이드를 준수했습니다.
  • 문서(주석, README 등)가 필요하다면 업데이트했습니다.

- Hamcrest를 사용해 테스트 코드 수정
- BDD 스타일 적용
@Srltas Srltas added the test Task with test code label Sep 21, 2024
@Srltas Srltas self-assigned this Sep 21, 2024
@Srltas Srltas added the needs review Code review required label Sep 22, 2024
Copy link
Collaborator

@f-lab-bradley f-lab-bradley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DisplayName 으로 테스트의 목적이 더 명확해 졌네요.
잘 적용 하셨어요 ㅎㅎ
DisplayName 에 테스트에 대한 목적을 잘 적어 두시면 test 함수의 이름은 조금 간결하게 작성 하셔도 됩니다~
왜냐하면 목적이 Display 에 전부 나타나 있기 때문이죠

@Srltas Srltas removed the needs review Code review required label Sep 22, 2024
@Srltas Srltas merged commit dcc8624 into main Sep 22, 2024
1 check passed
@Srltas Srltas deleted the feature_issue_16 branch September 22, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Task with test code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants