Skip to content
This repository was archived by the owner on Oct 16, 2024. It is now read-only.
This repository was archived by the owner on Oct 16, 2024. It is now read-only.

과제 코드 리뷰 #1

@mindori

Description

@mindori

테스트 코드 작성이 많이 안되어서 좀 아쉽지만 차츰 익숙해지시면 잘 작성하실 수 있으실 거에요!
3-layered architecture 전환은 매우 잘하셨습니다!

테스트 코드

  • 각 테스트 케이스 별로 테스트 케이스를 명확하게 구성하고 테스트하는데 사용할 가짜 데이터를 목적 별로 잘 구성해주셨네요. 테스트코드를 통해 사용하는 메서드들이 잘 설계되어 있는 것 같아요.
  • 다만, 지금은 정상적으로 호출하는 경우에 대해서만 test case가 작성이 되어있는데 비정상적으로 호출되는 경우에도 test case를 작성해주시는 것이 좋습니다.

src/controllers/resumes.controller.js

  • async/await 구문을 통해 비동기 처리를 잘 구현하였고, 에러 핸들링 부분을 꼼꼼하게 처리하셨네요.
    • 사용자의 요청에 대한 응답을 명확하게 반환하여 사용자들이 이용하기에 좋을 것 같습니다.

src/controllers/users.controller.js

  • 로그아웃할 때 단순 쿠키만 삭제한다고 로그아웃이 될 수 있을지는 생각해보면 좋겠습니다.
    • 혹여나 동일한 쿠키값을 누군가가 받아서 사용한다고 하면 어떻게 될까요?

리포지터리 코드

  • Prisma를 이용하여 DB와 상호작용하는 메서드를 잘 구현하셨네요. DB 접근이 잘 분리되어 있는 것 같습니다.

서비스 코드

  • 각각의 비즈니스 로직이 깔끔하게 구현되어 있고, 사용자의 요청에 따라 적절한 응답을 처리하고 있는 것 같습니다.
  • 에러에 대한 처리도 굉장히 꼼꼼하게 신경써주셨네요.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions