Skip to content

chore: 회원가입 후 추가 정보 입력 예외 처리 추가#66

Merged
kon28289 merged 2 commits intodevfrom
chore/user
Jan 17, 2026
Merged

chore: 회원가입 후 추가 정보 입력 예외 처리 추가#66
kon28289 merged 2 commits intodevfrom
chore/user

Conversation

@kon28289
Copy link
Contributor

@kon28289 kon28289 commented Jan 17, 2026

🚀 1. 개요

  • 회원가입 후 추가 정보 입력 시 학교 이메일 및 학번이 중복되는 경우 409 예외를 반환하도록 합니다.
  • userService 단위 테스트를 추가합니다.

📝 2. 주요 변경 사항

  • validateDuplication 메서드를 통해 학교 이메일 및 학번이 중복되는 경우 409 예외를 반환합니다.

📸 3. 스크린샷 (API 테스트 결과)

Summary by CodeRabbit

릴리스 노트

  • 신기능

    • 등록 시 학교 이메일 및 학생 ID 중복 검사 기능 추가
  • 개선사항

    • 학교 이메일 중복 오류 메시지 명확화
  • 테스트

    • 사용자 서비스 단위 테스트 추가

✏️ Tip: You can customize this high-level summary in your review settings.

@kon28289 kon28289 requested a review from Juhye0k January 17, 2026 03:10
@kon28289 kon28289 self-assigned this Jan 17, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 17, 2026

Walkthrough

사용자 회원 가입 완료 프로세스에 학교 이메일과 학번 중복 검증 로직을 추가했습니다. 예외 메시지를 명확히 하고, 저장소에 조회 메서드를 추가하며, 검증 서비스 로직을 구현했습니다. 포괄적인 단위 테스트를 신규 추가했습니다.

Changes

콘텐츠 / 파일(들) 변경 요약
예외 처리
src/main/java/com/gpt/geumpumtabackend/global/exception/ExceptionType.java
DUPLICATED_SCHOOL_EMAIL 열거형 상수의 사용자 친화적 메시지를 "이미 사용중인 이메일입니다"에서 "이미 사용중인 학교 이메일입니다"로 변경
저장소 메서드 추가
src/main/java/com/gpt/geumpumtabackend/user/repository/UserRepository.java
중복 검증을 위한 두 개의 존재 여부 확인 메서드 추가: existsByStudentId(String), existsBySchoolEmail(String)
검증 로직 구현
src/main/java/com/gpt/geumpumtabackend/user/service/UserService.java
completeRegistration에 validateDuplication() 프라이빗 메서드를 호출하여 학교 이메일 및 학번 중복 검증 추가; TODO 주석 제거
단위 테스트 추가
src/test/java/com/gpt/geumpumtabackend/unit/user/service/UserServiceTest.java
관리자 확인, 닉네임 생성, 회원 가입 완료, 프로필 조회, 닉네임 가용성, 프로필 업데이트, 로그아웃, 탈퇴/복구 흐름 등을 포함한 UserService 전체 기능에 대한 439줄의 포괄적인 단위 테스트 스위트 추가

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • Juhye0k
  • patulus

Poem

🐰 학교 이메일 중복, 이제는 명확히
검증 로직 탄탄하게, 학번도 함께
테스트 439줄로 든든히 지켜내니
사용자 가입, 더는 헷갈리지 않으리 ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 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 (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항의 주요 내용을 명확하게 요약하고 있으며, 회원가입 후 예외 처리 추가라는 핵심 변경을 잘 설명하고 있습니다.
Description check ✅ Passed PR 설명이 저장소의 템플릿 구조를 따르고 있으며, 개요, 주요 변경 사항 섹션이 채워져 있고 핵심 내용을 담고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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 and usage tips.

Copy link
Contributor

@Juhye0k Juhye0k left a comment

Choose a reason for hiding this comment

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

고생하셨습니다

@kon28289 kon28289 merged commit 3d7bedc into dev Jan 17, 2026
4 checks passed
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.

2 participants