[CHORE] 분석/히스토리 응답 데이터 추가

[ownue]

📍 개요

분석 마디 선택 정보 조회와 히스토리 도메인의 응답 데이터 추가

⛓️‍💥 관련 이슈

• Closes 🧹 Chore - 분석/히스토리 응답 DTO 수정 #209

---

🛠️ 작업 내용

• 프론트 요청에 따라 backingTrackId를 내려주도록 변경했습니다.
• 분석 마디 선택 정보 조회의 응답 데이터를 추가했습니다.
• 히스토리 상세, 목록 조회에도 동일한 응답 데이터를 추가했습니다.
• 해당 데이터를 추가하며 테스트 코드도 일부 수정했습니다.
• 단, 분석 마디 선택 정보 조회에는 백킹트랙이 미존재할 시 오류를 내려주지만 히스토리 도메인에서는 null을 내려줍니다. 해당 부분은 별도 정책을 확인하지 못해 기존 코드와 타 도메인 코드를 적절히 반영하여 작성한 것입니다.

---

🔥 리뷰 요청 사항

리뷰어가 중점적으로 확인해주었으면 하는 내용을 작성해주세요.

• 최소 수정만이 반영됐는지

---

✅ 체크리스트

• 코드 컨벤션을 준수했습니다.
• 불필요한 코드 및 import를 제거했습니다.
• 예외 처리를 적용했습니다.
• 테스트를 완료했습니다.
• 관련 Issue를 연결했습니다.

---

📎 참고 사항

Summary by CodeRabbit

• 새 기능

  • 분석 컨텍스트 응답에 백킹트랙 ID가 포함됩니다.
  • 분석 결과 조회 시 연결된 백킹트랙 정보를 함께 확인할 수 있습니다.

• 테스트

  • 백킹트랙 ID가 응답에 정확히 포함되는지 검증을 추가했습니다.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d9fbac70-4931-4bdd-bb11-6d8a61387f4d

📥 Commits

Reviewing files that changed from the base of the PR and between f401771 and 3389f49.

📒 Files selected for processing (3)

• src/main/java/com/mr/domain/playing/dto/res/AnalysisContextResponse.java
• src/test/java/com/mr/domain/playing/controller/PlayingControllerTest.java
• src/test/java/com/mr/domain/playing/service/PlayingServiceTest.java

---

📝 Walkthrough

Walkthrough

분석 컨텍스트 응답에 backingTrackId를 추가했습니다. BackingTrack의 ID를 응답에 매핑하고, 컨트롤러 및 서비스 테스트에서 반환값을 검증합니다.

Changes

분석 컨텍스트 응답

Layer / File(s)	Summary
응답 계약 및 ID 매핑 src/main/java/com/mr/domain/playing/dto/res/AnalysisContextResponse.java	AnalysisContextResponse에 backingTrackId를 추가했습니다. from 메서드는 BackingTrack.getId()를 응답에 전달합니다.
응답 검증 src/test/java/com/mr/domain/playing/controller/PlayingControllerTest.java, src/test/java/com/mr/domain/playing/service/PlayingServiceTest.java	컨트롤러와 서비스 테스트에서 backingTrackId 반환값을 검증합니다.

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

Possibly related PRs

• Musereview/BE#51: 백킹트랙 생성 및 수정 API를 추가한 변경입니다.
• Musereview/BE#62: 백킹트랙 API와 엔티티 참조를 변경한 PR입니다.
• Musereview/BE#161: AnalysisContextResponse와 관련 테스트를 변경한 PR입니다.

Poem

백킹트랙 ID가
응답에 살며시 도착해
서비스와 컨트롤러가
같은 값을 확인하니
재연주의 길이 열렸네 🎵

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	DTO와 정상 응답의 backingTrackId 테스트는 반영했지만, 백킹트랙이 없을 때 BACKING_TRACK_NOT_FOUND를 반환하는 구현은 확인되지 않습니다 [#209].	백킹트랙 ID가 없는 연주를 검증하고 BACKING_TRACK_NOT_FOUND를 반환하도록 서비스 로직과 테스트를 추가하세요.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Out of Scope Changes check	✅ Passed	변경 사항은 backingTrackId 응답 추가와 관련 테스트 수정으로 연결 이슈의 범위 안에 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목은 분석 응답에 데이터를 추가하는 주요 변경 사항을 간결하게 설명합니다.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/#209-analysis-context-response

---

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.}

[rkdehdrbs7885-oss]

기존 로직이나 불필요한 코드 변경 없이, DTO 응답 필드 추가와 관련된 테스트 코드 보완만 잘 적용된 것 같습니다! 수고하셨습니다!

[on1yoneprivate]

수고하셨습니다~!