User/login 불필요한 임포트 제거 / 에러 catch 시 리턴하는 코드 수정 / 에러객체 수정 / 테스트코드 진행중 / 파일명, 메소드명 수정

[ghost]

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);

[ghost]

어라... 2번 적용이 안 된 것 같아요;;; 다시 커밋해서 PR 해볼게요