-
Notifications
You must be signed in to change notification settings - Fork 5
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
구매 입찰 취소 기능 테스트 추가 #241
구매 입찰 취소 기능 테스트 추가 #241
Conversation
Test Results95 tests 85 ✅ 4s ⏱️ Results for commit a9c7fef. ♻️ This comment has been updated with latest results. |
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.
코드양이나 테스트 케이스 퀄리티를 보아하니
상당한 고민 끝에 작성하신 게 드러나네욥!!
고생 많으셨습니다!! 👍👏
int start = (int) pageable.getOffset(); | ||
int end = Math.min((start + pageable.getPageSize()), responseDtoList.size()); | ||
Page<BuyPriceToQuantityResponseDto> responseDtoPage = new PageImpl<>(responseDtoList.subList(start, end), pageable, | ||
responseDtoList.size()); | ||
|
||
given(buyRepository.findAllPriceToQuantityOf(any(Product.class), any(Pageable.class))).willReturn(responseDtoPage); | ||
// when | ||
Page<BuyPriceToQuantityResponseDto> returnPage = buyQueryService.findAllBuyBidsOf(TEST_PRODUCT, pageable); |
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.
요부분 mockito 로 하시기 쉽지 않으셨을텐데 고생하셨네요 👍
.buyId((long) i) | ||
.price(1000L * i) | ||
.discountPrice(1000L * i) | ||
.coupon((i % 2 == 0) ? TEST_COUPON_FIX : null) |
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.
어떤 건 쿠폰주고 어떤 건 쿠폰 안 주게끔 하신걸까요??
구매 관련 조회 테스트 추가
즉시 구매 테스트 코드 작성
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.
수고하셨습니다! 👍
개요
구매 입찰 취소 기능 테스트 추가
작업사항
관련 이슈