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

[BE] 시간에 따라 실패하는 테스트 수정 #538

Open
Hongdonggeon opened this issue Oct 20, 2022 · 0 comments · May be fixed by #539
Open

[BE] 시간에 따라 실패하는 테스트 수정 #538

Hongdonggeon opened this issue Oct 20, 2022 · 0 comments · May be fixed by #539
Assignees
Labels
🌾 backend Anything related to back-end 🐞 bug Something isn't working

Comments

@Hongdonggeon
Copy link
Collaborator

Hongdonggeon commented Oct 20, 2022

버그 기능

  • 시간에 따라서 테스트가 실패한다.

버그 시나리오

  • AttendanceAcceptanceTest의 showUserCoffeeStats()
    image

  • AttendanceAcceptanceTest의 useCoffeeStack()
    image

  • MeetingAcceptanceTEst의 findMy()
    image

기대 동작

  • 테스트가 통과해야한다.

현재 동작

  • 테스트가 실패하고 있다. 위의 세 테스트 모두 근본적인 문제는 attendance가 none에서 tardy 로 변경이 되어야하는 건데 변경이 안되고 있습니다. 위 테스트 메서드 모두 event를 EventFixtures의 createEventOnDate 메서드로 만들고 있습니다.
    image
    위 사진에서 보면 Event 시작시간은 10시입니다.

그리고 위 테스트 메서드 모두 attendanceScheduler의 updateToTardyAttendanceClosingTime()을 사용하고 있는데 이 메서드에서 tardy로 바꿔주는 repository 메서드를 보면 '오늘 날짜가 같고 시작 시간보다 더 큰 시간' 일 때 변경해주고 있습니다.
image

그래서 00시 부터 9시 59분까지는 repository의 쿼리에 조건에 부합하지 못하기 때문에 tardy로 변경되지 못하고 있어서 실패하고 있었습니다.

@Hongdonggeon Hongdonggeon added 🐞 bug Something isn't working 🌾 backend Anything related to back-end labels Oct 20, 2022
@Hongdonggeon Hongdonggeon self-assigned this Oct 20, 2022
@Hongdonggeon Hongdonggeon linked a pull request Oct 20, 2022 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌾 backend Anything related to back-end 🐞 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant