Refactor/314

back/src/main/java/com/back/domain/file/video/controller/VideoController.java

@@ -37,8 +37,8 @@ public RsData<UploadUrlGetResponse> getUploadUrl(@RequestParam String filename)
 
     @GetMapping("/videos/download")
     @Operation(summary = "다운로드용 URL 요청", description = "파일 다운로드를 위한 Presigned URL을 발급받습니다.")
-    public RsData<DownLoadUrlGetResponse> getDownloadUrls(@RequestParam String objectKey) {
-        PresignedUrlResponse downloadUrl = fileManager.getDownloadUrl(objectKey);
+    public RsData<DownLoadUrlGetResponse> getDownloadUrls(@RequestParam String uuid, @RequestParam String resolution) {
+        PresignedUrlResponse downloadUrl = fileManager.getDownloadUrl(uuid, resolution);
         DownLoadUrlGetResponse response = new DownLoadUrlGetResponse(downloadUrl.url().toString(), downloadUrl.expiresAt());
         return new RsData<>("200", "다운로드용 URL 요청완료", response);
     }

back/src/main/java/com/back/domain/file/video/repository/VideoRepository.java

@@ -9,4 +9,5 @@
 @Repository
 public interface VideoRepository extends JpaRepository<Video, Integer> {
     Optional<Video> findByUuid(String uuid);
+    boolean existsByUuid(String uuid);
 }

back/src/main/java/com/back/domain/file/video/service/FileManager.java

@@ -8,13 +8,15 @@
 
 import java.net.URL;
 import java.time.LocalDateTime;
+import java.util.Set;
 import java.util.UUID;
 
 @Service
 @RequiredArgsConstructor
 public class FileManager {
     private final VideoService videoService;
     private final S3Service s3Service;
+    private final Set<String> validResolutions = Set.of("480p", "720p", "1080p");
 
     public PresignedUrlResponse getUploadUrl(String filename) {
         String contentType = validateAndGetContentType(filename);
@@ -53,8 +55,11 @@ private String validateAndGetContentType(String filename) {
         }
     }
 
-    public PresignedUrlResponse getDownloadUrl(String objectKey) {
+    public PresignedUrlResponse getDownloadUrl(String uuid, String resolution) {
         Integer expires = 60;
+        validateResolution(resolution);
+        videoService.isExistByUuid(uuid);
+        String objectKey = "transcoded/videos/" + uuid + "/" + resolution + "/manifest.mpd";
         URL url = s3Service.generateDownloadUrl(objectKey, expires);
         LocalDateTime expiresAt = LocalDateTime.now().plusMinutes(expires);
         return new PresignedUrlResponse(url, expiresAt);
@@ -68,4 +73,10 @@ public void updateVideoStatus(String videoId, String status) {
             videoService.createVideo(videoId, status, "/", 0);
         }
     }
+
+    public void validateResolution(String resolution) {
+        if (!validResolutions.contains(resolution)) {
+            throw new ServiceException("400", "지원하지 않는 해상도입니다: " + resolution);
+        }
+    }
 }

back/src/main/java/com/back/domain/file/video/service/VideoService.java

@@ -29,4 +29,10 @@ public Video updateStatus(String uuid, String status) {
         news.updateStatus(status);
         return videoRepository.save(news);
     }
+
+    public void isExistByUuid(String uuid) {
+        if (!videoRepository.existsByUuid(uuid)) {
+            throw new ServiceException("404", "Video not found");
+        }
+    }
 }

back/src/main/java/com/back/global/security/SecurityConfig.java

@@ -30,7 +30,6 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                         .requestMatchers("/auth/**").permitAll()
                         .requestMatchers("/actuator/**").permitAll()
                         .requestMatchers("/swagger-ui/**", "/v3/api-docs/**", "/swagger-resources/**").permitAll()
-                        .requestMatchers("/videos/upload").permitAll()
                         .requestMatchers("/ws-chat/**").permitAll()
                         .anyRequest().authenticated()
                 )

back/src/test/java/com/back/domain/file/video/controller/VideoControllerTest.java

@@ -0,0 +1,100 @@
+package com.back.domain.file.video.controller;
+
+import com.back.domain.file.video.dto.service.PresignedUrlResponse;
+import com.back.domain.file.video.service.FileManager;
+import org.junit.jupiter.api.DisplayName;
+import org.junit.jupiter.api.Test;
+import org.mockito.Mockito;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.autoconfigure.web.servlet.AutoConfigureMockMvc;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.http.MediaType;
+import org.springframework.security.test.context.support.WithMockUser;
+import org.springframework.test.context.bean.override.mockito.MockitoBean;
+import org.springframework.test.web.servlet.MockMvc;
+
+import java.net.URL;
+import java.time.LocalDateTime;
+
+import static org.mockito.ArgumentMatchers.anyString;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.jsonPath;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+@SpringBootTest
+@AutoConfigureMockMvc
+class VideoControllerTest {
+
+    @Autowired
+    private MockMvc mockMvc;
+
+    @MockitoBean
+    private FileManager fileManager;
+
+    @Test
+    @DisplayName("업로드 URL 요청 - 성공")
+    @WithMockUser(roles = "ADMIN")
+    void testGetUploadUrl() throws Exception {
+        URL mockUrl = new URL("https://bucket.s3.amazonaws.com/123e4567-e89b-12d3-a456-426614174000");
+        LocalDateTime expiresAt = LocalDateTime.now().plusMinutes(10);
+
+        Mockito.when(fileManager.getUploadUrl(anyString()))
+                .thenReturn(new PresignedUrlResponse(mockUrl, expiresAt));
+
+        mockMvc.perform(get("/videos/upload")
+                        .param("filename", "test.mp4")
+                        .contentType(MediaType.APPLICATION_JSON))
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.resultCode").value("200"))
+                .andExpect(jsonPath("$.msg").value("업로드용 URL 요청완료"))
+                .andExpect(jsonPath("$.data.url").value(mockUrl.toString()))
+                .andExpect(jsonPath("$.data.uuid").value("123e4567-e89b-12d3-a456-426614174000"));
+    }
+
+    @Test
+    @DisplayName("업로드 URL 요청 - 권한 부족")
+    @WithMockUser(roles = "USER")
+        // ADMIN 아님
+    void testGetUploadUrl_Unauthorized() throws Exception {
+        mockMvc.perform(get("/videos/upload")
+                        .param("filename", "test.mp4")
+                        .contentType(MediaType.APPLICATION_JSON))
+                .andExpect(status().isForbidden()); // 403 권한 부족
+    }
+
+    @Test
+    @DisplayName("업로드 URL 요청 - 파일명에 확장자가 없는 경우")
+    @WithMockUser(roles = "ADMIN")
+    void testGetUploadUrl_NoExtension() throws Exception {
+        Mockito.when(fileManager.getUploadUrl(anyString()))
+                .thenThrow(new com.back.global.exception.ServiceException("400", "지원하지 않는 동영상 파일 형식입니다: testfile"));
+
+        mockMvc.perform(get("/videos/upload")
+                        .param("filename", "testfile")
+                        .contentType(MediaType.APPLICATION_JSON))
+                .andExpect(status().isBadRequest())
+                .andExpect(jsonPath("$.resultCode").value("400"))
+                .andExpect(jsonPath("$.msg").value("지원하지 않는 동영상 파일 형식입니다: testfile"));
+    }
+
+    @Test
+    @DisplayName("다운로드 URL 요청 - 성공")
+    @WithMockUser
+    void testGetDownloadUrls() throws Exception {
+        URL mockUrl = new URL("https://bucket.s3.amazonaws.com/test.mp4");
+        LocalDateTime expiresAt = LocalDateTime.now().plusMinutes(10);
+
+        Mockito.when(fileManager.getDownloadUrl(anyString(), anyString()))
+                .thenReturn(new PresignedUrlResponse(mockUrl, expiresAt));
+
+        mockMvc.perform(get("/videos/download")
+                        .param("uuid", "123e4567-e89b-12d3-a456-426614174000")
+                        .param("resolution", "1080p")
+                        .contentType(MediaType.APPLICATION_JSON))
+                .andExpect(status().isOk())
+                .andExpect(jsonPath("$.resultCode").value("200"))
+                .andExpect(jsonPath("$.msg").value("다운로드용 URL 요청완료"))
+                .andExpect(jsonPath("$.data.url").value(mockUrl.toString()));
+    }
+
+}

back/src/test/java/com/back/domain/file/entity/VideoTest.java → back/src/test/java/com/back/domain/file/video/entity/VideoTest.java

@@ -1,4 +1,4 @@
-package com.back.domain.file.entity;
+package com.back.domain.file.video.entity;
 
 import com.back.domain.file.video.entity.Video;
 import org.junit.jupiter.api.DisplayName;

back/src/test/java/com/back/domain/file/service/S3ServiceTest.java → back/src/test/java/com/back/domain/file/video/service/S3ServiceTest.java

@@ -1,4 +1,4 @@
-package com.back.domain.file.service;
+package com.back.domain.file.video.service;
 
 import com.back.domain.file.video.service.S3Service;
 import com.back.global.exception.ServiceException;

back/src/test/java/com/back/domain/file/service/VideoServiceTest.java → back/src/test/java/com/back/domain/file/video/service/VideoServiceTest.java

@@ -1,4 +1,4 @@
-package com.back.domain.file.service;
+package com.back.domain.file.video.service;
 
 import com.back.domain.file.video.entity.Video;
 import com.back.domain.file.video.repository.VideoRepository;