Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

주문 테스트 데이터 및 서비스 로직 개선 #175

Merged
merged 4 commits into from
Mar 16, 2024
Merged

Conversation

jihwooon
Copy link
Owner

@jihwooon jihwooon commented Mar 14, 2024

작업 내용 (Content)

  • 주문 테스트 데이터 (server/src/fixture/orders.fixture.ts) 를 정리했습니다.
    • 실제 주문 데이터와 모의 주문 데이터를 구분하기 위해 별도의 배열로 분리했습니다.
    • 실제 주문 데이터는 orders 배열에, 모의 주문 데이터는 mockOrders 배열에 저장합니다.
  • 주문 목록 조회 서비스 (server/src/orders/application/orders-list.service.test.ts) 의 테스트 케이스를 작성했습니다.
    • findAll 함수를 모킹하여 테스트 데이터를 반환하도록 설정했습니다.
    • 테스트 케이스를 통해 서비스가 주문 목록을 정확하게 조회하는지 확인합니다.

링크 (Links)

기타 사항 (Etc)

Merge 전 필요 작업 (Checklist before merge)

  • PR 올리기 전 rebase 동기화를 하셨나요?
  • 마지막 줄에 공백 처리를 하셨나요?
  • 커밋 단위를 의미 단위로 나눴나요?
    • 예시
      • 코드 가독성을 위해 메서드를 추출하라
      • if-else 문을 if 문으로 분리하라
      • 불필요한 메서드를 인라인화하라
  • 커밋 본문을 작성하셨나요?
    • 예시
      • 함수는 한 가지 일을 해야 한다는 원칙에 따라 메서드를 추출합니다.
      • if-else는 컴파일 시 처리가 되어 재컴파일 없이 수정 할 수 없습니다.
        이에 따라 코드가 실행되는 순간에 실행이 결정되는 if 문으로 수정합니다.
  • CI 파이프라인이 통과가 되었나요?
  • 리뷰 요청 전 Self-Review로 의문점을 해결 하셨나요?
  • PR 리뷰 가능한 크기를 유지하셨나요?

- 주문 테스트 데이터 (server/src/fixture/orders.fixture.ts) 를 정리했습니다.
  - 실제 주문 데이터와 모의 주문 데이터를 구분하기 위해 별도의 배열로 분리했습니다.
  - 실제 주문 데이터는 orders 배열에, 모의 주문 데이터는 mockOrders 배열에 저장합니다.
- 주문 목록 조회 서비스 (server/src/orders/application/orders-list.service.test.ts) 의 테스트 케이스를 작성했습니다.
  - findAll 함수를 모킹하여 테스트 데이터를 반환하도록 설정했습니다.
  - 테스트 케이스를 통해 서비스가 주문 목록을 정확하게 조회하는지 확인합니다.
- 주문 목록 조회 서비스 (server/src/orders/application/orders-list.service.ts)에 에러 처리 로직을 추가했습니다.
  - 주문 목록이 비어있는 경우 NOT_FOUND 에러를 발생시킵니다.
- 테스트 코드 (server/src/orders/application/orders-list.service.test.ts)를 업데이트하여 추가된 에러 처리 로직을 검증합니다.
  - 모의 주문 데이터가 없는 경우 서비스가 예외를 발생하는지 테스트합니다.
- 주문 상세 조회 서비스 (server/src/orders/application/orders-detail.service.ts)에 대한 테스트 코드를 추가했습니다.
- 테스트 데이터 (server/src/fixture/orders.fixture.ts)에 주문 상세 정보를 정의했습니다.
- 주문 목록 조회 컨트롤러 (server/src/orders/web/orders-list.controller.ts)에 에러 처리 로직을 추가했습니다.
  - 주문 목록 조회에 실패하면 NOT_FOUND 에러를 발생시킵니다.
- 테스트 코드 (server/src/orders/web/orders-list.controller.test.ts)를 업데이트하여 추가된 에러 처리 로직을 검증합니다.
  - 모의 서비스 에러를 발생시켜 컨트롤러가 예외를 처리하는지 테스트합니다.
@jihwooon jihwooon merged commit dafae9f into main Mar 16, 2024
1 check passed
@jihwooon jihwooon deleted the issue-101 branch March 16, 2024 04:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant