Skip to content

Conversation

@1winhyun
Copy link
Member

@1winhyun 1winhyun commented Mar 12, 2025

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 리팩토링
  • 코드에 영향을 주지 않는 변경사항(주석, 개행 등등..)
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 테스트 코드 추가

✏️ 작업 내용

1. invitationInformationRepository 테스트 코드를 작성하였습니다.

  • 코드 작성 중 다음과 같이 코드를 작성할 경우 오류가 발생하는 것을 알아냈습니다. 이는 두 값이 모두 LocalDateTime 타입이어도 내부 nanosecond 정밀도가 다르기 때문에 문제가 발생하는 것이었습니다.
  • 따라서 다음과 같이 코드를 수정하여 정밀도를 동일하게 설정하였습니다.
//수정 전
foundInfo.schedule shouldBe invitationInformationVO.schedule

//수정 후
foundInfo.schedule.truncatedTo(ChronoUnit.MILLIS) shouldBe invitationInformationVO.schedule.truncatedTo(ChronoUnit.MILLIS)

2. invitationThumbailRepository 테스트 코드를 작성하였습니다.


🔗 관련 이슈


💡 추가 사항

@1winhyun 1winhyun added 🎨 style 코드 스타일 변경 ✅ test 테스트 코드 labels Mar 12, 2025
@1winhyun 1winhyun requested a review from yechan-kim March 12, 2025 08:35
@1winhyun 1winhyun self-assigned this Mar 12, 2025
@1winhyun 1winhyun linked an issue Mar 12, 2025 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Mar 12, 2025

Test Results

31 files  31 suites   2s ⏱️
87 tests 87 ✅ 0 💤 0 ❌
88 runs  88 ✅ 0 💤 0 ❌

Results for commit aae3656.

♻️ This comment has been updated with latest results.

Copy link
Member

@yechan-kim yechan-kim left a comment

Choose a reason for hiding this comment

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

승현님 고생하셨습니다!

PR과 코드를 읽어보면서 딱 궁금한 점 하나 있는데, 코멘트 달아놨으니 답변 부탁드립니다!

@1winhyun 1winhyun merged commit a4096f4 into main Mar 13, 2025
2 checks passed
@1winhyun 1winhyun deleted the refactor/#51 branch March 15, 2025 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎨 style 코드 스타일 변경 ✅ test 테스트 코드

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ Information, Thumnbail 관련 Repository 테스트 코드 작성

3 participants