feature: findPostByPostId 의 날짜를 필수 형태로, findPostByPostUUID 를 고도화, 테스트 업데이트

Author: Nuung

src/controllers/post.controller.ts

@@ -11,7 +11,7 @@ import {
 } from '@/types';
 
 export class PostController {
-  constructor(private postService: PostService) {}
+  constructor(private postService: PostService) { }
 
   getAllPosts: RequestHandler = async (
     req: Request<object, object, object, GetAllPostsQuery>,
@@ -70,10 +70,9 @@ export class PostController {
   ) => {
     try {
       const postId = Number(req.params.postId);
-      const { start, end } = req.query;
 
+      const { start, end } = req.query;
       const post = await this.postService.getPostByPostId(postId, start, end);
-
       const response = new PostResponseDto(true, '단건 post 조회에 성공하였습니다.', { post }, null);
 
       res.status(200).json(response);

src/repositories/__test__/post.repo.integration.test.ts

@@ -3,6 +3,7 @@ import { Pool } from 'pg';
 import pg from 'pg';
 import { PostRepository } from '../post.repository';
 import logger from '@/configs/logger.config';
+import { getCurrentKSTDateString, getKSTDateStringWithOffset } from '@/utils/date.util';
 
 
 dotenv.config();
@@ -407,7 +408,9 @@ describe('PostRepository 통합 테스트', () => {
    */
   describe('findPostByPostId', () => {
     it('게시물 ID로 통계 데이터를 조회할 수 있어야 한다', async () => {
-      const result = await repo.findPostByPostId(TEST_DATA.POST_ID);
+      const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+      const endKST = getCurrentKSTDateString();
+      const result = await repo.findPostByPostId(TEST_DATA.POST_ID, sevenDayAgoKST, endKST);
 
       expect(result).toBeDefined();
       expect(Array.isArray(result)).toBe(true);
@@ -437,7 +440,9 @@ describe('PostRepository 통합 테스트', () => {
     });
 
     it('날짜 오름차순으로 정렬된 결과를 반환해야 한다', async () => {
-      const result = await repo.findPostByPostId(TEST_DATA.POST_ID);
+      const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+      const endKST = getCurrentKSTDateString();
+      const result = await repo.findPostByPostId(TEST_DATA.POST_ID, sevenDayAgoKST, endKST);
 
       // 2개 이상의 결과가 있는 경우에만 정렬 검증
       if (result.length >= 2) {
@@ -459,16 +464,20 @@ describe('PostRepository 통합 테스트', () => {
     });
 
     it('존재하지 않는 게시물 ID에 대해 빈 배열을 반환해야 한다', async () => {
+      const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+      const endKST = getCurrentKSTDateString();
       const nonExistentPostId = 9999999;
-      const result = await repo.findPostByPostId(nonExistentPostId);
+      const result = await repo.findPostByPostId(nonExistentPostId, sevenDayAgoKST, endKST);
 
       expect(result).toBeDefined();
       expect(Array.isArray(result)).toBe(true);
       expect(result.length).toBe(0);
     });
 
     it('날짜 형식이 올바르게 변환되어야 한다', async () => {
-      const result = await repo.findPostByPostId(TEST_DATA.POST_ID);
+      const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+      const endKST = getCurrentKSTDateString();
+      const result = await repo.findPostByPostId(TEST_DATA.POST_ID, sevenDayAgoKST, endKST);
 
       if (result.length <= 0) {
         logger.info('존재하지 않는 게시물 ID에 대해 빈 배열을 테스트를 위한 충분한 데이터가 없습니다.');
@@ -490,7 +499,9 @@ describe('PostRepository 통합 테스트', () => {
     });
 
     it('일일 조회수와 좋아요 수가 숫자 타입이어야 한다', async () => {
-      const result = await repo.findPostByPostId(TEST_DATA.POST_ID);
+      const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+      const endKST = getCurrentKSTDateString();
+      const result = await repo.findPostByPostId(TEST_DATA.POST_ID, sevenDayAgoKST, endKST);
 
       if (result.length <= 0) {
         logger.info('일일 조회수와 좋아요 수가 숫자 타입인지 테스트를 위한 충분한 데이터가 없습니다.');

src/repositories/__test__/post.repo.test.ts

@@ -240,7 +240,7 @@ describe('PostRepository', () => {
 
       mockPool.query.mockResolvedValue(createMockQueryResult(mockStats));
 
-      const result = await repo.findPostByPostId(1);
+      const result = await repo.findPostByPostId(1, '2025-05-01', '2025-05-08');
       expect(result).toEqual(mockStats);
     });
   });

src/repositories/post.repository.ts

@@ -270,48 +270,56 @@ export class PostRepository {
     }
   }
 
-  async findPostByPostId(postId: number, start?: string, end?: string) {
-    try {
-      // 기본 쿼리 부분
-      const baseQuery = `
-        SELECT
-          pds.date,
-          pds.daily_view_count,
-          pds.daily_like_count
-        FROM posts_postdailystatistics pds
-        WHERE pds.post_id = $1
-      `;
-
-      // 날짜 필터링 조건 구성
-      let dateFilterQuery = '';
-      const queryParams: Array<number | string> = [postId];
-
-      if (start && end) {
-        dateFilterQuery = `
-          AND pds.date >= $2
-          AND pds.date <= $3
-        `;
-        queryParams.push(start, end);
-      }
-
-      // 정렬 조건 추가
-      const orderByQuery = `ORDER BY pds.date ASC`;
-
-      // 최종 쿼리 조합
-      const fullQuery = [baseQuery, dateFilterQuery, orderByQuery].join(' ');
+  /**
+   * 특정 게시물의 주어진 날짜 범위 내 일별 통계를 조회합니다.
+   *
+   * @param postId - 통계를 조회할 게시물의 ID.
+   * @param start - 조회 범위의 시작 날짜(포함), 'YYYY-MM-DD' 형식.
+   * @param end - 조회 범위의 종료 날짜(포함), 'YYYY-MM-DD' 형식.
+   * @returns 주어진 날짜 범위 내 일별 통계 배열을 반환하는 Promise:
+   *          - `date`: 통계 날짜.
+   *          - `daily_view_count`: 해당 날짜의 조회수.
+   *          - `daily_like_count`: 해당 날짜의 좋아요 수.
+   * @throws {DBError} 데이터베이스 조회 중 오류가 발생한 경우.
+   */
+  async findPostByPostId(postId: number, start: string, end: string) {
+    const query = `
+      SELECT
+        pds.date,
+        pds.daily_view_count,
+        pds.daily_like_count
+      FROM posts_postdailystatistics pds
+      WHERE pds.post_id = $1
+      AND pds.date >= $2
+      AND pds.date <= $3
+      ORDER BY pds.date ASC
+    `;
 
-      // 쿼리 실행
-      const result = await this.pool.query(fullQuery, queryParams);
+    try {
+      const values: Array<number | string> = [postId, start, end];
+      const result = await this.pool.query(query, values);
       return result.rows;
     } catch (error) {
       logger.error('Post Repo findPostByPostId error:', error);
       throw new DBError('단건 post 조회 중 문제가 발생했습니다.');
     }
   }
 
+  /**
+   * 특정 게시물의 uuid 값 기반으로 날짜 범위 내 일별 통계를 조회합니다. 익스텐션에서 사용합니다.
+   *
+   * @param postUUUID - 통계를 조회할 게시물의 UUID.
+   * @param start - 조회 범위의 시작 날짜(포함), 'YYYY-MM-DD' 형식.
+   * @param end - 조회 범위의 종료 날짜(포함), 'YYYY-MM-DD' 형식.
+   * @returns 주어진 날짜 범위 내 일별 통계 배열을 반환하는 Promise:
+   *          - `date`: 통계 날짜.
+   *          - `daily_view_count`: 해당 날짜의 조회수.
+   *          - `daily_like_count`: 해당 날짜의 좋아요 수.
+   * @throws {DBError} 데이터베이스 조회 중 오류가 발생한 경우.
+   */
   async findPostByPostUUID(postUUUID: string, start: string, end: string) {
-    try {
-      const query = `
+    // findPostByPostId 와 다르게 UUID 가 기준이라 join 필수
+    const query = `
       SELECT
         pds.date,
         pds.daily_view_count,
@@ -323,10 +331,10 @@ export class PostRepository {
         AND pds.date >= $2
         AND pds.date <= $3
       ORDER BY pds.date ASC
-      `;
+    `;
 
+    try {
       const values = [postUUUID, start, end];
-
       const result = await this.pool.query(query, values);
       return result.rows;
     } catch (error) {

src/services/__test__/post.service.test.ts

@@ -398,8 +398,8 @@ describe('PostService', () => {
       // 7일 범위 설정 확인 (현재 날짜 2025-03-15 기준)
       expect(postRepo.findPostByPostUUID).toHaveBeenCalledWith(
         'uuid-1234',
-        '2025-03-15', // 현재 날짜 (테스트에서 고정된 날짜)
-        '2025-03-15'  // 현재 날짜 (테스트에서 고정된 날짜)
+        '2025-03-08 00:00:00+09', // 현재 날짜 (테스트에서 고정된 날짜)
+        '2025-03-15 00:00:00+09'  // 현재 날짜 (테스트에서 고정된 날짜)
       );    
     });
 

src/services/post.service.ts

@@ -1,6 +1,7 @@
 import logger from '@/configs/logger.config';
 import { PostRepository } from '@/repositories/post.repository';
 import { RawPostType } from '@/types';
+import { getCurrentKSTDateString, getKSTDateStringWithOffset } from '@/utils/date.util';
 
 export class PostService {
   constructor(private postRepo: PostRepository) { }
@@ -62,35 +63,27 @@ export class PostService {
 
   async getPostByPostId(postId: number, start?: string, end?: string) {
     // start, end 가 yyyy-mm-dd 만 넘어옴, 이를 kst 형태로 바꿔줘야 함
-    const kstStart = `${start} 00:00:00+09`;
-    const kstEnd = `${end} 00:00:00+09`;
+    const startKST = `${start} 00:00:00+09`;
+    const endKST = `${end} 00:00:00+09`;
 
     try {
-      const posts = await this.postRepo.findPostByPostId(postId, kstStart, kstEnd);
-
+      const posts = await this.postRepo.findPostByPostId(postId, startKST, endKST);
       const transformedPosts = this.transformPosts(posts);
-
       return transformedPosts;
     } catch (error) {
       logger.error('PostService getPost error : ', error);
       throw error;
     }
   }
 
-  // !! 해당 함수 사용하지 않는 것으로 보임, 추후 정리 필요
   async getPostByPostUUID(postUUUID: string) {
-    try {
-      const seoulNow = new Date(new Date().getTime() + 9 * 60 * 60 * 1000);
-      const sevenDaysAgo = new Date(seoulNow);
-
-      const start = sevenDaysAgo.toISOString().split('T')[0];
-      const end = seoulNow.toISOString().split('T')[0];
-      sevenDaysAgo.setDate(seoulNow.getDate() - 6);
-
-      const posts = await this.postRepo.findPostByPostUUID(postUUUID, start, end);
+    // 날짜가 넘어오지 않기에 기본적으로 7일로 세팅
+    const sevenDayAgoKST = getKSTDateStringWithOffset(-24 * 60 * 7);
+    const endKST = getCurrentKSTDateString(); // now
 
+    try {
+      const posts = await this.postRepo.findPostByPostUUID(postUUUID, sevenDayAgoKST, endKST);
       const transformedPosts = this.transformPosts(posts);
-
       return transformedPosts;
     } catch (error) {
       logger.error('PostService getPostByPostUUID error : ', error);

src/types/dto/requests/getPostQuery.type.ts

@@ -1,4 +1,4 @@
-import { IsString, Validate } from 'class-validator';
+import { IsNotEmpty, IsString, Validate } from 'class-validator';
 
 export interface PostParam extends Record<string, string> {
   postId: string;
@@ -20,29 +20,36 @@ export interface GetPostQuery {
  *           type: string
  *           format: date
  *           description: 조회 시작 날짜
- *           nullable: true
+ *           nullable: false
  *         end:
  *           type: string
  *           format: date
  *           description: 조회 종료 날짜
- *           nullable: true
+ *           nullable: false
  */
 export class GetPostQueryDto {
   @IsString()
+  @IsNotEmpty()
   @Validate((value: string) => {
     const date = new Date(value);
     return !isNaN(date.getTime());
   }, {
     message: '유효한 날짜 형식이 아닙니다. (예: YYYY-MM-DD)'
   })
-  start?: string;
+  start: string;
 
   @IsString()
+  @IsNotEmpty()
   @Validate((value: string) => {
     const date = new Date(value);
     return !isNaN(date.getTime());
   }, {
     message: '유효한 날짜 형식이 아닙니다. (예: YYYY-MM-DD)'
   })
-  end?: string;
+  end: string;
+
+  constructor(start: string, end: string) {
+    this.start = start;
+    this.end = end;
+  }
 }