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

[FIX] 통과하지 않는 테스트를 통과시킨다. #145

Merged
merged 12 commits into from
Aug 26, 2024

Conversation

kssumin
Copy link
Contributor

@kssumin kssumin commented Aug 24, 2024

📌 관련 이슈

closed #143

✒️ 작업 내용

  • 기존에 주석으로 처리되었던 테스트 코드를 작성했습니다.

테스트용 DB 삭제

  • 테스트용 DB가 따로 사용되고 있지 않아 삭제처리 하였습니다.

CommandMemberService -> AdminMemberService

  • CommandMemberService 클래스는 어드민이 멤버에 대해서 할 수 있는 메서드만 가지고 있었습니다.
  • 따라서 해당 클래스의 책임을 명확하게 하고자 AdminMemberService로 변경하였습니다.
  • 또한 해당 테스트는 데이터베이스에 값이 제대로 저장이 되는지도 확인이 필요하다고 생각하여 통합테스트를 이용하였습니다.

DataBaseCleaner

  • 테스트 격리를 위해 각 테스트의 전에 데이터베이스의 데이터를 지우는 역할을 수행합니다.

AttendModel에서 상태를 변경하는 방법이 일반 멤버와 어드민이 다른 이유에 대해 궁금합니다.

프로그램 생성 시 github url을 포함

  • 기존 테스트에서는 github url을 포함하고 있지 않아 프로그램 생성 가능 여부 메서드를 호출할 때 예외가 발생했습니다.
  • 따라서 Fixture 객체를 사용하여 프로그램 생성 시 github url을 포함하도록 하였습니다.

스크린샷 🏞️ (선택)

💬 REVIEWER에게 요구사항 💬

@kssumin kssumin added the bug Something isn't working label Aug 24, 2024
@kssumin kssumin self-assigned this Aug 24, 2024
Copy link

github-actions bot commented Aug 24, 2024

Unit Test Results

75 tests  +75   74 ✔️ +74   1s ⏱️ -1s
29 suites +29     1 💤 +  1 
29 files   +29     0 ±  0 

Results for commit 28399c6. ± Comparison against base commit 5d93087.

♻️ This comment has been updated with latest results.

@kssumin kssumin merged commit 537ff32 into develop Aug 26, 2024
2 checks passed
@kssumin kssumin deleted the MD/fix/143/not-working-test branch August 27, 2024 01:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 통과하지 않는 테스트를 통과시킨다.
1 participant