ASAP-456 비회원 이미지 업로드 로직 추가 및 테스트 보완

- `userId`를 nullable로 변경하여 비회원 이미지 업로드 지원.
- 비회원 업로드 시 기본 ID로 "anonymous" 설정.
- `ImageCommandServiceTest` 및 Controller 테스트에 관련 시나리오 추가.
- API 문서 변경: 액세스 토큰 필드를 선택사항으로 수정하고 설명 추가.

Author: tlarbals824

Application-Module/src/main/kotlin/com/asap/application/image/port/in/UploadImageUsecase.kt

@@ -9,10 +9,10 @@ interface UploadImageUsecase {
 
     data class Command(
         val image: FileMetaData,
-        val userId: String
+        val userId: String? = null
     )
 
     data class Response(
         val imageUrl: String
     )
-}
+}

Application-Module/src/main/kotlin/com/asap/application/image/service/ImageCommandService.kt

@@ -10,19 +10,32 @@ import org.springframework.stereotype.Service
 @Service
 class ImageCommandService(
     private val imageManagementPort: ImageManagementPort,
-    private val userManagementPort: UserManagementPort
+    private val userManagementPort: UserManagementPort,
 ) : UploadImageUsecase {
+    companion object {
+        private const val ANONYMOUS_OWNER_ID = "anonymous"
+    }
+
     override fun upload(command: UploadImageUsecase.Command): UploadImageUsecase.Response {
-        val user = userManagementPort.getUserNotNull(DomainId(command.userId))
+        val owner =
+            when {
+                command.userId != null -> {
+                    val user = userManagementPort.getUserNotNull(DomainId(command.userId))
+                    user.id.value
+                }
 
-        val uploadedImage = imageManagementPort.save(
-            ImageMetadata(
-                owner = user.id.value,
-                fileMetaData = command.image
+                else -> ANONYMOUS_OWNER_ID
+            }
+
+        val uploadedImage =
+            imageManagementPort.save(
+                ImageMetadata(
+                    owner = owner,
+                    fileMetaData = command.image,
+                ),
             )
-        )
         return UploadImageUsecase.Response(
-            imageUrl = uploadedImage.imageUrl
+            imageUrl = uploadedImage.imageUrl,
         )
     }
-}
+}

Application-Module/src/test/kotlin/com/asap/application/image/service/ImageCommandServiceTest.kt

@@ -2,14 +2,18 @@ package com.asap.application.image.service
 
 import com.asap.application.image.port.`in`.UploadImageUsecase
 import com.asap.application.image.port.out.ImageManagementPort
+import com.asap.application.image.vo.ImageMetadata
 import com.asap.application.image.vo.UploadedImage
 import com.asap.application.user.port.out.UserManagementPort
 import com.asap.common.file.FileMetaData
 import com.asap.domain.UserFixture
 import io.kotest.core.spec.style.BehaviorSpec
 import io.kotest.matchers.nulls.shouldNotBeNull
+import io.kotest.matchers.shouldBe
 import io.mockk.every
 import io.mockk.mockk
+import io.mockk.slot
+import io.mockk.verify
 import java.io.InputStream
 
 class ImageCommandServiceTest :
@@ -24,6 +28,38 @@ class ImageCommandServiceTest :
                 mockUserManagementPort,
             )
 
+        given("익명 사용자의 이미지 업로드 요청이 들어올 때") {
+            val command =
+                UploadImageUsecase.Command(
+                    userId = null,
+                    image =
+                        FileMetaData(
+                            name = "name",
+                            contentType = "contentType",
+                            size = 1L,
+                            inputStream = InputStream.nullInputStream(),
+                        ),
+                )
+            val imageMetadataSlot = slot<ImageMetadata>()
+            every {
+                mockImageManagementPort.save(capture(imageMetadataSlot))
+            } returns
+                UploadedImage(
+                    imageUrl = "anonymousImageUrl",
+                )
+            `when`("이미지 업로드 요청을 처리하면") {
+                val response = imageCommandService.upload(command)
+                then("이미지가 저장되어야 한다") {
+                    response.imageUrl shouldNotBeNull {
+                        this.isNotBlank()
+                        this.isNotEmpty()
+                    }
+                    verify { mockImageManagementPort.save(any()) }
+                    imageMetadataSlot.captured.owner shouldBe "anonymous"
+                }
+            }
+        }
+
         given("이미지 업로드 요청이 들어올 때") {
             val mockUser = UserFixture.createUser()
             val command =

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/common/config/WebConfig.kt

@@ -8,11 +8,11 @@ import org.springframework.web.servlet.config.annotation.WebMvcConfigurer
 
 @Configuration
 class WebConfig(
-    private val accessUserArgumentResolver: AccessUserArgumentResolver
+    private val accessUserArgumentResolver: AccessUserArgumentResolver,
 ) : WebMvcConfigurer {
-
     override fun addCorsMappings(registry: CorsRegistry) {
-        registry.addMapping("/**")
+        registry
+            .addMapping("/**")
             .allowedOrigins("*")
             .allowedMethods("GET", "POST", "PUT", "DELETE")
             .allowedHeaders("*")
@@ -21,4 +21,4 @@ class WebConfig(
     override fun addArgumentResolvers(resolvers: MutableList<HandlerMethodArgumentResolver>) {
         resolvers.add(accessUserArgumentResolver)
     }
-}
+}

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/common/security/annotation/AccessUserArgumentResolver.kt

@@ -10,20 +10,17 @@ import org.springframework.web.method.support.HandlerMethodArgumentResolver
 import org.springframework.web.method.support.ModelAndViewContainer
 
 @Component
-class AccessUserArgumentResolver: HandlerMethodArgumentResolver {
-
-    override fun supportsParameter(parameter: MethodParameter): Boolean {
-        return parameter.hasParameterAnnotation(AccessUser::class.java)
-    }
+class AccessUserArgumentResolver : HandlerMethodArgumentResolver {
+    override fun supportsParameter(parameter: MethodParameter): Boolean = parameter.hasParameterAnnotation(AccessUser::class.java)
 
     override fun resolveArgument(
         parameter: MethodParameter,
         mavContainer: ModelAndViewContainer?,
         webRequest: NativeWebRequest,
-        binderFactory: WebDataBinderFactory?
+        binderFactory: WebDataBinderFactory?,
     ): Any? {
-        val userAuthentication = SecurityContextHolder.getContext().getAuthentication() as UserAuthentication
-        val userId = userAuthentication.getDetails()
-        return userId
+        val authentication = SecurityContextHolder.getContext()?.getAuthentication()
+        val userAuthentication = authentication as? UserAuthentication
+        return userAuthentication?.getDetails()
     }
-}
+}

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/web/image/api/ImageApi.kt

@@ -17,7 +17,7 @@ import org.springframework.web.multipart.MultipartFile
 interface ImageApi {
     @Operation(
         summary = "이미지 업로드",
-        description = "이미지를 업로드합니다.",
+        description = "이미지를 업로드합니다. 회원과 비회원 모두 이용 가능합니다.",
     )
     @PostMapping(consumes = ["multipart/form-data"])
     @ApiResponses(
@@ -28,8 +28,8 @@ interface ImageApi {
                 headers = [
                     Header(
                         name = "Authorization",
-                        description = "액세스 토큰",
-                        required = true,
+                        description = "액세스 토큰 (선택사항)",
+                        required = false,
                     ),
                 ],
             ),
@@ -41,6 +41,6 @@ interface ImageApi {
     )
     fun uploadImage(
         @RequestPart image: MultipartFile,
-        @AccessUser userId: String,
+        @AccessUser userId: String?,
     ): UploadImageResponse
 }

Bootstrap-Module/src/main/kotlin/com/asap/bootstrap/web/image/controller/ImageController.kt

@@ -14,7 +14,7 @@ class ImageController(
 ) : ImageApi {
     override fun uploadImage(
         image: MultipartFile,
-        userId: String,
+        userId: String?,
     ): UploadImageResponse {
         val response =
             uploadImageUsecase.upload(

Bootstrap-Module/src/test/kotlin/com/asap/bootstrap/acceptance/image/controller/ImageControllerTest.kt

@@ -54,4 +54,39 @@ class ImageControllerTest : AcceptanceSupporter() {
             }
         }
     }
+
+    @Test
+    fun uploadImageWithoutAuthentication() {
+        // given
+        val mockFile = MockMultipartFile("image", "test.jpg", "image/jpeg", "test".toByteArray())
+        val mockFileMetaData = FileMetaData("test.jpg", 4, "image/jpeg", mockFile.inputStream)
+        BDDMockito
+            .given(fileConverter.convert(mockFile))
+            .willReturn(mockFileMetaData)
+        BDDMockito
+            .given(
+                uploadImageUsecase.upload(
+                    UploadImageUsecase.Command(
+                        image = mockFileMetaData,
+                        userId = null,
+                    ),
+                ),
+            ).willReturn(UploadImageUsecase.Response("imageUrl"))
+        // when
+        val response =
+            mockMvc.multipart("/api/v1/images") {
+                file(mockFile)
+                contentType = MediaType.MULTIPART_FORM_DATA
+                // No Authorization header
+            }
+        // then
+        response.andExpect {
+            status { isOk() }
+            jsonPath("$.imageUrl") {
+                exists()
+                isString()
+                isNotEmpty()
+            }
+        }
+    }
 }

Common-Module/src/main/kotlin/com/asap/common/security/SecurityContextHolder.kt

@@ -4,9 +4,7 @@ class SecurityContextHolder {
     companion object {
         private val contextHolder = ThreadLocal<SecurityContext<*, *>>()
 
-        fun getContext(): SecurityContext<*, *> {
-            return contextHolder.get()
-        }
+        fun getContext(): SecurityContext<*, *>? = contextHolder.get()
 
         fun setContext(context: SecurityContext<*, *>) {
             contextHolder.set(context)
@@ -16,4 +14,4 @@ class SecurityContextHolder {
             contextHolder.remove()
         }
     }
-}
+}