refactor: 단일/목록 응답 클래스 네이밍 통일

Author: Gyuhyeok99

src/main/java/com/example/solidconnection/news/controller/NewsController.java

@@ -3,7 +3,7 @@
 import com.example.solidconnection.common.resolver.AuthorizedUser;
 import com.example.solidconnection.news.dto.NewsCommandResponse;
 import com.example.solidconnection.news.dto.NewsCreateRequest;
-import com.example.solidconnection.news.dto.NewsResponse;
+import com.example.solidconnection.news.dto.NewsListResponse;
 import com.example.solidconnection.news.dto.NewsUpdateRequest;
 import com.example.solidconnection.news.service.NewsCommandService;
 import com.example.solidconnection.news.service.NewsQueryService;
@@ -34,11 +34,11 @@ public class NewsController {
 
     // todo: 추후 Slice 적용
     @GetMapping
-    public ResponseEntity<NewsResponse> findNewsBySiteUserId(
+    public ResponseEntity<NewsListResponse> findNewsBySiteUserId(
             @RequestParam(value = "site-user-id") Long siteUserId
     ) {
-        NewsResponse newsResponse = newsQueryService.findNewsBySiteUserId(siteUserId);
-        return ResponseEntity.ok(newsResponse);
+        NewsListResponse newsListResponse = newsQueryService.findNewsBySiteUserId(siteUserId);
+        return ResponseEntity.ok(newsListResponse);
     }
 
     @RequireRoleAccess(roles = {Role.ADMIN, Role.MENTOR})

src/main/java/com/example/solidconnection/news/dto/NewsItemResponse.java

@@ -0,0 +1,11 @@
+package com.example.solidconnection.news.dto;
+
+import java.util.List;
+
+public record NewsListResponse(
+        List<NewsResponse> newsResponseList
+) {
+    public static NewsListResponse from(List<NewsResponse> newsResponseList) {
+        return new NewsListResponse(newsResponseList);
+    }
+}

src/main/java/com/example/solidconnection/news/dto/NewsListResponse.java

@@ -1,11 +1,25 @@
 package com.example.solidconnection.news.dto;
 
-import java.util.List;
+import com.example.solidconnection.news.domain.News;
+
+import java.time.ZonedDateTime;
 
 public record NewsResponse(
-        List<NewsItemResponse> newsItemsResponseList
+        long id,
+        String title,
+        String description,
+        String thumbnailUrl,
+        String url,
+        ZonedDateTime updatedAt
 ) {
-    public static NewsResponse from(List<NewsItemResponse> newsItemsResponseList) {
-        return new NewsResponse(newsItemsResponseList);
+    public static NewsResponse from(News news) {
+        return new NewsResponse(
+                news.getId(),
+                news.getTitle(),
+                news.getDescription(),
+                news.getThumbnailUrl(),
+                news.getUrl(),
+                news.getUpdatedAt()
+        );
     }
 }

src/main/java/com/example/solidconnection/news/dto/NewsResponse.java

@@ -1,8 +1,8 @@
 package com.example.solidconnection.news.service;
 
 import com.example.solidconnection.news.domain.News;
-import com.example.solidconnection.news.dto.NewsItemResponse;
 import com.example.solidconnection.news.dto.NewsResponse;
+import com.example.solidconnection.news.dto.NewsListResponse;
 import com.example.solidconnection.news.repository.NewsRepository;
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Service;
@@ -17,11 +17,11 @@ public class NewsQueryService {
     private final NewsRepository newsRepository;
 
     @Transactional(readOnly = true)
-    public NewsResponse findNewsBySiteUserId(Long siteUserId) {
+    public NewsListResponse findNewsBySiteUserId(Long siteUserId) {
         List<News> newsList = newsRepository.findAllBySiteUserIdOrderByUpdatedAtDesc(siteUserId);
-        List<NewsItemResponse> newsItemsResponseList = newsList.stream()
-                .map(NewsItemResponse::from)
+        List<NewsResponse> newsResponseList = newsList.stream()
+                .map(NewsResponse::from)
                 .toList();
-        return NewsResponse.from(newsItemsResponseList);
+        return NewsListResponse.from(newsResponseList);
     }
 }

src/main/java/com/example/solidconnection/news/service/NewsQueryService.java

@@ -1,8 +1,8 @@
 package com.example.solidconnection.news.service;
 
 import com.example.solidconnection.news.domain.News;
-import com.example.solidconnection.news.dto.NewsItemResponse;
 import com.example.solidconnection.news.dto.NewsResponse;
+import com.example.solidconnection.news.dto.NewsListResponse;
 import com.example.solidconnection.news.fixture.NewsFixture;
 import com.example.solidconnection.siteuser.domain.SiteUser;
 import com.example.solidconnection.siteuser.fixture.SiteUserFixture;
@@ -40,12 +40,12 @@ class NewsQueryServiceTest {
         List<News> newsList = List.of(news1, news2);
 
         // when
-        NewsResponse response = newsQueryService.findNewsBySiteUserId(user1.getId());
+        NewsListResponse response = newsQueryService.findNewsBySiteUserId(user1.getId());
 
         // then
-        assertThat(response.newsItemsResponseList()).hasSize(newsList.size());
-        assertThat(response.newsItemsResponseList())
-                .extracting(NewsItemResponse::updatedAt)
+        assertThat(response.newsResponseList()).hasSize(newsList.size());
+        assertThat(response.newsResponseList())
+                .extracting(NewsResponse::updatedAt)
                 .isSortedAccordingTo(Comparator.reverseOrder());
     }
 }