[feat] 팔로잉 조회 여부 api 개발#106
Conversation
|
""" Walkthrough팔로우 관계의 삭제 방식이 소프트 딜리트에서 실제 삭제로 변경되었습니다. 또한, 사용자가 특정 사용자를 팔로우하고 있는지 확인하는 새로운 API 엔드포인트와 관련 서비스, 응답 객체, 테스트가 추가되었습니다. 기존 팔로우 관련 메서드명과 인터페이스도 이에 맞게 조정되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant UserQueryController
participant UserIsFollowingService
participant FollowingCommandPort
Client->>UserQueryController: GET /users/{targetUserId}/is-following (userId)
UserQueryController->>UserIsFollowingService: isFollowing(userId, targetUserId)
UserIsFollowingService->>FollowingCommandPort: findByUserIdAndTargetUserId(userId, targetUserId)
FollowingCommandPort-->>UserIsFollowingService: Following 존재 여부 반환
UserIsFollowingService-->>UserQueryController: UserIsFollowingResponse(isFollowing)
UserQueryController-->>Client: BaseResponse<UserIsFollowingResponse>
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15–20 minutes Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🧹 Nitpick comments (3)
src/main/java/konkuk/thip/user/adapter/out/persistence/FollowingCommandPersistenceAdapter.java (1)
63-63: 메서드명이 실제 동작과 불일치
updateUserFollowerCount메서드가 이제updateFrom(targetUser)를 호출하여 팔로워 수 외에도 다른 필드를 업데이트할 수 있습니다. 메서드명이 실제 동작을 정확히 반영하지 못합니다.메서드명을 다음과 같이 변경하는 것을 고려해보세요:
-private UserJpaEntity updateUserFollowerCount(User targetUser) { +private UserJpaEntity updateTargetUser(User targetUser) {src/main/java/konkuk/thip/user/adapter/in/web/UserQueryController.java (1)
58-65: 팔로잉 여부 조회 엔드포인트가 올바르게 구현됨새로운 GET 엔드포인트가 적절한 경로 패턴과 파라미터를 사용하여 구현되었습니다.
@UserId와@PathVariable사용이 적절합니다.하지만 메서드명에 오타가 있습니다:
-public BaseResponse<UserIsFollowingRespone> checkisFollowing(@UserId final Long userId, +public BaseResponse<UserIsFollowingRespone> checkIsFollowing(@UserId final Long userId,camelCase 규칙에 따라
checkIsFollowing으로 수정해야 합니다.src/test/java/konkuk/thip/user/adapter/in/web/UserIsFollowingApiTest.java (1)
60-63: 테스트 데이터 생성 방식을 개선할 수 있습니다.현재
FollowingJpaEntity를 직접 빌더로 생성하고 있는데, 다른 엔티티들처럼TestEntityFactory를 사용하는 것이 일관성 측면에서 더 좋을 것 같습니다.다음과 같이 개선할 수 있습니다:
- // 팔로잉 관계 저장 - followingJpaRepository.save(FollowingJpaEntity.builder() - .userJpaEntity(user) - .followingUserJpaEntity(target) - .build()); + // 팔로잉 관계 저장 + followingJpaRepository.save(TestEntityFactory.createFollowing(user, target));단, 이는
TestEntityFactory에createFollowing메서드가 존재하는 경우에만 적용 가능합니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (13)
src/main/java/konkuk/thip/user/adapter/in/web/UserQueryController.java(3 hunks)src/main/java/konkuk/thip/user/adapter/in/web/response/UserIsFollowingRespone.java(1 hunks)src/main/java/konkuk/thip/user/adapter/out/jpa/FollowingJpaEntity.java(0 hunks)src/main/java/konkuk/thip/user/adapter/out/jpa/UserJpaEntity.java(2 hunks)src/main/java/konkuk/thip/user/adapter/out/persistence/FollowingCommandPersistenceAdapter.java(3 hunks)src/main/java/konkuk/thip/user/adapter/out/persistence/repository/following/FollowingQueryRepositoryImpl.java(1 hunks)src/main/java/konkuk/thip/user/application/port/in/UserIsFollowingUsecase.java(1 hunks)src/main/java/konkuk/thip/user/application/port/out/FollowingCommandPort.java(2 hunks)src/main/java/konkuk/thip/user/application/service/UserIsFollowingService.java(1 hunks)src/main/java/konkuk/thip/user/application/service/following/UserFollowService.java(2 hunks)src/test/java/konkuk/thip/user/adapter/in/web/UserFollowApiTest.java(4 hunks)src/test/java/konkuk/thip/user/adapter/in/web/UserIsFollowingApiTest.java(1 hunks)src/test/java/konkuk/thip/user/application/service/UserFollowServiceTest.java(2 hunks)
💤 Files with no reviewable changes (1)
- src/main/java/konkuk/thip/user/adapter/out/jpa/FollowingJpaEntity.java
🧰 Additional context used
🧠 Learnings (2)
src/main/java/konkuk/thip/user/application/port/out/FollowingCommandPort.java (1)
Learnt from: seongjunnoh
PR: #43
File: src/main/java/konkuk/thip/book/application/port/out/BookCommandPort.java:0-0
Timestamp: 2025-07-03T03:05:05.031Z
Learning: THIP 프로젝트에서는 CQRS Port 분리 시 다음 컨벤션을 따름: CommandPort에는 findByXXX를 통해 도메인 엔티티를 찾아오는 메서드를 추가하고, QueryPort에는 조회 API의 response에 해당하는 데이터들을 DB로부터 조회하는 메서드를 추가함.
src/main/java/konkuk/thip/user/adapter/out/persistence/FollowingCommandPersistenceAdapter.java (2)
Learnt from: seongjunnoh
PR: #43
File: src/main/java/konkuk/thip/book/application/port/out/BookCommandPort.java:0-0
Timestamp: 2025-07-03T03:05:05.031Z
Learning: THIP 프로젝트에서는 CQRS Port 분리 시 다음 컨벤션을 따름: CommandPort에는 findByXXX를 통해 도메인 엔티티를 찾아오는 메서드를 추가하고, QueryPort에는 조회 API의 response에 해당하는 데이터들을 DB로부터 조회하는 메서드를 추가함.
Learnt from: seongjunnoh
PR: #36
File: src/main/java/konkuk/thip/user/adapter/out/persistence/UserJpaRepository.java:7-7
Timestamp: 2025-06-29T09:47:31.299Z
Learning: Spring Data JPA에서 findBy{FieldName} 패턴의 메서드는 명시적 선언 없이 자동으로 생성되며, Optional 반환 타입을 사용하는 것이 null 안전성을 위해 권장됩니다.
🔇 Additional comments (17)
src/main/java/konkuk/thip/user/adapter/out/jpa/UserJpaEntity.java (1)
6-6: 도메인 객체 import 추가 확인.
User도메인 객체를 import하여 새로운updateFrom메서드에서 활용하는 것이 적절합니다.src/main/java/konkuk/thip/user/adapter/out/persistence/repository/following/FollowingQueryRepositoryImpl.java (1)
32-33: ACTIVE 상태 필터링 추가가 적절합니다.팔로잉 관계 조회 시
StatusType.ACTIVE조건을 추가하여 활성화된 팔로잉만 반환하도록 수정한 것이 올바릅니다. 이는 soft delete에서 실제 삭제로 변경된 전체적인 리팩토링과 일치하며, 새로운 "팔로잉 여부 확인" API의 정확한 동작을 보장합니다.src/main/java/konkuk/thip/user/application/service/following/UserFollowService.java (1)
42-42: soft delete에서 실제 삭제로 변경이 적절합니다.
updateStatus대신deleteFollowing를 호출하도록 변경한 것이 PR 목표와 일치합니다. soft delete 방식을 제거하고 실제 삭제를 수행하는 것이 올바른 접근입니다.src/test/java/konkuk/thip/user/application/service/UserFollowServiceTest.java (1)
69-69: 테스트 검증 로직 업데이트가 적절합니다.서비스 구현체의 변경사항(
updateStatus→deleteFollowing)에 맞춰 테스트의 검증 로직을 올바르게 업데이트했습니다. 테스트가 실제 서비스 동작을 정확히 검증하도록 보장합니다.Also applies to: 133-133
src/main/java/konkuk/thip/user/application/port/in/UserIsFollowingUsecase.java (1)
5-7: 인터페이스 구현이 올바르게 정의됨팔로잉 여부 확인을 위한 유스케이스 인터페이스가 명확하게 정의되어 있습니다.
src/test/java/konkuk/thip/user/adapter/in/web/UserFollowApiTest.java (2)
54-54: 테스트 표시명이 새로운 동작을 정확히 반영소프트 삭제에서 실제 삭제로 변경된 비즈니스 로직에 맞게 테스트 표시명이 적절히 업데이트되었습니다.
98-100: 엔티티 삭제 검증 로직이 올바르게 구현됨이전의 상태 변경 확인에서 실제 엔티티 삭제 확인으로 변경된 검증 로직이 정확합니다.
Optional.isPresent()를 사용한 존재 여부 확인이 적절합니다.src/main/java/konkuk/thip/user/application/service/UserIsFollowingService.java (1)
16-20: 팔로잉 확인 로직이 올바르게 구현됨
Optional.isPresent()를 사용하여 팔로잉 관계 존재 여부를 확인하는 로직이 정확하고 간결합니다.src/main/java/konkuk/thip/user/adapter/out/persistence/FollowingCommandPersistenceAdapter.java (1)
32-32: ACTIVE 팔로잉만 조회한다는 주석이 명확함소프트 삭제 제거로 인해 ACTIVE 상태만 조회한다는 것을 명시한 주석이 코드 이해에 도움이 됩니다.
src/main/java/konkuk/thip/user/adapter/in/web/UserQueryController.java (1)
9-9: 필요한 의존성들이 적절히 추가됨새로운 기능을 위한 응답 클래스와 유스케이스 인터페이스 import가 올바르게 추가되었습니다.
Also applies to: 12-12, 26-26
src/main/java/konkuk/thip/user/application/port/out/FollowingCommandPort.java (3)
3-4: 새로운 import 문이 적절히 추가되었습니다.
EntityNotFoundException과ErrorCode를 사용하는 새로운 메서드를 위해 필요한 import가 올바르게 추가되었습니다.
14-17: 기본 메서드 구현이 올바르고 일관성 있습니다.
getByUserIdAndTargetUserIdOrThrow메서드는 Optional을 반환하는 메서드의 throwing 변형으로 일반적인 패턴을 따르고 있습니다. 에러 코드도 적절하게 사용되었습니다.
21-21: 메서드명 변경이 PR 목적과 잘 일치합니다.
updateStatus에서deleteFollowing으로 메서드명을 변경한 것은 soft delete 제거 정책과 일치하며, 실제 동작을 더 명확하게 표현합니다.src/test/java/konkuk/thip/user/adapter/in/web/UserIsFollowingApiTest.java (4)
24-28: 테스트 클래스 설정이 적절합니다.Spring Boot 통합 테스트를 위한 어노테이션들이 올바르게 설정되었고, 한글 DisplayName으로 테스트 목적이 명확하게 표현되었습니다.
42-47: 테스트 격리를 위한 cleanup이 올바르게 구현되었습니다.외래키 관계를 고려하여
followingJpaRepository.deleteAllInBatch()먼저 실행하고, 이후 다른 엔티티들을 삭제하는 순서가 적절합니다.
49-71: 팔로잉 관계 존재 시나리오 테스트가 잘 구현되었습니다.테스트 데이터 설정, API 호출, 응답 검증이 모두 적절하게 작성되었습니다. JSON path를 통한 응답 검증도 정확합니다.
73-89: 팔로잉 관계 없는 시나리오 테스트가 적절합니다.팔로잉 관계가 존재하지 않는 경우의 테스트도 올바르게 구현되어 있어 API의 두 가지 주요 시나리오를 모두 커버합니다.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
src/main/java/konkuk/thip/user/application/service/UserIsFollowingService.java (1)
15-15: primitive boolean 타입 사용을 고려해보세요.성능상 Boolean 래퍼 타입보다는 primitive boolean을 사용하는 것이 좋습니다. null을 반환할 필요가 없다면 primitive 타입을 권장합니다.
-public Boolean isFollowing(Long userId, Long targetUserId) { +public boolean isFollowing(Long userId, Long targetUserId) {src/main/java/konkuk/thip/user/application/port/in/UserIsFollowingUsecase.java (1)
4-4: primitive boolean 타입 사용을 권장합니다.유스케이스 인터페이스에서는 일반적으로 primitive 타입을 사용하는 것이 좋습니다. Boolean 래퍼 타입보다는 boolean을 사용해주세요.
-Boolean isFollowing(Long userId, Long targetUserId); +boolean isFollowing(Long userId, Long targetUserId);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
src/main/java/konkuk/thip/user/adapter/in/web/UserQueryController.java(3 hunks)src/main/java/konkuk/thip/user/adapter/in/web/response/UserIsFollowingResponse.java(1 hunks)src/main/java/konkuk/thip/user/application/port/in/UserIsFollowingUsecase.java(1 hunks)src/main/java/konkuk/thip/user/application/service/UserIsFollowingService.java(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
- src/main/java/konkuk/thip/user/adapter/in/web/UserQueryController.java
- src/main/java/konkuk/thip/user/adapter/in/web/response/UserIsFollowingResponse.java
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build
🔇 Additional comments (2)
src/main/java/konkuk/thip/user/application/service/UserIsFollowingService.java (1)
1-19: 아키텍처 문제가 해결되었습니다.이전 리뷰에서 지적된 웹 어댑터 응답 객체에 대한 의존성 문제가 해결되었습니다. 서비스가 이제 올바르게 Boolean을 반환하고 있습니다.
src/main/java/konkuk/thip/user/application/port/in/UserIsFollowingUsecase.java (1)
1-5: Clean Architecture 원칙 위반 문제가 해결되었습니다.이전 리뷰에서 지적된 웹 어댑터 응답 객체에 대한 의존성 문제가 해결되었습니다. 유스케이스 인터페이스가 이제 올바르게 도메인 레이어의 관심사만 다루고 있습니다.
| Optional<Following> findByUserIdAndTargetUserId(Long userId, Long targetUserId); | ||
|
|
||
| default Following getByUserIdAndTargetUserIdOrThrow(Long userId, Long targetUserId) { | ||
| return findByUserIdAndTargetUserId(userId, targetUserId) | ||
| .orElseThrow(() -> new EntityNotFoundException(ErrorCode.FOLLOW_NOT_FOUND)); | ||
| } |
3 participants
#️⃣ 연관된 이슈
📝 작업 내용
팔로잉 조회 여부 api 개발했습니다.
추가적으로, 이전 pr #76 에서 팔로잉 상태 변경 시에 소프트 딜리트를 적용했었는데, 회의에서 결정한 대로 팔로잉에는 소프트 딜리트를 해제하고 이에 따른 서비스 로직도 변경했습니다.
📸 스크린샷
💬 리뷰 요구사항
📌 PR 진행 시 이러한 점들을 참고해 주세요
Summary by CodeRabbit
신규 기능
버그 수정
테스트
리팩터