Skip to content

feat(datingexam): 선택 과목 정보 조회 API 삭제 (#451) - #455

Merged
stemmmm merged 1 commit into
developfrom
feat/451
Jul 20, 2026
Merged

feat(datingexam): 선택 과목 정보 조회 API 삭제 (#451)#455
stemmmm merged 1 commit into
developfrom
feat/451

Conversation

@stemmmm

@stemmmm stemmmm commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

개요

GET /dating-exam/optional 선택 과목 정보 조회 API를 삭제합니다.

변경 사항

  • 컨트롤러 DatingExamApi.getOptionalExamInfo 제거
  • 삭제로 사용되지 않게 된 DatingExamFinder.findOptionalExamInfo (인터페이스 + DatingExamQueryService 구현) 제거
  • SubjectType.OPTIONAL enum은 제출/소울메이트 흐름에서 계속 사용되므로 유지

검증

  • ./gradlew compileJava 통과
  • getOptionalExamInfo/findOptionalExamInfo 잔존 참조 없음 (남은 1건은 레포지토리 테스트 메서드명으로 무관)

Closes #451

Summary by CodeRabbit

  • 변경 사항
    • 선택 과목 정보 조회 기능이 제거되었습니다.
    • 이제 시험 정보 조회는 필수 과목 정보와 대표 성격 유형 정보 조회를 제공합니다.

GET /dating-exam/optional 엔드포인트와 이로 인해 사용되지 않게 된
DatingExamFinder.findOptionalExamInfo (인터페이스 + 구현) 제거.
SubjectType.OPTIONAL enum은 제출/소울메이트 흐름에서 계속 사용되므로 유지.
@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 21c9810a-1bfd-4720-974a-92f227179ca5

📥 Commits

Reviewing files that changed from the base of the PR and between b6410de and 3ca9b95.

📒 Files selected for processing (3)
  • src/main/java/deepple/deepple/datingexam/adapter/webapi/DatingExamApi.java
  • src/main/java/deepple/deepple/datingexam/application/DatingExamQueryService.java
  • src/main/java/deepple/deepple/datingexam/application/provided/DatingExamFinder.java
💤 Files with no reviewable changes (3)
  • src/main/java/deepple/deepple/datingexam/application/provided/DatingExamFinder.java
  • src/main/java/deepple/deepple/datingexam/adapter/webapi/DatingExamApi.java
  • src/main/java/deepple/deepple/datingexam/application/DatingExamQueryService.java

Walkthrough

GET /dating-exam/optional endpoint가 DatingExamApi에서 제거되었습니다. 이에 따라 DatingExamFinderfindOptionalExamInfo 계약과 DatingExamQueryService의 구현 메서드도 삭제되었습니다. 필수 과목 조회 다음에는 대표 성격 유형 조회 경로가 이어집니다.

Estimated code review effort: 1 (Trivial) | ~3 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 선택 과목 정보 조회 API 삭제라는 핵심 변경을 짧고 명확하게 요약합니다.
Description check ✅ Passed 개요, 변경 사항, 검증, Closes #451이 포함되어 주요 내용은 충분히 전달됩니다.
Linked Issues check ✅ Passed 직접 연결된 #451의 요구사항인 /dating-exam/optional 삭제를 코드 변경이 충족합니다.
Out of Scope Changes check ✅ Passed 요구사항과 무관한 변경은 보이지 않고 optional 조회 제거에만 집중되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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 feat/451

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

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

@github-actions

Copy link
Copy Markdown

Test results

814 tests   814 ✅  15s ⏱️
246 suites    0 💤
246 files      0 ❌

Results for commit 3ca9b95.

@stemmmm
stemmmm merged commit be39db0 into develop Jul 20, 2026
4 checks passed
@stemmmm
stemmmm deleted the feat/451 branch July 20, 2026 03:46
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.

1 participant