-
Notifications
You must be signed in to change notification settings - Fork 2
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
동걸 TC 정의 #27
동걸 TC 정의 #27
Conversation
📝 테스트 커버리지 리포트입니다
|
📝 테스트 커버리지 리포트입니다
|
@Test | ||
@DisplayName("`Then: 주문이 '결제 대기' 혹은 '발송 대기' 상태가 아니면, IllegalStateException 발생한다.`") | ||
fun tc1() { | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
주문의 결제 상태에 따라 각각의 테스트 케이스가 있는 것도 괜찮은 것 같습니다. (결제 대기, 발송 대기, 발송 시작, 발송 취소)
|
||
@Nested | ||
@DisplayName("When: 상품 등록 요청 시, ") | ||
internal inner class when_store_create_request_is_received { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
상품명이 빈값이면 예외처리
도 추가하면 좋을 것 같습니다.
internal inner class when_store_create_is_completed { | ||
|
||
@Test | ||
@DisplayName("Then: 상품은 '판매 대기' 혹은 '판매중' 상태이다.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
상품을 등록하고 나서 상태가 바로 판매 대기
가 될지 판매중
이 될지 팀끼리 의논해서 결정하여 테스트 진행하면 좋을 것 같습니다.
|
||
@Nested | ||
@DisplayName("When: 조건을 전달할 시, ") | ||
internal inner class when_condition_is_provided { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
검색어의 최대 길이 제한 케이스도 생각 해 볼 수 있을것 같아요
class CartItemRegisterServiceTest { | ||
|
||
@Nested | ||
@DisplayName("When: A, B 판매자의 상품의 재고가 각각 5개일 떄,") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A,B 판매자의 상품의 재고가 0인 케이스도 생각해보면 좋을 것 같습니다
안녕하세요 동걸님. 짧은 기간내에 테스트 케이스를 상세하게 주셔서 깜짝 놀랐습니다. 수고 많으셨습니다. 도메인에서 기능별로 클래스를 나누어 셔서 저는 도메인 별로 구성된 테스트 케이스를 이해하는데 편했습니다. 테스트 클래스 안에 inner class를 사용하여 같은 조건에서의 여러개의 테스트를 구현한것도 저는 좋다고 생각합니다. 제가 딱히 수정 드릴 부분은 없고 테스트 케이스에서 몇 가지 추가되었으면 좋겠다 하는 부분에 대해 커멘트 달았습니다. 그리고 제 개인적인 생각으로는 동걸님이 만들어 주신 테스트 케이스에서 추가적으로 추가할 부분 추가하거나 수정하는 식으로 진행하면 될 것 같습니다. |
📌 요약
📕 무엇을 작업하셨나요?
📖 변경사항
✅ 체크리스트