Fix/#169 샘플 도서 예시 대목/의견 미표시

[Byesol]

📌 관련 이슈

• Close fix: 샘플 도서 예시 대목/의견 미표시 - OpinionGuestSampleSeedRunner가 옛 하드코딩 bookId 참조 #169

🚀 개요

• prod에서 샘플 도서(홈 화면 고정 카드) 상세로 들어가면 예시 대목/의견이 0개로 보이던 문제를 수정했습니다. 원인은 실제 대목/의견을 씨딩하는 OpinionGuestSampleSeedRunner가 fix: 샘플 도서 404 - 하드코딩된 로컬 book id가 환경마다 달라 prod에서 존재하지 않는 도서를 가리킴 #166 이전의 옛 하드코딩 bookId=18로 book을 찾고 있었기 때문입니다.

📄 작업 내용

• OpinionGuestSampleSeedRunner: bookRepository.findById(18L) 대신 bookRepository.findByIsbn(SampleLibraryBook.ISBN)으로 샘플 book을 조회하도록 수정 (#166에서 도입한 ISBN 기준 조회 방식과 일치)
• SampleLibraryBookSeeder(book row 생성)가 OpinionGuestSampleSeedRunner(대목/의견 생성)보다 먼저 실행되도록 두 ApplicationRunner에 @Order 지정 — 순서 보장 없이는 최초 기동 시 book이 아직 없어 씨딩이 건너뛰어질 수 있음
• OpinionGuestSampleSeedRunnerTest 신규 작성: 샘플 book 존재 시 대목 2개/의견 2개 생성, book 없을 때 건너뜀, 재실행 시 중복 생성 안 함 검증

📸 스크린샷 / 테스트 결과 (선택)

• ./gradlew test 통과 (관련 테스트: OpinionGuestSampleSeedRunnerTest 3건, SampleLibraryBookSeederTest 2건, BookServiceTest 27건 모두 통과)
• 무관한 기존 실패: AladinBookApiClientCacheTest(로컬 Redis 미실행), NotificationRepositoryTest(동시각 정렬 플레이키) — 이번 변경과 무관, 단독 실행 시 통과 확인

✅ 체크리스트

• 브랜치 전략(GitHub Flow)을 준수했나요?
• 메서드 단위로 코드가 잘 쪼개져 있나요?
• 테스트 통과 확인
• 서버 실행 확인
• API 동작 확인

---

🔍 리뷰 포인트 (Review Points)

• 홈 카드에 표시되는 "대목 13개 / 의견 17개"는 실제 book row와 무관한 고정 표시값(BookService.SAMPLE_LIBRARY_BOOK_PASSAGE_COUNT/..._OPINION_COUNT)이라, 이번 수정 후에도 실제 상세 화면 개수(2/2)와는 다르게 보입니다. 이 고정값을 실제 씨딩 개수에 맞출지는 별도 논의가 필요할 것 같습니다.
• ApplicationRunner 실행 순서를 @Order로 명시적으로 강제했습니다 — 기존에는 두 러너 모두 순서 미지정이라 최초 배포 시 우연에 의존하고 있었습니다.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CGkkcMzuV3iWsbSRkqeuwv

Summary by CodeRabbit

• 개선

  • 샘플 도서와 관련 의견 데이터가 정해진 순서에 따라 안정적으로 생성됩니다.
  • 도서를 ISBN 기준으로 조회하여 환경별 식별자 차이로 인한 오류를 줄였습니다.
  • 대상 도서가 없을 경우 안내 후 데이터 생성을 건너뜁니다.
  • 초기화 작업을 반복 실행해도 샘플 데이터가 중복 생성되지 않습니다.

• 테스트

  • 샘플 데이터 생성, 대상 도서 부재, 중복 실행 상황을 검증하는 테스트를 추가했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: f2fd798a-12ee-4a97-9cac-3df27ba8cb3c

📥 Commits

Reviewing files that changed from the base of the PR and between 14bbbe1 and f93d010.

📒 Files selected for processing (3)

• src/main/java/com/nexters/palang/domain/book/infrastructure/SampleLibraryBookSeeder.java
• src/main/java/com/nexters/palang/domain/opinion/infrastructure/OpinionGuestSampleSeedRunner.java
• src/test/java/com/nexters/palang/domain/opinion/infrastructure/OpinionGuestSampleSeedRunnerTest.java

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

---

Walkthrough

샘플 도서 시더의 실행 순서를 지정했습니다. 의견 시더는 고정 도서 ID 대신 SampleLibraryBook.ISBN으로 도서를 조회합니다. 도서 존재 여부와 중복 실행에 대한 테스트를 추가했습니다.

Changes

샘플 데이터 시딩

Layer / File(s)	Summary
샘플 시더 실행 순서 src/main/java/com/nexters/palang/domain/book/infrastructure/SampleLibraryBookSeeder.java, src/main/java/com/nexters/palang/domain/opinion/infrastructure/OpinionGuestSampleSeedRunner.java	SampleLibraryBookSeeder에 ORDER와 @Order를 추가했습니다. OpinionGuestSampleSeedRunner는 샘플 도서 시더 다음에 실행됩니다.
ISBN 기반 의견 시딩 src/main/java/com/nexters/palang/domain/opinion/infrastructure/OpinionGuestSampleSeedRunner.java, src/test/java/com/nexters/palang/domain/opinion/infrastructure/OpinionGuestSampleSeedRunnerTest.java	고정 PK 18L 조회를 SampleLibraryBook.ISBN 조회로 변경했습니다. 샘플 도서가 없으면 시딩하지 않습니다. 샘플 계정과 대목·의견 생성 및 재실행 시 중복 방지를 테스트합니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to f93d0

샘플 도서는 ISBN으로 안정적으로 조회되고 도서 생성 후 예시 대목과 의견이 시딩됩니다. 중복 생성도 방지되어 현재 변경은 병합 가능한 상태입니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 3 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	[169]의 모든 코딩 목표를 충족합니다. OpinionGuestSampleSeedRunner는 SampleLibraryBook.ISBN으로 도서를 조회하고, SampleLibraryBookSeeder의 실행 순서를 보장합니다. 도서 존재 여부, 대목·의견 2개 생성, 중복 생성 방지 테스트도 추가했습니다. 홈 카드의 고정 표시값 차이는 별도 논…
Out of Scope Changes check	✅ Passed	변경 사항은 ISBN 기반 조회, 시딩 실행 순서 보장, 관련 테스트 추가로 제한됩니다. 모든 변경 사항이 [169]의 목표와 직접 관련됩니다.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 샘플 도서의 예시 대목과 의견이 표시되지 않는 문제를 수정하는 주요 변경 사항을 정확하게 요약합니다.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/#169-sample-opinions-missing

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}