[BE] 테스트 코드 리팩토링

[Jo-Minseok]

테스트 코드 일관성

• DTO 테스트가 전부 존재하는지 확인 - @Jo-Minseok
• 테스트 코드 describe 범위 확인(어떤 건 메소드별, 어떤 건 API별임) - @Jo-Minseok
• E2E 테스트에서 모든 응답값을 테스트하는지 확인 - @Jo-Minseok
• 테스트 코드 파일 명칭 및 컨벤션 일치 확인 - @Jo-Minseok
• given when then 주석 확인 - @Jo-Minseok
• E2E 테스트 데이터 반환 API는 데이터 값 검증 (but, 메세지는 제외) - @Jo-Minseok
• E2E에서 테스트 수행시 DTO 사용 확인 - @Jo-Minseok
• 응답 코드 객체화 - @Jo-Minseok
• 테스트 케이스 멘트 수정 - @Jo-Minseok
• 테스트 코드 하드 코딩 값 제거 - @Jo-Minseok
• 이메일 및 Fetch 등 외부 의존 함수 및 의존 데이터 제거 - @Jo-Minseok
• 데이터베이스 및 Redis 값 검증 - @Jo-Minseok
• Redis 값 삭제 검증 - @Jo-Minseok
• 테스트 코드 비동기 개선 - @Jo-Minseok
• 코드에서 불필요한 변수 및 로직 제거 - @Jo-Minseok

테스트 코드 설정 격리

• 테스트 예제 파일 생성
• 테스트 코드 설정 파일 디렉터리 격리

테스트 코드 병렬 실행

• 테스트 코드 파일끼리 완전 독립성 개선
• 테스트 속도 개선
• 테스트 코드 픽스처 개선으로 병렬 테스트 가능
• MySQL DB Reset 구문 지우기

After Job

• 테스트 코드에 따른 소스 코드 2차 리팩토링 (Transaction Callback 방식으로 변경, TypeORM 조회 코드 변경)
• logger.ts (피드 크롤러, NestJS 둘 다)에서 Console이 2개가 들어가 있어서 두번 출력되는 버그 - email worker: @CodeVac513
• User Module에 AdminModule import 제거
• oauth provider를 객체로 받아오는 검증 코드가 있는데 이미 Request DTO에서 검증 하고 있어서 불필요
• 유저 로그인에서 Refresh Token 재발급 하는 코드 필요
• Socket.IO Chat 코드가 지금은 익명 채팅방 하나로 국한되어있기에 확장성 개선
• File Delete쪽 주인 확인 추가
• AuthGuard에 유저 찾기로 못 찾으면 오류 + 테스트 코드도 작성 - @asn6878
• callback AccessToken 던져주는거 URL로 주면 위험 - @asn6878
• uuidv4 → uuid.v4 로 변경 - @Jo-Minseok
• uuid → xxxcode로 변경
• Chat Message 핸들러 타입 안쓰이는 속성 제거
• 조회수 중복증가 레이어 만들기 (OR 조건 사용 중) → 설계대로 수정
• rss 승인시 transaction 코드가 비동기로 돌아가서 낙관적 업데이트 개선
• transaction 코드 queryrunner → callback 형식으로 변경

인수조건

• 테스트를 통과한다.
• 테스트 속도 개선을 성공한다.