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.

Sign up for GitHub

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

Jump to bottom

정복 리포트 로직 & 테스트 수정 #347

Merged
merged 4 commits into from
Jul 10, 2024
Merged

정복 리포트 로직 & 테스트 수정 #347

merged 4 commits into from
Jul 10, 2024
+31 −44

Conversation

jyoo0515
Copy link
Member

@jyoo0515 jyoo0515 commented Jul 10, 2024 •
edited
Loading

  • 테스트에서 이번달 전체에 대해서 매일 12시에 장소 정복을 하도록 테스트에서 세팅했는데,
  • 실제 구현 로직에서 이번 주 마지막 날, 이번 달 마지막 날 등을 createdAt 의 쿼리 조건으로 넣어서 모든 데이터를 다 가져왔습니다
  • 실제 앱에서는 createdAt 이 미래인 데이터가 없을테니 문제가 안되고 테스트에서만 터짐

Checklist

  • 충분한 양의 자동화 테스트를 작성했는가?
    • 계단정복지도 서비스는 사이드 프로젝트로 진행되는 만큼 충분한 QA 없이 배포되는 경우가 많습니다. 따라서 자동화 테스트를 꼼꼼하게 작성하는 것이 서비스 품질을 유지하는 데 매우 중요합니다.

@jyoo0515 jyoo0515 requested a review from a team as a code owner July 10, 2024 14:26
@github-actions GitHub Actions
Copy link

🔥🔥🔥 Backend CI Failed. github action link 🔥🔥🔥

@github-actions GitHub Actions
Copy link

🔥🔥🔥 Backend CI Failed. github action link 🔥🔥🔥

@github-actions GitHub Actions
Copy link

🔥🔥🔥 Backend CI Failed. github action link 🔥🔥🔥

@jyoo0515 jyoo0515 merged commit 32d2a2c into main Jul 10, 2024
1 check passed
@jyoo0515 jyoo0515 deleted the jason/20240710-fix-flaky-test branch July 10, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yongseongkim yongseongkim yongseongkim approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jyoo0515 @yongseongkim