[FEATURE] CLIPScore 추가 #206
Conversation
📝 WalkthroughSummary by CodeRabbit
Walkthrough
Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Service
participant DTO
participant Entity
participant DB
Client->>Service: 이미지 업로드 요청 (ImageUploadResponseDTO)
Service->>DTO: DTO에 clipScore 포함
Service->>Entity: GenerateImage.createGenerateImage(DTO, House)
Entity->>DB: clipScore 포함하여 저장
DB-->>Entity: 저장된 GenerateImage 반환
Entity-->>Service: GenerateImage 반환
Service-->>Client: 결과 응답 (clipScore 포함)
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ 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
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. 🪧 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
added
🔢 chore
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
src/main/java/or/sopt/houme/domain/generateImage/entity/GenerateImage.java(2 hunks)src/main/java/or/sopt/houme/global/dto/ImageUploadResponseDTO.java(1 hunks)src/test/java/or/sopt/houme/domain/generateImage/facade/GenerateImageFacadeTest.java(1 hunks)src/test/java/or/sopt/houme/domain/generateImage/repository/GenerateImageRepositoryImplTest.java(1 hunks)src/test/java/or/sopt/houme/domain/generateImage/service/GenerateImageServiceImplTest.java(4 hunks)src/test/java/or/sopt/houme/domain/house/repository/HouseCustomRepositoryImplTest.java(3 hunks)src/test/java/or/sopt/houme/domain/taste/repository/TagRepositoryImplTest.java(1 hunks)src/test/java/or/sopt/houme/domain/user/repository/UserRepositoryImplTest.java(1 hunks)
🧰 Additional context used
🧠 Learnings (2)
src/test/java/or/sopt/houme/domain/house/repository/HouseCustomRepositoryImplTest.java (1)
Learnt from: gdbs1107
PR: TEAM-HOUME/HOUME-SERVER#181
File: src/main/java/or/sopt/houme/domain/user/service/UserLandingServiceImpl.java:0-0
Timestamp: 2025-07-16T15:00:04.016Z
Learning: HOUME-SERVER 프로젝트에서 UserLandingServiceImpl의 getHasGeneratedImage API는 리프레시 토큰 유효성 검증이 목적이 아니라 사용자의 이미지 생성 여부를 확인하는 API임. 리프레시 토큰 검증은 별도의 refresh rotate 로직에서 처리하며, 이 API는 토큰이 없거나 유효하지 않은 경우 단순히 true를 반환하는 것이 의도된 동작임.
src/test/java/or/sopt/houme/domain/generateImage/service/GenerateImageServiceImplTest.java (2)
Learnt from: gdbs1107
PR: TEAM-HOUME/HOUME-SERVER#181
File: src/main/java/or/sopt/houme/domain/user/service/UserLandingServiceImpl.java:0-0
Timestamp: 2025-07-16T15:00:04.016Z
Learning: HOUME-SERVER 프로젝트에서 UserLandingServiceImpl의 getHasGeneratedImage API는 리프레시 토큰 유효성 검증이 목적이 아니라 사용자의 이미지 생성 여부를 확인하는 API임. 리프레시 토큰 검증은 별도의 refresh rotate 로직에서 처리하며, 이 API는 토큰이 없거나 유효하지 않은 경우 단순히 true를 반환하는 것이 의도된 동작임.
Learnt from: PBEM22
PR: TEAM-HOUME/HOUME-SERVER#120
File: src/main/java/or/sopt/houme/domain/generateImage/entity/GenerateImage.java:37-46
Timestamp: 2025-07-11T18:45:34.816Z
Learning: HOUME-SERVER 프로젝트에서 OpenAI 이미지 생성 API와 연동할 때, ImageUploadResponseDTO의 contentType 필드는 일반적인 MIME 타입(예: "image/jpeg")이 아닌 실제 파일 확장자(예: "jpg", "png")를 포함한다.
🔇 Additional comments (12)
src/test/java/or/sopt/houme/domain/generateImage/facade/GenerateImageFacadeTest.java (2)
230-230: 파일 끝 줄바꿈 추가 승인파일 끝에 줄바꿈을 추가하는 것은 좋은 관례입니다.
116-118: 테스트에서 clipScore 필드 누락 확인 필요새로 추가된 clipScore 필드가 ImageUploadResponseDTO에 포함되었지만, 이 테스트에서는 해당 필드를 설정하지 않고 있습니다. 다른 테스트 파일들과의 일관성을 위해 clipScore 값을 설정하는 것을 고려해보세요.
Also applies to: 201-203
src/test/java/or/sopt/houme/domain/taste/repository/TagRepositoryImplTest.java (1)
81-81: clipScore 필드 추가 승인새로 추가된 clipScore 필드를 테스트 데이터에 적절히 설정했습니다. 테스트값 0.1234F는 적절합니다.
src/test/java/or/sopt/houme/domain/user/repository/UserRepositoryImplTest.java (1)
85-85: clipScore 필드 추가 승인테스트 데이터 초기화에서 clipScore 필드를 올바르게 설정했습니다. 다른 테스트 파일들과 일관성을 유지하고 있습니다.
src/test/java/or/sopt/houme/domain/generateImage/repository/GenerateImageRepositoryImplTest.java (1)
68-68: clipScore 필드 추가 승인테스트 데이터에서 clipScore 필드를 적절히 설정했습니다. 다른 테스트 파일들과 동일한 값을 사용하여 일관성을 유지하고 있습니다.
src/main/java/or/sopt/houme/global/dto/ImageUploadResponseDTO.java (1)
16-17: clipScore 필드 추가 승인새로운 clipScore 필드가 적절히 추가되었습니다. nullable Float 타입과 @Setter 어노테이션을 사용하여 pullPrompt와 동일한 패턴을 따르고 있습니다.
src/test/java/or/sopt/houme/domain/house/repository/HouseCustomRepositoryImplTest.java (2)
72-72: clipScore 필드 추가가 적절합니다.테스트 데이터 설정에서 clipScore 필드를 0.1234F로 설정한 것이 올바릅니다. 이는 새롭게 추가된 필드에 대한 테스트 데이터 일관성을 보장합니다.
82-82: 테스트 DisplayName에서 이모지 제거가 좋은 개선입니다.테스트 메서드의 DisplayName에서 이모지를 제거한 것은 테스트 리포트의 가독성과 일관성을 향상시키는 좋은 변경입니다.
Also applies to: 94-94
src/main/java/or/sopt/houme/domain/generateImage/entity/GenerateImage.java (2)
33-34: clipScore 필드 정의가 올바릅니다.새로운 clipScore 필드가 적절하게 정의되었습니다. Float 타입과 nullable = false 설정이 CLIP 점수 데이터의 특성에 맞습니다.
47-47: 정적 팩토리 메서드 업데이트가 적절합니다.createGenerateImage 메서드에서 ImageUploadResponseDTO로부터 clipScore를 설정하는 로직이 올바르게 추가되었습니다.
src/test/java/or/sopt/houme/domain/generateImage/service/GenerateImageServiceImplTest.java (2)
46-46: 테스트 데이터 설정 개선이 좋습니다.responseDTO를 인스턴스 변수로 추출하고 setUp() 메서드에서 초기화하는 것은 테스트 코드의 일관성과 재사용성을 향상시킵니다.
Also applies to: 74-80
88-88: 테스트 로직 업데이트가 올바릅니다.createGenerateImage 테스트에서 사전 초기화된 responseDTO를 사용하고 clipScore에 대한 검증을 추가한 것이 적절합니다.
Also applies to: 92-95
src/test/java/or/sopt/houme/domain/generateImage/service/GenerateImageServiceImplTest.java
Show resolved
Hide resolved
…into feat/#197/clip-score # Conflicts: # src/test/java/or/sopt/houme/domain/user/repository/UserRepositoryImplTest.java
📝 Code Coverage
|
Test Results131 tests 131 ✅ 8s ⏱️ Results for commit d025b5e. ♻️ This comment has been updated with latest results. |
📣 Related Issue
📝 Summary
🙏 Question & PR point
📬 Postman