search

jiyoon-na / najiArchitec

0 stars 0 forks source link

심화 주차 개인과제 피드백 - 정영훈 튜터 #1

Open jyh7a opened 2 months ago

jyh7a commented 2 months ago

심화 주차 생각보다 어려웠을 텐데 그래도 잘 구현해 주셨습니다! 배운 디버깅 기능 잘 활용해서 몇가지 기능 수정하고 유닛테스트도 앞으로 진행 하면 좋을거 같습니다.

아래는 포로젝트 피드백입니다.

1. 주석 제거

https://github.com/jiyoon-na/najiArchitec/blob/6c5681604d4cfe0cdafa4c146970a8a9e872f46f/node-advanced-template/src/routers/auth.router.js#L24-L26 코드 작성후 필요없는 주석은 적절히 제거해 주세요. router에서 AuthController를 불러서 사용하고 있기 때문에 해당 주석들은 삭제하면 됩니다.



2. 로그인 기능 수정

회원가입 기능 잘 동작하고 로그인 기능을 테스트하려고 하는데 로그인이 정상적으로 동작하지 않습니다. 제 local에서는 정상적으로 회원가입, 로그인 성공하지만 올려주신 서버에서는 회원가입 - 성공, 로그인 - 실패 로 나오기 때문에 해당 서버에서 직접 log를 보고 어떤 에러인지 파악이 필요합니다. 실제로 사용되는 서버에 API를 호출해서 어떤 에러인지 파악 후 수정해 주세요.



3. 이력서 상세조회/삭제 수정

https://github.com/jiyoon-na/najiArchitec/blob/6c5681604d4cfe0cdafa4c146970a8a9e872f46f/node-advanced-template/src/repositories/resumes.repository.js#L53 이력사 상세조회시 해당 부분에서 에러가 납니다. 에러를 살펴보면 "class NotFound"라고 나오고 있습니다. NotFound가 클래스이기 떄문에 이 부분에는 메세지가 오는게 맞습니다. 에러를 살펴보고 한번 수정해 주세요.

에러 내용) 

Error: class NotFound {
  constructor(message = NotFound.name) {
    this.message = message;
    this.status = HTTP_STATUS.NOT_FOUND;
  }
}
    at ResumesRepository.findresumeById (file:///Users/jyh7a/Documents/projects/sparta-student-projects/5rd-generation/week-6-resume/najiArchitec/node-advanced-template/src/repositories/resumes.repository.js:53:13)
    at async ResumesService.findresumeById (file:///Users/jyh7a/Documents/projects/sparta-student-projects/5rd-generation/week-6-resume/najiArchitec/node-advanced-template/src/services/resumes.service.js:33:18)
    at async getResumeById (file:///Users/jyh7a/Documents/projects/sparta-student-projects/5rd-generation/week-6-resume/najiArchitec/node-advanced-template/src/controllers/resumes.controller.js:67:22)



4. 이력서 수정 컨드롤러 수정

https://github.com/jiyoon-na/najiArchitec/blob/6c5681604d4cfe0cdafa4c146970a8a9e872f46f/node-advanced-template/src/controllers/resumes.controller.js#L87 이력서 수정 컨트롤러에서 createResume 서비스를 잘못 불러오고 있습니다. 아래 예시처럼 updateResume를 불러오도록 수정해 주세요.

예)

      const updatedResume = await this.resumesService.updateResume(
        authorId,
        title,
        content,
      );



5. 나머지 기능 완성

위에 오류를 해결하고 추가적인 유닛테스트 기능들도 마저 구현해 보세요. 항상 오류가 나면 에러코드를 console.log 혹은 저번에 배운 디버깅 기능을 이용해서 에러를 해결해 주세요!

jiyoon-na commented 2 months ago

넵 피드백 확인했습니다 감사합니다 :)