Commit 7ef425a
authored
feat: 멘토링 관련 도메인 추가 (#362)
* feat: 멘토링 신청 구현
* feat: 멘토링 요청 목록 조회, 멘토링 수락/거절 구현
* feat: 멘토링 확인 구현
* feat: 멘토링 신규 신청 건수 조회 구현
* chore: 전반적으로 로직 수정
- 통일성 있게 .ok(response) 사용
- 단일 객체는 단수형(response) 사용
* chore: 불필요한 개행 제거
* chore: rebase 시 반영되지 않는 로직 추가
* chore: 멘토 관련 로직에 권한 부여
* test: 멘토 관련 픽스처 생성
* test: 멘토링 조회 서비스 테스트 구현
* fix: 이미 멘토인 사용자 검증 로직 수정
- 파라미터는 siteUserId이므로 이를 통해 멘토 존재 여부를 구해야 함.
* chore: 멘토가 자기 자신에 대해 멘토링을 신청하는 경우 검증 삭제
- 멘토링을 신청하는 사용자가 멘토인지 여부로 검증 가능
* chore: 미사용 테스트 메서드 삭제
* test: 멘토링 CUD 테스트 구현
* refactor: List 자체를 Response로 리턴하도록 수정
* refactor: Long -> long으로 변경
* refactor: 개행 추가
* refactor: 처음 개행 추가, Long -> long
* refactor: 메서드 간 개행 추가
* chore: 불필요한 테스트 코드 제거
* refactor: 테스트 코드 리팩터링
- import 문 개행 추가
- 테스트 이름 컨벤션 준수하도록 변경
* refactor: 컨벤션에 맞게 코드 수정
- private 함수는 마지막으로 사용되는 public 함수 아래에 작성한다.
- Long -> long
* refactor: 비즈니스 로직 흐름에 맞게 순서 변경
* chore: 코드 컨벤션 따르도록 수정
- DTO 첫 개행 제거
- import 문 개행 수정
* refactor: 멘토 여부 검증을 서비스 말고 컨트롤러에서 처리하도록 수정
- RequireRoleAccess 어노테이션을 사용하여 멘티 여부 검증
* chore: 컨벤션 관련 수정
- Long -> long
- 의미에 맞도록 서비스 계층의 validate 함수명 변경
* refactor: API 명세에 맞게 응답 수정
* chore: 오타 수정
* refactor: 빌더 패턴 제거
* chore: validateMentoringOwnership 메서드 바디 수정
- 주석의 내용을 잘 대변하도록 변경
* chore: 코딩 컨벤션에 맞도록 수정
- 파라미터 3개인 경우 개행하지 않는다
- 적절한 이름으로 컨트롤러 메서드명 수정
* chore: 코딩 컨벤션에 맞도록 수정
- 적절한 이름으로 컨트롤러 메서드명 수정
* chore: 코딩 컨벤션에 맞도록 수정
- 불필요한 개행 제거1 parent fe7aefc commit 7ef425a
File tree
22 files changed
+957
-0
lines changed- src
- main/java/com/example/solidconnection
- common/exception
- mentor
- controller
- domain
- dto
- repository
- service
- test/java/com/example/solidconnection/mentor
- fixture
- service
22 files changed
+957
-0
lines changedLines changed: 7 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
104 | 104 | | |
105 | 105 | | |
106 | 106 | | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
107 | 114 | | |
108 | 115 | | |
109 | 116 | | |
| |||
Lines changed: 83 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
48 | 52 | | |
Lines changed: 20 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
59 | 65 | | |
60 | 66 | | |
61 | 67 | | |
62 | 68 | | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
63 | 83 | | |
Lines changed: 9 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
Lines changed: 10 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
0 commit comments