Skip to content

refactor: 선수 로드맵 영속 상태 해석 책임 분리 - #717

Merged
chaeyn merged 2 commits into
mainfrom
refactor/section-prerequisite-persistence/#716
Jul 29, 2026
Merged

refactor: 선수 로드맵 영속 상태 해석 책임 분리#717
chaeyn merged 2 commits into
mainfrom
refactor/section-prerequisite-persistence/#716

Conversation

@Finefinee

Copy link
Copy Markdown
Member

변경사항

  • SectionJpaMapper에서 Repository 의존성을 제거하고, 선수 Section을 인자로 받아 순수 변환만 수행하도록 변경했습니다.
  • Persistence Adapter에서 선수 Section ID의 null·존재 여부를 검증하고 경량 조회로 managed 엔티티를 연결했습니다.
  • 선수 관계가 있는 Section 단건 조회의 MultipleBagFetchException을 방지하도록 fetch 전략을 조정했습니다.
  • 선수 관계 저장과 미존재 선수 Section 차단 회귀 테스트를 추가했습니다.

관련 이슈

Closes #716

추가 컨텍스트

  • 검증: ./gradlew test --tests com.process.clash.application.roadmap.section.service.UpdateSectionServiceIntegrationTest
  • 테스트 컨텍스트에 Redis 의존성이 있어 테스트 전용 Redis를 기동해 실행했습니다.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@chaeyn chaeyn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

수고하셨습니다.

@chaeyn
chaeyn merged commit e961214 into main Jul 29, 2026
1 check passed
@chaeyn
chaeyn deleted the refactor/section-prerequisite-persistence/#716 branch July 29, 2026 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 선수 로드맵 영속 상태 해석 책임 분리

2 participants