Skip to content

feat: 지원 대학 단건 조회 API 추가 - #818

Merged
whqtker merged 1 commit into
developfrom
feat/817-add-univ-apply-info-detail-api
Aug 1, 2026
Merged

feat: 지원 대학 단건 조회 API 추가#818
whqtker merged 1 commit into
developfrom
feat/817-add-univ-apply-info-detail-api

Conversation

@whqtker

@whqtker whqtker commented Aug 1, 2026

Copy link
Copy Markdown
Member

관련 이슈

작업 내용

GET /admin/univ-apply-infos/{id}를 추가해 전체 필드를 담은
AdminUnivApplyInfoResponse를 단건으로 조회할 수 있게 구현했습니다

특이 사항

리뷰 요구사항 (선택)

어드민 지원 대학 관리에서 수정 모달을 열 때 목록 조회(search/text)
응답만 사용하고 있어 정원 외 필드가 채워지지 않는 문제가 있었다.
search/text가 반환하는 미리보기 응답에는 애초에 해당 필드들이
포함되어 있지 않아 프론트에서 해결할 수 없었다.

GET /admin/univ-apply-infos/{id}를 추가해 전체 필드를 담은
AdminUnivApplyInfoResponse를 단건으로 조회할 수 있게 했다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EwpgqdNTrn8dGDr7g7WnYg
@whqtker whqtker self-assigned this Aug 1, 2026
@whqtker whqtker added the 기능 label Aug 1, 2026
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Walkthrough

1. **관리자 단건 조회 API 추가**  
	`GET /admin/univ-apply-infos/{id}` 요청을 처리하고 조회 결과를 반환합니다.
2. **서비스 조회 및 예외 처리 추가**  
	ID로 지원 정보를 조회하고, 대상이 없으면 `UNIV_APPLY_INFO_NOT_FOUND` 예외를 발생시킵니다.
3. **서비스 테스트 추가**  
	정상 조회 결과의 주요 필드와 존재하지 않는 ID의 예외를 검증합니다.

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

Suggested reviewers: gyuhyeok99, wibaek, hexeong

🚥 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%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 지원 대학 단건 조회 API 추가라는 주요 변경 사항을 명확하게 설명합니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 변경된 API를 포함해 설명이 대부분 완전합니다.
Linked Issues check ✅ Passed PR은 어드민의 지원 대학 수정에 필요한 단건 조회 API와 조회 실패 예외 처리를 구현해 #817의 목표를 충족합니다.
Out of Scope Changes check ✅ Passed 컨트롤러, 서비스, 단위 테스트 변경이 모두 단건 조회 API 구현 범위에 포함됩니다.
✨ 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 feat/817-add-univ-apply-info-detail-api

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
src/test/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoServiceTest.java (1)

521-543: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick win

1. 전체 필드 응답 계약을 검증하세요.

이 테스트는 koreanName, semesterAvailableForDispatch, extraInfo, languageRequirements를 검증하지 않습니다.

이 필드는 단건 상세 조회 API의 핵심 계약입니다. Fixture에 각 값을 명시적으로 설정하고 응답값을 검증하세요.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@src/test/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoServiceTest.java`
around lines 521 - 543, Update 존재하는_id로_조회하면_상세_정보를_반환한다() to explicitly set
koreanName, semesterAvailableForDispatch, extraInfo, and languageRequirements on
the UnivApplyInfo fixture, then add assertions that the response returns each
configured value through the corresponding AdminUnivApplyInfoResponse accessors.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In
`@src/test/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoServiceTest.java`:
- Around line 521-543: Update 존재하는_id로_조회하면_상세_정보를_반환한다() to explicitly set
koreanName, semesterAvailableForDispatch, extraInfo, and languageRequirements on
the UnivApplyInfo fixture, then add assertions that the response returns each
configured value through the corresponding AdminUnivApplyInfoResponse accessors.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 8d294572-3240-4db9-83df-b17402f4112f

📥 Commits

Reviewing files that changed from the base of the PR and between 065564c and 69140cd.

📒 Files selected for processing (3)
  • src/main/java/com/example/solidconnection/admin/university/controller/AdminUnivApplyInfoController.java
  • src/main/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoService.java
  • src/test/java/com/example/solidconnection/admin/university/service/AdminUnivApplyInfoServiceTest.java

@whqtker
whqtker merged commit 5e577bf into develop Aug 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 지원 대학 단건 조회 API 구현

1 participant