User/login User디렉토리 리팩토링 / 불필요한 임포트 제거 / 에러 catch 시 리턴하는 코드 수정 / 에러객체 수정 / 테스트코드 진행중 / 파일명, 메소드명 수정 / 닉네임 변경 예외 처리 추가

[ghost]

0. 코드 리팩토링 #175

• EC2 테스트 완료 (EC2 테스트는 배포서버에서 브랜치를 따로 생성해서 테스트했습니다.)
  1. 유저 플레이 전적 가공하는 부분 하나로 모아서 메소드로 정리, user-repo.js에서 외부 파일로 분리(user/util/user-function.js)
  2. user-provider.js 의 메소드 중 라우터에서 바로 넘어온 메소드 외의 메소드들 Service로 이동
  3. user-provider.js 의 메소드 중 라우터에서 바로 넘어온 메소드들 비즈니스 로직 분리( getKakaoUserInfo 제외)
  4. getKakaoUserInfo 비즈니스 로직 분리
  5. repo에 있는 createNewUser 함수 DB 작업과 무관한 부분 로직 분리
  6. UserService 의 메인 함수 loginInfo 내에서 사용하는 서브 함수 exUserGetToken 내부 로직 단순화
  7. 기본적인 에러 핸들링 적용, 수시로 보완해나갈 예정

1. 불필요한 모듈 임포트 제거

• user-repo.js에서 사용하지 않고 있던 require('dotenv').config() 삭제

2. try/catch 로 에러를 확인했을 때, 클라이언트로 return 하는 코드 수정

(기존)

if (e.name === 'UserError') {
    return res.status(e.statusCode).send({ errorMessage: e.message });
} else {
    res.send({ errorMessage: e.message });
}

(변경)

return res.status((e.statusCode ??= 500)).send({ errorMessage: e.message });

3. 에러객체 수정

• 리턴할 때 커스텀 에러 여부를 구분하기 위해 name 프로퍼티를 사용했었는데 statusCode 의 존재여부로 확인할 수 있어서 name프로퍼티 삭제

4. 파일명 / 메소드 변경

• user-provider.js => user-controller.js 로 변경
• user-controller.js의 메소드와 통일성을 위해 user-service.js 의 loginInfo -> getLoginInfo 로 변경

5. 테스트 코드 (진행중)

• 빈 테스트 파일이 있을 경우 에러가 발생하여 현재 작성 중인 테스트 코드 파일 제외하고 삭제

6. user-controller.js 의 updateNick 메소드 에러 처리 추가

• 2번 작동 테스트를 위해 커스텀 에러 외의 에러 발생시키려고 하다가 아무 값이 오지 않아도 오류가 발생하지 않는 부분 확인

  if (!nickname) throw new UserError('변경할 닉네임을 입력해주세요', 400);
  if (nickname.match(/\s/g))
  throw new UserError('닉네임에 공백이 포함될 수 없습니다.', 400);