-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: 멘토 권한 부여 로직 추가 #712
feat: 멘토 권한 부여 로직 추가 #712
Conversation
Walkthrough
Changes
Assessment against linked issues
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/main/java/com/gdschongik/gdsc/domain/study/application/AdminStudyService.java (1 hunks)
Additional comments not posted (3)
src/main/java/com/gdschongik/gdsc/domain/study/application/AdminStudyService.java (3)
33-33
: 멘토 객체 가져오기
getMemberById
메서드를 호출하여 멘토의Member
객체를 가져옵니다. 이 부분은 멘토 권한 부여 로직의 첫 단계로 적절합니다.코드 변경 사항이 승인되었습니다.
34-34
: 멘토 권한 부여
mentor.assignToMentor()
메서드를 호출하여 멘토에게 명시적으로 권한을 부여합니다. 이 부분은 멘토 권한 부여 로직의 핵심입니다.코드 변경 사항이 승인되었습니다.
35-35
: 멘토 상태 저장
memberRepository.save(mentor)
를 호출하여 멘토의 상태 변경 사항을 데이터베이스에 저장합니다. 이 부분은 데이터의 무결성을 보장합니다.코드 변경 사항이 승인되었습니다.
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- src/test/java/com/gdschongik/gdsc/domain/study/application/AdminStudyServiceTest.java (1 hunks)
Additional comments not posted (5)
src/test/java/com/gdschongik/gdsc/domain/study/application/AdminStudyServiceTest.java (5)
1-2
: 패키지 선언 확인패키지 선언이 올바르게 되어 있습니다.
패키지 선언은 적절합니다.
3-13
: 적절한 임포트임포트된 클래스와 상수들이 테스트에 필요합니다.
임포트는 적절합니다.
14-18
: 클래스 설정 확인클래스 설정이 적절하며,
IntegrationTest
를 확장하고AdminStudyService
를 주입받습니다.클래스 설정은 적절합니다.
19-21
: 중첩 클래스 설정 확인중첩 클래스 설정이 적절하며, 관련된 테스트를 조직화하는 데 유용합니다.
중첩 클래스 설정은 적절합니다.
22-46
: 테스트 메서드 확인테스트 메서드가 잘 구조화되어 있으며, 멘토 할당 로직을 검증합니다.
테스트 메서드는 적절합니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit