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

[TEST] 방과 연관된 정보 삭제 테스트 활성화 #398

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

leegwichan
Copy link
Contributor

Issue Number

Closed #397

As-Is

  • @LastModifiedDate 문제로 비활성화 되어 있는 테스트 존재
  • 이를 수정하여 해당 기능이 정상 작동하는지 확인할 수 있도록 테스트 활성화 필요

To-Be

  • 변경_시간이_특정_시각_이전_방_삭제.해당_방과_연관된_모든_정보를_삭제할_수_있다 테스트 수정

    • 특정 시간 이후에 방 정보들이 삭제하는지 확인하는 테스트
    • 방 생성 후에, 특정 시간을 LocalDateTime.now()로 이용해서 설정 -> '특정 시간'을 방 생성 시간 이후로 설정 함
  • init.sql에 이미 저장되어 있는 방까지 같이 삭제되어 옮긴 투표 수가 1개 초과로 늘어남

    • 추후 작업에서 init.sql 삭제 시, isGreaterThanOrEqualToisEqualTo로 변경 필요

Check List

  • 테스트가 전부 통과되었나요?
  • 모든 commit이 push 되었나요?
  • merge할 branch를 확인했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

Test Screenshot

image

(Optional) Additional Description

@leegwichan leegwichan added ✨ feat 기능 추가 🍃 BE back end labels Nov 7, 2024
@leegwichan leegwichan self-assigned this Nov 7, 2024
Copy link
Member

@PgmJun PgmJun left a comment

Choose a reason for hiding this comment

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

고생하셨어요 커찬~ 코멘트 확인 부탁해요~!

Comment on lines +612 to +613
() -> assertThat(afterCountOfTotalVotes).isGreaterThanOrEqualTo(countOfTotalVotes + 1)
// TODO init.sql 제거 후 isEqualTo로 변경 (현재 다른 방들의 표까지 이관되어 1표 이상으로 더 늘어남)
Copy link
Member

Choose a reason for hiding this comment

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

확실히 이 부분은 테스트간의 격리가 좀 더 추진된 이후에 적용해야하겠네요
또 TODO를 만드는 것보다 같이 처리해보는 게 어떨까요..?
제가 Fixture 관련 테스트 도구 만들고 init.sql 제거하는 작업을 먼저 해서 merge 시키고
커찬이 이 부분 좀 더 손 대주신 뒤에 해당 PR을 Merge 해볼까요??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍃 BE back end ✨ feat 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[TEST] 방과 연관된 정보 삭제 테스트 활성화
2 participants