refactor: 이미지 리스트 업로드 롤백 기능 추가

woowacourse-teams · Nov 22, 2023 · c53bec6 · c53bec6
1 parent c9d0b50
commit c53bec6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/backend/backend-submodule b/backend/backend-submodule
diff --git a/backend/src/main/java/hanglog/trip/service/ImageService.java b/backend/src/main/java/hanglog/trip/service/ImageService.java
@@ -1,7 +1,6 @@
 package hanglog.trip.service;
 
-import static hanglog.global.exception.ExceptionCode.EMPTY_IMAGE_LIST;
-import static hanglog.global.exception.ExceptionCode.EXCEED_IMAGE_LIST_SIZE;
+import static hanglog.global.exception.ExceptionCode.*;
 
 import hanglog.global.exception.ImageException;
 import hanglog.image.domain.ImageFile;
@@ -35,7 +34,11 @@ public ImagesResponse save(final List<MultipartFile> images) {
 
     private List<String> uploadImages(final List<ImageFile> imageFiles) {
         try {
-            return imageUploader.uploadImages(imageFiles);
+            final List<String> uploadedImageNames = imageUploader.uploadImages(imageFiles);
+            if(uploadedImageNames.size() != imageFiles.size()) {
+                throw new ImageException(INVALID_IMAGE_PATH);
+            }
+            return uploadedImageNames;
         } catch (final ImageException e) {
             imageFiles.forEach(imageFile -> publisher.publishEvent(new S3ImageEvent(imageFile.getHashedName())));
             throw e;