pricelees/roomescape-refactored
1
0
Fork 0
generated from pricelees/issue-pr-template
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

[#11] Payment 도메인 코드 코틀린 마이그레이션 #12

Merged
pricelees merged 32 commits from refactor/#11 into main 2025-07-16 09:19:28 +00:00
Conversation 0 Commits 32 Files Changed 48 +1195 -955
pricelees commented 2025-07-16 09:19:14 +00:00
Owner
Copy Link

📝 관련 이슈 및 PR

PR과 관련된 이슈 번호

✨ 작업 내용

payment 패키지 내 코드, 테스트를 코틀린으로 전환했고 일부 로직은 개선하였음. 전체적으로 구조를 개선하려고 했으나, 얽혀있는 예약 관련 로직이 많아 전체 코드의 코틀린 전환이 끝난 이후 개선할 예정

🧪 테스트

  1. @DataJpaTest를 이용하는 Repository 테스트를 추가
  2. Service는 mocking 방식으로 수정하였고, 테스트가 불필요하다고 여겨지는 단순 로직(변환 또는 Repository만 사용하는 경우)은 제외하였음. (8577b68496)
  • 전체 로직이 테스트되어있는 기존의 테스트는 유지하였고, 전체 코틀린 전환이 마무리 된 후 제거 예정

📚 참고 자료 및 기타

<!-- 제목 양식 --> <!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) --> ## 📝 관련 이슈 및 PR **PR과 관련된 이슈 번호** - #11 ## ✨ 작업 내용 <!-- 어떤 작업을 했는지 알려주세요! --> payment 패키지 내 코드, 테스트를 코틀린으로 전환했고 일부 로직은 개선하였음. 전체적으로 구조를 개선하려고 했으나, 얽혀있는 예약 관련 로직이 많아 전체 코드의 코틀린 전환이 끝난 이후 개선할 예정 ## 🧪 테스트 <!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! --> 1. \@DataJpaTest를 이용하는 Repository 테스트를 추가 2. Service는 mocking 방식으로 수정하였고, 테스트가 불필요하다고 여겨지는 단순 로직(변환 또는 Repository만 사용하는 경우)은 제외하였음. (8577b68496) - 전체 로직이 테스트되어있는 기존의 테스트는 유지하였고, 전체 코틀린 전환이 마무리 된 후 제거 예정 ## 📚 참고 자료 및 기타 <!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
pricelees added the
Kind/Enhancement
 label 2025-07-16 09:19:14 +00:00
pricelees self-assigned this 2025-07-16 09:19:14 +00:00
pricelees added 32 commits 2025-07-16 09:19:14 +00:00 
- 읽고 null이 아니면 필드 조회 -> 필드를 쿼리로 바로 조회
- 단순한 존재 여부 판단 -> find => exists로 수정
- 검증이 불필요하다고 느껴지는 로직은 생략
pricelees merged commit ed383c3092 into main 2025-07-16 09:19:28 +00:00
pricelees deleted branch refactor/#11 2025-07-16 09:19:28 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Kind/Enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
pricelees
No Assignees pricelees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pricelees/roomescape-refactored#12
No description provided.
Delete Branch "refactor/#11"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?