<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #13
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
theme 패키지 내 코드 및 테스트 코틀린 전환
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
다른 테스트는 코틀린으로 전환 시 크게 문제가 없었으나, GET /themes/most-reserved-last-week API의 경우 쿼리에 크게 의존하여 mocking을 사용하는 기존 테스트로 처리하기 애매한 부분이 있었음.
따라서, API 테스트는 mocking이 아닌 RestAssured를 이용한 실제 테스트로 진행하였고 \@RequestParam, 날짜 등 실제 비즈니스와 관련된 부분을 위주로 처리하고 쿼리 자체는 Repository 테스트에서 상세하게 검증하였음.
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
패키지를 reservation 안에 넣는 것은 고민이 조금 더 필요할 것 같음. 현재는 단일 매장에 대한 서비스지만 매장별로 분리하는 것을 고민중인 만큼 코틀린 마이그레이션이 끝난 이후 생각해볼 예정
Reviewed-on: #15
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #11
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
payment 패키지 내 코드, 테스트를 코틀린으로 전환했고 일부 로직은 개선하였음. 전체적으로 구조를 개선하려고 했으나, 얽혀있는 예약 관련 로직이 많아 전체 코드의 코틀린 전환이 끝난 이후 개선할 예정
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
1. \@DataJpaTest를 이용하는 Repository 테스트를 추가
2. Service는 mocking 방식으로 수정하였고, 테스트가 불필요하다고 여겨지는 단순 로직(변환 또는 Repository만 사용하는 경우)은 제외하였음. (8577b68496)
- 전체 로직이 테스트되어있는 기존의 테스트는 유지하였고, 전체 코틀린 전환이 마무리 된 후 제거 예정
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #12
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #9
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
기존에 정의된 \@SpringbootTest 기반 API 테스트를 MockK + WebMvcTest 기반으로 전환하였음.
- 기존 \@SpringbootTest 기반의 테스트도 대부분 로직을 mocking하고 있기에, 불필요한 Spring Context 로딩 비용을 줄일 수 있다고 판단하였음.
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
- 기존에 있는 3개의 클래스를 리팩터링 전후 비교 결과 리팩터링 후 소요시간이 3회 측정 평균 1.5736초에서 0.6153초로 61% 정도 감소하였음.
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #10
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #7
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
1. 808c6675 에서 작업했던 정상 / 오류를 통합하는 객체를 사용하려 했으나, Swagger-UI상에서 응답 형식에 null 필드가 포함되는 문제로 다시 정상 / 오류 별도로 분리
2. Swagger-UI(문서화, 명세) 관련 코드는 인지하기 쉽도록 ../web -> ../docs 패키지로 이전
3. 현재까지 코틀린으로 마이그레이션 된 서비스를 대상으로, 응답에 ResponseEntity를 적용하고 \@ResponseStatus 제거
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
예정으로는 Issue에 작성했던 테스트까지 처리하려고 했으나, 테스트는 바로 다음에 진행 예정
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #8
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #5
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
1. 추후 모듈 분리를 위해 패키지를 조금 더 직관적으로 분리하고자 했음. 기존 system 패키지 내부에 있는 auth는 ../auth로, dto 및 exception은 common/.. 하위로 이동함.
2. 이동한 클래스들은 모두 코틀린으로 전환하였고, 일부 기능까지 수정하려 했으나 추후에 한 번에 수정하는게 낫다고 판단함.
3. 다음 PR에서는 현재까지 코틀린으로 변환된 클래스를 대상으로 808c6675 에 있는 새로운 응답 객체를 적용할 예정
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
기존의 API 테스트가 SpringbootTest를 사용하여 속도가 상당히 느림. 808c6675 의 테스트를 진행하며 작성해둔 MockMvcTest의 틀을 바탕으로, 코틀린으로 전환된 클래스를 대상으로 MockMvc로 전환할 예정.
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #6
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #3
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
### 0. 공통
- 패키지 구조 수정(web, business, infrastructure 구조)
- Swagger-UI 어노테이션을 별도의 인터페이스를 만들어 컨트롤러에서 분리
- 결합도 높은 클래스 통합(ex: Member 엔티티와 Role enum 등)
### 1. 회원 도메인
- 기능 자체가 적어서 변화된 내용이 크게 없음. 패키지 구조 수정과 클래스 통합 정도의 과정이 대부분이었음.
### 2. 인증 도메인
- 전체적으로 코드 중복이 많아, 확장함수 및 클래스 통합으로 중복 코드를 상당히 많이 제거하였음.
- JwtHandler와 Interceptor에서 모두 이뤄지던 null 예외 처리를 JwtHandler에서만 처리하도록 수정
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
- 모든 테스트는 Kotest 기반으로 수정 & 로그인 및 토큰 처리가 필요한 API 테스트는 mocking을 활용하도록 수정
- 향후 테스트도 꼭 필요하다고 느껴지는 테스트가 아니라면 DB 사용보다는 mocking을 활용할 예정
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #4
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
<!-- 제목 양식 -->
<!-- [이슈번호] 작업 요약 (예시: [#10] Gitea 템플릿 생성) -->
## 📝 관련 이슈 및 PR
**PR과 관련된 이슈 번호**
- #1
## ✨ 작업 내용
<!-- 어떤 작업을 했는지 알려주세요! -->
### 1. 코틀린 의존성 추가 및 정상 작동 확인
- Kotest, RestAssured Extension 등 코틀린 관련 플러그인 추가
### 2. View 관련 코드 코틀린 마이그레이션
- 애플리케이션에서는 3개 클래스 78 라인 -> 단일 클래스 48 라인으로 코드량 감소
- 테스트에서는 3개 클래스 344 라인 -> 단일 클래스 161 라인으로 코드량 감소
## 🧪 테스트
<!-- 어떤 테스트를 생각했고 진행했는지 알려주세요! -->
코틀린 마이그레이션 이후 테스트 진행 및 기존 테스트와 동일한 커버리지가 나오는 것을 확인하였음.
## 📚 참고 자료 및 기타
<!-- 참고한 자료, 또는 논의할 사항이 있다면 알려주세요! -->
Reviewed-on: #2
Co-authored-by: pricelees <priceelees@gmail.com>
Co-committed-by: pricelees <priceelees@gmail.com>
