🐛 fix: coderabbit 코드 리뷰 반영

Author: Jo-Minseok

server/src/file/controller/file.controller.ts

@@ -41,7 +41,7 @@ export class FileController {
         validators: [
           new MaxFileSizeValidator({
             maxSize: FILE_SIZE_LIMITS.IMAGE,
-            message: 'file size limit',
+            message: `File size must not exceed ${FILE_SIZE_LIMITS.IMAGE / (1024 * 1024)}MB`,
           }),
           new FileTypeValidator({
             fileType: /image\/(png|jpg|jpeg|webp|gif)/,
@@ -54,11 +54,9 @@ export class FileController {
     @Query() query: UploadFileQueryRequestDto,
     @CurrentUser() user: Payload,
   ) {
-    file.path = await this.fileService.handleUpload(file, query.uploadType);
-
     return ApiResponse.responseWithData(
       '파일 업로드에 성공했습니다.',
-      await this.fileService.create(file, user.id),
+      await this.fileService.handleUpload(file, query.uploadType, user.id),
     );
   }
 

server/src/file/service/file.service.ts

@@ -18,7 +18,11 @@ export class FileService {
     private readonly logger: WinstonLoggerService,
   ) {}
 
-  async handleUpload(file: Express.Multer.File, uploadType: FileUploadType) {
+  async handleUpload(
+    file: Express.Multer.File,
+    uploadType: FileUploadType,
+    userId: number,
+  ) {
     const today = this.getDateString();
     const targetDir = path.join(this.basePath, uploadType, today);
 
@@ -30,7 +34,17 @@ export class FileService {
 
     await fs.writeFile(filePath, file.buffer);
 
-    return filePath;
+    const { originalname, mimetype, size } = file;
+    const savedFile = await this.fileRepository.save({
+      originalName: originalname,
+      mimetype,
+      size,
+      path: filePath,
+      user: { id: userId },
+    });
+    const accessUrl = this.generateAccessUrl(filePath);
+
+    return UploadFileResponseDto.toResponseDto(savedFile, accessUrl);
   }
 
   private async ensureDirectory(dir: string) {
@@ -42,23 +56,6 @@ export class FileService {
     return now.toISOString().split('T')[0];
   }
 
-  async create(
-    file: Express.Multer.File,
-    userId: number,
-  ): Promise<UploadFileResponseDto> {
-    const { originalname, mimetype, size, path } = file;
-    const savedFile = await this.fileRepository.save({
-      originalName: originalname,
-      mimetype,
-      size,
-      path,
-      user: { id: userId },
-    });
-    const accessUrl = this.generateAccessUrl(path);
-
-    return UploadFileResponseDto.toResponseDto(savedFile, accessUrl);
-  }
-
   private generateAccessUrl(filePath: string): string {
     return filePath.replace(this.basePath, '/objects');
   }

server/src/user/provider/github.provider.ts

@@ -65,7 +65,7 @@ export class GithubOAuthProvider implements OAuthProvider {
         scope,
       };
     } catch (error) {
-      this.logger.error(`Oauth 로그인 중 에러 발생, ${error}`);
+      this.logger.error(`Failed to fetch token info from GitHub: ${error}`);
       throw new BadGatewayException(
         '현재 외부 서비스와의 연결에 실패했습니다.',
       );
@@ -93,6 +93,7 @@ export class GithubOAuthProvider implements OAuthProvider {
         picture: avatar_url,
       } as UserInfo;
     } catch (error) {
+      this.logger.error(`Failed to fetch user info from GitHub: ${error}`);
       throw new BadGatewayException(
         '현재 외부 서비스와의 연결에 실패했습니다.',
       );

server/src/user/provider/google.provider.ts

@@ -8,9 +8,12 @@ import {
   UserInfo,
 } from '../constant/oauth.constant';
 import axios from 'axios';
+import { WinstonLoggerService } from '../../common/logger/logger.service';
 
 @Injectable()
 export class GoogleOAuthProvider implements OAuthProvider {
+  constructor(private readonly logger: WinstonLoggerService) {}
+
   getAuthUrl() {
     const googleOAuthUrl = OAUTH_URL_PATH.GOOGLE.AUTH_URL;
 
@@ -66,6 +69,7 @@ export class GoogleOAuthProvider implements OAuthProvider {
         expires_in,
       };
     } catch (error) {
+      this.logger.error(`Failed to fetch token info from Google: ${error}`);
       throw new BadGatewayException(
         '현재 외부 서비스와의 연결에 실패했습니다.',
       );
@@ -93,6 +97,7 @@ export class GoogleOAuthProvider implements OAuthProvider {
         picture,
       } as UserInfo;
     } catch (error) {
+      this.logger.error(`Failed to fetch user info from Google: ${error}`);
       throw new BadGatewayException(
         '현재 외부 서비스와의 연결에 실패했습니다.',
       );

server/test/feed/e2e/up-view-count.e2e-spec.ts

@@ -131,7 +131,7 @@ describe(`POST ${URL}/{feedId} E2E Test`, () => {
     });
     const savedFeedReadRedis = await redisService.sismember(
       redisKeyMake(feed.id.toString()),
-      testIp,
+      testNewIp,
     );
 
     // DB, Redis then

server/test/file/e2e/delete.e2e-spec.ts

@@ -56,11 +56,13 @@ describe(`DELETE ${URL}/{fileId} E2E Test`, () => {
     expect(data).toBeUndefined();
   });
 
-  it('[200] DB에서 파일을 삭제했지만 FS 라이브러리를 통해서 실패했을 경우에 서비스에서 파일 삭제를 성공한다.', async () => {
+  it('[200] DB에서 파일을 삭제했지만 FS 라이브러리의 삭제 문제일 경우에 서비스에서 파일 삭제를 성공한다.', async () => {
     // given
     file = await fileRepository.save(FileFixture.createFileFixture({ user }));
     const fs = await import('fs/promises');
-    jest.spyOn(fs, 'access').mockResolvedValue(undefined);
+    jest
+      .spyOn(fs, 'unlink')
+      .mockRejectedValue(new Error('EACCES: permission denied'));
 
     const accessToken = createAccessToken(user);
 
@@ -82,12 +84,14 @@ describe(`DELETE ${URL}/{fileId} E2E Test`, () => {
     // DB, Redis then
     expect(savedFile).toBeNull();
   });
-
-  it('[200] DB에서 파일을 삭제했지만 FS 라이브러리에서 권한 문제일 경우 서비스에서 파일 삭제를 성공한다.', async () => {
+  it('[200] DB에서 파일을 삭제했지만 FS 라이브러리의 접근 문제일 경우 서비스에서 파일 삭제를 성공한다.', async () => {
     // given
     file = await fileRepository.save(FileFixture.createFileFixture({ user }));
     const fs = await import('fs/promises');
-    jest.spyOn(fs, 'unlink').mockResolvedValue(undefined);
+    jest
+      .spyOn(fs, 'access')
+      .mockRejectedValue(new Error('EACCES: permission denied'));
+
     const accessToken = createAccessToken(user);
 
     // Http when

server/test/file/e2e/upload.e2e-spec.ts

@@ -104,14 +104,13 @@ describe(`POST ${URL} E2E Test`, () => {
 
     // DB, Redis when
     const savedFile = await fileRepository.findOneBy({
-      originalName: 'test.png',
-      mimetype: 'image/png',
+      originalName: 'test.txt',
+      mimetype: 'text/plain',
     });
 
     // DB, Redis then
     expect(savedFile).toBeNull();
   });
-
   it('[400] 파일 크기가 일치하지 않을 경우 파일 업로드를 실패한다. ', async () => {
     // given
     const requestDto = new UploadFileQueryRequestDto({

server/test/oauth/e2e/callback.e2e-spec.ts

@@ -58,7 +58,7 @@ describe(`GET ${URL} E2E Test`, () => {
     // DB, Redis when
     const savedProvider = await providerRepository.findOneBy({
       providerUserId: '1',
-      providerType: 'github',
+      providerType: OAuthType.Github,
     });
 
     // DB, Redis then
@@ -104,7 +104,7 @@ describe(`GET ${URL} E2E Test`, () => {
     // DB, Redis when
     const savedProvider = await providerRepository.findOneBy({
       providerUserId: '1',
-      providerType: 'google',
+      providerType: OAuthType.Google,
     });
 
     // DB, Redis then

server/test/rss/e2e/history/reject.e2e-spec.ts

@@ -21,11 +21,11 @@ describe(`GET ${URL} E2E Test`, () => {
     agent = supertest(app.getHttpServer());
     const rssRejectRepository = app.get(RssRejectRepository);
     const redisService = app.get(RedisService);
-    const rssAccepts = Array.from({ length: 2 }).map((_, i) =>
+    const rssRejects = Array.from({ length: 2 }).map((_, i) =>
       RssRejectFixture.createRssRejectFixture({}, i),
     );
     [rssRejectList] = await Promise.all([
-      rssRejectRepository.save(rssAccepts),
+      rssRejectRepository.save(rssRejects),
       redisService.set(redisKeyMake(sessionKey), 'test1234'),
     ]);
     rssRejectList.reverse();

server/test/user/e2e/certificate.e2e-spec.ts

@@ -70,7 +70,7 @@ describe(`POST ${URL} E2E Test`, () => {
 
     // DB, Redis then
     expect(savedRegisterCode).toBeNull();
-    expect(savedUser).not.toBeUndefined();
+    expect(savedUser).not.toBeNull();
     expect(
       await bcrypt.compare(
         UserFixture.GENERAL_USER.password,