[장인혁] sprint9 & 10

[CodeinHyuk]

Sprint 9

• 미션 목표

  • Jest와 Supertest를 사용해 유닛 테스트, 통합 테스트 작성하기

• 기본 요구사항

  • Jest의 테스트 커버리지 도구를 사용하도록 설정해 주세요.
  • 인증이 필요하지 않은 상품 API에 대한 통합 테스트를 작성해 주세요.
  • 인증이 필요하지 않은 게시글 API에 대한 통합 테스트를 작성해 주세요.
  • 로그인, 회원가입 API에 대한 통합 테스트를 작성해 주세요.
  • 인증이 필요한 상품 API에 대해 통합 테스트를 작성해 주세요.
  • 인증이 필요한 게시글 API에 대해 통합 테스트를 작성해 주세요.

• 심화 요구사항

  • 상품 API의 비즈니스 로직에 대해 Mock, Spy를 활용해 유닛 테스트를 작성해 주세요.

---

Sprint 10

• 기본 요구사항

  • 프로젝트에 프로덕션 배포를 위한 환경 변수 설정을 해 주세요.

• AWS S3 적용

  • AWS S3 버킷을 생성하고, 퍼블릭 액세스를 허용해 주세요.
  • 일반 사용자가 S3 업로드된 파일에 접근할 수 있도록 S3 버킷 정책을 설정해 주세요.
  • AWS EC2에서 AWS S3를 사용하기 위한 액세스 키를 AWS IAM에서 발급하세요.
  • 프로덕션 환경에서는 파일 업로드에 AWS S3를 사용하도록 구현을 수정해 주세요.

• AWS RDS 적용

  • AWS RDS 프리티어에 해당하는 인스턴스를 생성합니다.
  • RDS 인스턴스에 대한 보안 그룹을 설정합니다.
  • 프로덕션 환경에서는 Prisma에 프로젝트 데이터베이스와 연결하도록 합니다.

• AWS EC2에 Express 서버 배포하기

  • AWS EC2 프리티어에 해당하는 인스턴스를 생성합니다.
  • SSH를 사용해 EC2 인스턴스에 접속해 Express 서버를 배포해 주세요.

• 심화 요구사항

  • EC2 인스턴스에서 pm2 프로세스 매니저를 사용하여 애플리케이션을 실행해 주세요.
  • EC2 인스턴스에서 nginx 리버스 프록시를 설정해 서버를 80번 포트로 서비스합니다.

API 엔드포인트 주소: http://54.180.29.185

[peterhyun1234]

@CodeinHyuk 님, 안녕하세요!
이번 Sprint 9 & 10 테스트 미션 제출하시느라 고생 많으셨습니다 👋
Jest + Supertest 설정부터 통합 테스트, 유닛 테스트, 심화 요구사항까지 전체 스펙을 빠짐없이 완성하셨고, main.ts의 서버 분리나 secure 쿠키 플래그 추가 같은 테스트 외적인 개선도 함께 챙겨주신 점이 인상적이었습니다.

---

LGTM! (Looks Good To Me) 👍

이번 PR은 요구사항 완성도 + 테스트 구조 + 비즈니스 로직 검증 측면에서 전반적으로 잘 갖춰져 있어 LGTM 드립니다.

---

칭찬하고 싶은 점 ✨

1. 테스트 환경 분리를 제대로 챙겼습니다

   • main.ts에 require.main === module 가드를 추가해 테스트 실행 시 서버가 자동으로 뜨지 않도록 처리한 점, 소켓 초기화도 NODE_ENV !== 'test'로 분리한 점이 아주 좋습니다.
   • cross-env NODE_ENV=test로 환경변수를 명시적으로 주입하는 스크립트 구성도 깔끔합니다.

2. Jest 커버리지 설정이 꼼꼼합니다

   • collectCoverage, coverageDirectory, collectCoverageFrom에서 .d.ts와 main.ts를 제외한 점까지 세심하게 처리됐습니다.
   • setupFilesAfterEnv로 공통 teardown을 setup.ts에 분리한 구조도 좋습니다.

3. productService 유닛 테스트의 비즈니스 로직 검증이 훌륭합니다

   • 가격 변경 시 좋아요 유저들에게 알림을 전송하는 핵심 로직을, findById / findLikers / update / notificationService.createAndSend 네 가지 의존성을 모두 Spy로 모킹하고 호출 횟수와 인자까지 검증하셨습니다.
   • 단순히 "테스트가 통과한다"를 넘어서 "이 로직이 올바르게 동작한다"를 증명하는 테스트입니다. 심화 요구사항을 제대로 이해하고 작성하셨습니다.

4. authController 보안 개선

   • 쿠키에 secure: NODE_ENV === 'production' 플래그를 추가해 프로덕션에서 HTTPS 전용 쿠키가 되도록 개선한 점도 좋습니다. 테스트 미션이라 그냥 넘어갈 수도 있는 부분인데 챙겨주셨습니다.

---

한 단계 더 성장하기 위한 제안 🌱

1. prisma.$disconnect() 중복 호출

   • tests/setup.ts에 이미 prisma.$disconnect()가 있는데, 각 테스트 파일의 afterAll에도 중복 호출되고 있습니다. setup.ts에서 한 번만 호출하고 나머지는 제거하면 됩니다.

2. testProductId 의존성 문제

   • '상품 생성' 테스트가 실패하면 이후 수정 테스트에서 undefined로 요청이 나갑니다. beforeAll에서 미리 상품을 생성해두는 방식이 더 안전합니다.

   beforeAll(async () => {
     const res = await request(app).post('/products').set('Cookie', authCookie).send({...});
     testProductId = res.body.id;
   });

3. article.test.ts - 테스트 데이터 정리 누락

   • afterAll에서 'Test Article'만 삭제하는데, 수정 테스트에서 생성한 'Update Test' 게시글이 정리되지 않습니다. afterAll에 함께 포함시켜 주세요.

4. '존재하지 않는 게시글 조회' 테스트 위치

   • GET /articles/999999는 인증이 필요 없는 API인데 '인증이 필요한 API' describe 블록 안에 있습니다. 올바른 블록으로 옮기면 테스트 구조가 더 명확해집니다.

5. 엣지 케이스 보강

   • 이미 가입된 이메일로 재가입 시 409, 잘못된 비밀번호로 로그인 시 401 같은 실패 케이스를 추가하면 커버리지가 더 탄탄해집니다.
   • articleService.test.ts에 권한 없는 유저가 삭제 시도 시 에러가 발생하는지 검증하는 케이스도 추가해보세요.

---

총평 🚀

이번 PR은 테스트 환경 구성 → 통합 테스트 → 유닛 테스트 심화까지 요구사항을 완전히 달성한, 완성도 높은 제출물입니다.
특히 productService 유닛 테스트에서 비즈니스 로직의 핵심 흐름을 Spy로 꼼꼼하게 검증한 부분은 이번 미션의 백미였습니다.

고생 많으셨습니다! Merge 진행하겠습니다.
다음 스프린트도 기대할게요 💪