feature/#218 club 이미지 추가 #219
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Walkthrough이번 PR은 동아리 생성 기능에 파일 연관 기능을 추가하는 변경 사항입니다. 주요 변경으로는 동아리 생성 메서드에 Changes
Possibly related PRs
Suggested labels
Suggested reviewers
🪧 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 using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Test Coverage Report
Files
|
Codecov ReportAttention: Patch coverage is
@@ Coverage Diff @@
## develop #219 +/- ##
=============================================
- Coverage 92.23% 91.82% -0.41%
Complexity 148 148
=============================================
Files 49 49
Lines 412 416 +4
Branches 4 5 +1
=============================================
+ Hits 380 382 +2
- Misses 27 28 +1
- Partials 5 6 +1
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Nitpick comments (5)
aics-api/src/main/java/kgu/developers/api/club/presentation/response/ClubDetailResponse.java (1)
22-25: Swagger 문서 예제 개선 필요
file필드에 대한 Swagger 문서가 추가되었습니다. 그러나 예제 값인"/cloud/file/3/2025-curriculum"은 동아리 이미지보다는 교육과정과 관련된 것처럼 보입니다. 동아리 이미지에 맞는 더 적절한 예제를 사용하는 것이 좋겠습니다.@Schema(description = "첨부 파일 정보", - example = "{\"physicalPath\": \"/cloud/file/3/2025-curriculum\"}", + example = "{\"physicalPath\": \"/cloud/file/3/club-logo\"}", requiredMode = NOT_REQUIRED) FilePathResponse fileaics-domain/src/testFixtures/java/club/application/ClubQueryServiceTest.java (1)
53-53: 테스트 내용 업데이트 필요
Club클래스에file필드가 추가되었으므로, 테스트 검증부(then부분)에서 파일 필드의null상태도 함께 검증하는 것이 좋을 것 같습니다.// then assertEquals("Club a", result.getName()); assertEquals("a 동아리입니다.", result.getDescription()); assertEquals("http://club-a.kyonggi.ac.kr", result.getSite()); +assertEquals(null, result.getFile());aics-domain/src/testFixtures/java/club/domain/ClubDomainTest.java (1)
25-35: 파일 연관성 테스트 케이스 추가 필요현재는
null을 전달하는 테스트만 있습니다. 실제FileEntity가 있는 경우의 테스트 케이스도 추가하면 코드 커버리지가 향상될 것입니다.파일 엔티티를 전달하는 테스트 케이스를 추가하는 것이 좋겠습니다:
@Test @DisplayName("Club 객체를 파일과 함께 생성할 수 있다") public void createClubWithFile_Success() { // given FileEntity mockFile = mock(FileEntity.class); // when Club newClub = Club.create(NAME, DESCRIPTION, SITE, mockFile); // then assertNotNull(newClub); assertEquals(NAME, newClub.getName()); assertEquals(DESCRIPTION, newClub.getDescription()); assertEquals(SITE, newClub.getSite()); assertEquals(mockFile, newClub.getFile()); }aics-admin/src/main/java/kgu/developers/admin/file/presentation/FileAdminControllerImpl.java (1)
66-73: 파일 유효성 검사 고려 필요현재 구현에는 파일 유효성 검사가 없습니다. 이미지 파일만 허용하거나 특정 크기로 제한하는 등의 유효성 검사를 추가하는 것이 좋을 것 같습니다.
파일 유효성 검사 로직을 추가하는 것을 고려해보세요:
@Override @PostMapping(value = "/club", consumes = MULTIPART_FORM_DATA_VALUE) public ResponseEntity<FilePathResponse> clubFileUpload( @RequestPart(value = "file") MultipartFile file ) { // 파일 유효성 검사 if (file.isEmpty()) { throw new IllegalArgumentException("파일이 비어 있습니다"); } // 이미지 파일 형식 검사 String contentType = file.getContentType(); if (contentType == null || !contentType.startsWith("image/")) { throw new IllegalArgumentException("이미지 파일만 업로드 가능합니다"); } FilePathResponse path = fileAdminFacade.saveFile(file, CLUB); return ResponseEntity.status(CREATED).body(path); }aics-admin/src/testFixtures/java/club/application/ClubAdminFacadeTest.java (1)
54-58: 파일 연결에 대한 테스트 검증이 부족합니다.테스트가 동아리 생성은 확인하지만, 생성된 동아리에 파일이 실제로 연결되었는지 검증하는 로직이 누락되었습니다. 파일 연결이 제대로 이루어졌는지 확인하는 assertion을 추가하는 것이 좋겠습니다.
다음과 같이 파일 연결을 검증하는 assertion을 추가해보세요:
// then assertEquals(2, resultData.size()); assertEquals(2L, result.id()); +Club savedClub = fakeClubRepository.findById(2L).get(); +assertEquals(fileId, savedClub.getFile() != null ? savedClub.getFile().getId() : null);
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (18)
aics-admin/src/main/java/kgu/developers/admin/club/application/ClubAdminFacade.java(1 hunks)aics-admin/src/testFixtures/java/club/application/ClubAdminFacadeTest.java(2 hunks)aics-domain/src/main/java/kgu/developers/domain/club/application/command/ClubCommandService.java(1 hunks)aics-domain/src/main/java/kgu/developers/domain/club/domain/Club.java(2 hunks)aics-domain/src/testFixtures/java/mock/FakeTestContainer.java(2 hunks)aics-domain/src/testFixtures/java/mock/TestContainer.java(1 hunks)aics-admin/src/main/java/kgu/developers/admin/club/presentation/ClubAdminController.java(2 hunks)aics-admin/src/main/java/kgu/developers/admin/club/presentation/ClubAdminControllerImpl.java(2 hunks)aics-admin/src/main/java/kgu/developers/admin/file/presentation/FileAdminController.java(2 hunks)aics-admin/src/main/java/kgu/developers/admin/file/presentation/FileAdminControllerImpl.java(3 hunks)aics-domain/src/main/java/kgu/developers/domain/file/domain/FileDomain.java(1 hunks)aics-admin/src/main/java/kgu/developers/admin/club/presentation/ClubAdminControllerImpl.java(1 hunks)aics-api/src/main/java/kgu/developers/api/club/presentation/response/ClubDetailResponse.java(2 hunks)aics-domain/src/testFixtures/java/club/application/ClubCommandServiceTest.java(3 hunks)aics-domain/src/testFixtures/java/club/application/ClubQueryServiceTest.java(1 hunks)aics-domain/src/testFixtures/java/club/domain/ClubDomainTest.java(1 hunks)aics-api/src/main/resources/db/data.sql(2 hunks)aics-api/src/main/resources/db/schema.sql(1 hunks)
🧰 Additional context used
🪛 GitHub Check: codecov/patch
aics-domain/src/main/java/kgu/developers/domain/club/application/command/ClubCommandService.java
[warning] 22-22: aics-domain/src/main/java/kgu/developers/domain/club/application/command/ClubCommandService.java#L22
Added line #L22 was not covered by tests
🔇 Additional comments (30)
aics-domain/src/main/java/kgu/developers/domain/file/domain/FileDomain.java (1)
13-13: 변경사항이 적절합니다
CLUB항목을FileDomain열거형에 추가한 것은 요구사항에 잘 부합합니다. 기존 패턴을 따라 올바르게 구현되었습니다.aics-api/src/main/java/kgu/developers/api/club/presentation/response/ClubDetailResponse.java (3)
8-8: 적절한 import 추가필요한
FilePathResponse클래스를 정확히 import 했습니다.
20-21: 문법적 수정 적절함코드 구조를 유지하기 위해 필요한 문법적 변경(콤마 추가)이 적절하게 이루어졌습니다.
32-33: null 처리가 잘 구현됨
Club객체에서 파일이 존재하지 않는 경우(null)에 대한 처리가 적절하게 구현되었습니다.aics-domain/src/testFixtures/java/club/application/ClubQueryServiceTest.java (2)
24-24: 테스트 코드 업데이트 적절함
Club.create메소드 호출을 변경된 시그니처에 맞게 업데이트했습니다. 현재는 파일을 사용하지 않는 테스트이므로null을 전달하는 것이 적절합니다.
28-28: 테스트 코드 업데이트 적절함두 번째
Club.create메소드 호출도 시그니처 변경에 맞게 적절히 업데이트되었습니다.aics-domain/src/testFixtures/java/mock/TestContainer.java (1)
126-130: 클럽 관련 서비스 초기화 리팩터링
clubRepository,clubQueryService및clubCommandService의 초기화가 이제 파일 후반부로 이동되었고,clubCommandService는 이제fileQueryService를 의존성으로 추가했습니다. 이는 클럽 이미지 기능 추가에 필요한 변경 사항으로 보입니다.aics-admin/src/main/java/kgu/developers/admin/club/application/ClubAdminFacade.java (1)
20-21: 클럽 생성 메서드에 파일 연관 기능 추가클럽 생성 메서드에
fileId매개변수가 추가되어 이미지를 클럽에 연결할 수 있게 되었습니다. 이는 PR 목표에 맞게 클럽 엔티티에 이미지 기능을 추가하는 변경입니다.aics-domain/src/testFixtures/java/club/domain/ClubDomainTest.java (2)
20-20: Club.create 메서드에 추가된 파일 매개변수에 null 전달테스트 설정에서
Club.create메서드 호출 시 파일 매개변수로null을 전달하고 있습니다. 이는 변경된 메서드 시그니처에 맞게 수정된 것입니다.
28-28: 테스트 케이스에서 Club.create 메서드 호출 업데이트
createClub_Success테스트 케이스에서도Club.create메서드 호출 시 파일 매개변수로null을 전달하도록 수정되었습니다.aics-admin/src/main/java/kgu/developers/admin/file/presentation/FileAdminControllerImpl.java (2)
3-10: 정적 임포트 및 패키지 구성 변경파일 도메인 상수를 정적 임포트하고 패키지 구성을 정리했습니다. 이렇게 하면 코드가 더 간결해지고 가독성이 향상됩니다.
66-73: 클럽 파일 업로드 API 추가클럽 이미지 업로드를 위한 새로운 엔드포인트가 추가되었습니다. 이 구현은 다른 파일 업로드 엔드포인트와 일관된 패턴을 따르고 있어 좋습니다.
aics-admin/src/testFixtures/java/club/application/ClubAdminFacadeTest.java (6)
19-21: 새로운 의존성 추가가 적절히 이루어졌습니다.파일 관련 기능 추가에 필요한 import들이 잘 추가되었습니다.
29-31: 테스트 초기화에 파일 저장소 설정이 적절히 추가되었습니다.FakeFileRepository와 FileQueryService를 생성하여 테스트 환경을 적절히 구성했습니다.
34-35: ClubCommandService 의존성 주입에 fileQueryService가 추가되었습니다.ClubCommandService 생성자에 필요한 파라미터가 올바르게 전달되고 있습니다.
40-41: Club.create 메서드에 파일 파라미터 추가가 적절합니다.Club 엔티티 생성 시 null을 전달하여 파일 없이도 생성이 가능하도록 했습니다.
54-54: 테스트용 fileId 설정이 적절합니다.테스트에서 사용할 fileId 값을 적절히 설정했습니다.
57-57: createClub 메서드 호출에 fileId 파라미터가 추가되었습니다.변경된 메서드 시그니처에 맞게 fileId를 전달하도록 테스트 코드가 수정되었습니다.
aics-admin/src/main/java/kgu/developers/admin/club/presentation/ClubAdminControllerImpl.java (1)
36-36: fileId 파라미터가 적절히 전달되었습니다.ClubAdminFacade.createClub 호출 시 fileId 파라미터가 올바르게 전달되고 있습니다.
aics-admin/src/main/java/kgu/developers/admin/file/presentation/FileAdminController.java (2)
3-8: import 문 정리가 잘 되었습니다.정적 import와 일반 import를 적절히 구분하여 코드 가독성이 향상되었습니다.
80-93: 동아리 로고 이미지 업로드 API가 적절히 추가되었습니다.동아리 로고 이미지 업로드를 위한 API가 기존 패턴과 일관되게 잘 구현되었습니다. API 설명과 파라미터 설명이 명확합니다.
aics-admin/src/main/java/kgu/developers/admin/club/presentation/ClubAdminController.java (2)
6-6: 필요한 import가 적절히 추가되었습니다.RequestParam 어노테이션 사용을 위한 import가 잘 추가되었습니다.
30-33: fileId 파라미터에 대한 문서화가 잘 되어있습니다.fileId 파라미터에 대한 설명과 예시가
명확히 작성되어 있습니다. required=false로 설정하여 파일 없이도 동아리 생성이 가능하도록 한 것이 좋습니다.aics-domain/src/main/java/kgu/developers/domain/club/domain/Club.java (2)
38-40: 파일 엔티티와의 관계가 적절히 설정되었습니다.
Club엔티티와FileEntity간의 일대일 관계가 적절하게 설정되었습니다.@JoinColumn어노테이션을 사용하여 관계의 외래 키를 명확하게 지정한 점이 좋습니다.
42-48: 클럽 생성 메서드가 파일 엔티티를 포함하도록 적절히 수정되었습니다.
create메서드에 파일 파라미터를 추가하고 빌더 패턴에도 이를 반영하여 도메인 모델의 일관성을 유지했습니다.aics-domain/src/testFixtures/java/mock/FakeTestContainer.java (1)
59-61: ClubCommandService 초기화가 적절히 수정되었습니다.
ClubCommandService의 생성자 매개변수에FileQueryService를 추가하여 실제 구현과 테스트 환경이 일치하도록 수정한 것이 좋습니다.aics-domain/src/main/java/kgu/developers/domain/club/application/command/ClubCommandService.java (1)
17-17: FileQueryService 의존성이 추가되었습니다.클럽 생성 시 파일 처리를 위해 필요한 의존성이 적절하게 추가되었습니다.
aics-domain/src/testFixtures/java/club/application/ClubCommandServiceTest.java (1)
22-25: FileQueryService 테스트 설정이 적절히 추가되었습니다.테스트 환경에
FakeFileRepository와FileQueryService를 초기화하여ClubCommandService의 새로운 의존성을 올바르게 처리했습니다.aics-api/src/main/resources/db/data.sql (2)
53-57: 새로운 동아리 로고 이미지 데이터 추가 검토
이 구간에서는 CLUB 관련 로고 이미지에 대한 새 INSERT 구문이 추가되었습니다.
- 추가된 항목들이 file_entity 테이블에 올바르게 삽입되고 있으며, 이후 클럽 테이블과의 매핑을 위해 순서대로 배정된 file_id 값(21~24)과 일치하는지 확인하시기 바랍니다.
72-78: 동아리 INSERT 문에 file_id 컬럼 추가 검토
이 부분에서는 클럽 테이블의 INSERT 구문에 새롭게 file_id 컬럼이 포함되어 클럽과 파일 엔티티 사이의 1:1 관계를 반영하고 있습니다.
- 각 클럽 항목에 할당된 file_id(21, 22, 23, 24)가 file_entity 테이블 내 클럽 로고 데이터 순번과 올바르게 매칭되는지 다시 한 번 검증하시기 바랍니다.
...-domain/src/main/java/kgu/developers/domain/club/application/command/ClubCommandService.java
Show resolved
Hide resolved
aics-domain/src/testFixtures/java/club/application/ClubCommandServiceTest.java
Show resolved
Hide resolved
minjo-on
added
✨feature
Summary
club 이미지 추가
Tasks
Screenshot