fix: 멘토 지원서 승인 시 유저 Role 을 Mentor로 승격

[sukangpunch]

관련 이슈

• resolves: fix: 멘토 지원서 승인 시 사용자 role을 MENTOR로 변경하도록 수정 #636

작업 내용

어드민에서 멘토 지원서를 승인 할 때, 유저의 Role 을 Mentor 로 승격하도록 기능을 수정하였습니다.

특이 사항

리뷰 요구사항 (선택)

[coderabbitai]

Walkthrough

이 변경사항은 멘토 신청 승인 프로세스에 사용자의 역할 업데이트 기능을 추가합니다:

1. AdminMentorApplicationService: 멘토 신청 승인 후 관련 SiteUser를 조회하여 becomeMentor() 메서드를 호출하는 로직 추가 (5줄 추가). SiteUser를 찾지 못할 경우 USER_NOT_FOUND 예외 발생 가능성 발생.

2. SiteUser 도메인: 사용자의 역할을 MENTOR로 설정하는 becomeMentor() 공개 메서드 신규 추가 (4줄 추가). 기존 메서드나 필드는 변경 없음.

3. AdminMentorApplicationServiceTest: SiteUserRepository 의존성 주입 및 Role 관련 import 추가. 신청 승인 테스트에서 승인 후 사용자의 역할이 MENTOR인지 검증하는 단언문 추가 (20줄 추가, 13줄 제거).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested reviewers

• wibaek
• whqtker
• Hexeong
• JAEHEE25

🚥 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	PR 제목은 주요 변경사항인 '멘토 지원서 승인 시 유저 Role을 Mentor로 승격'을 명확하게 설명하고 있습니다.
Description check	✅ Passed	PR 설명은 필수 섹션(관련 이슈, 작업 내용)을 포함하고 있으며, 요청된 템플릿의 핵심 정보를 다루고 있습니다.
Linked Issues check	✅ Passed	PR의 코드 변경사항이 #636 이슈의 요구사항을 완전히 충족합니다. 멘토 지원서 승인 후 사용자 Role을 MENTOR로 변경하는 기능이 구현되었습니다.
Out of Scope Changes check	✅ Passed	모든 변경사항이 #636 이슈 범위 내에 있으며, 멘토 승인 시 Role 승격 기능 구현과 관련된 필요한 수정만 포함되어 있습니다.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

[whqtker]

고생하셨습니다 ~!

다만 현재는 siteUser의 role 만 MENTOR 로 변경하는 거 같은데, mentor 엔티티 또한 생성해야 합니다 ! 이후 관련하여 테스트(승인하면 멘토 생성)도 추가해주시면 좋겠습니다 !!

[sukangpunch]

고생하셨습니다 ~!

다만 현재는 siteUser의 role 만 MENTOR 로 변경하는 거 같은데, mentor 엔티티 또한 생성해야 합니다 ! 이후 관련하여 테스트(승인하면 멘토 생성)도 추가해주시면 좋겠습니다 !!

알겠습니다!