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

feat: 판매자는 거래 내역 상세 조회시, 자신의 경매 상품의 거래내역을 조회할 수 있다. #132

Merged
merged 5 commits into from
Aug 14, 2024

Conversation

chhs2131
Copy link
Collaborator

@chhs2131 chhs2131 commented Aug 14, 2024

📄 Summary

  • 테스트 추가
    • 거래 내역 단건 조회 성공 케이스를 '구매자 입장으로 조회', '판매자 입장으로 조회' 모두 검증한다.
    • 판매자의 거래 내역을 다른 판매자가 조회하는 경우 예외를 검증한다.
  • 테스트 수정
    • fix: AuctionServiceTest의 경매 입찰 성공 케이스의 주문 수량을 수정한다.
    • chore: BidHistoryInfoTest 테스트 시 출력되는 이름 수정

🙋🏻 More

close #117

- 거래 내역 단건 조회 성공 케이스를 '구매자', '판매자' 모두 검증한다.
- 판매자의 거래 내역을 다른 판매자가 조회하는 경우 예외를 검증한다.
@chhs2131 chhs2131 added the FEATURE 새로운 기능 또는 변경 label Aug 14, 2024
@chhs2131 chhs2131 requested a review from a team August 14, 2024 02:26
@chhs2131 chhs2131 self-assigned this Aug 14, 2024
yudonggeun
yudonggeun previously approved these changes Aug 14, 2024
Copy link
Contributor

@yudonggeun yudonggeun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍 👍 👍 👍 👍 👍 👍 👍 👍

Copy link
Collaborator

@minseok-oh minseok-oh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 😃

Comment on lines -33 to +53
@Test
void 성공하는_경우_BidHistoryInfo를_반환받는다() {
static Stream<Arguments> provideMembersForSuccess() {
Member seller = Member.builder().id(1L).signInId("판매자").role(Role.SELLER).build(); // 소유자
Member buyer = Member.builder().id(2L).signInId("구매자").role(Role.BUYER).build(); // 소유자

return Stream.of(
Arguments.of(seller, "판매자의 구매이력 조회"),
Arguments.of(buyer, "구매자의 구매이력 조회")
);
}

@ParameterizedTest(name = "{1} 시 성공한다")
@MethodSource("provideMembersForSuccess")
void 소유자가_거래내역_조회시_성공한다(Member member, String description) {
// given
Member seller = Member.builder().id(1L).signInId("판매자").build(); // 소유자
Member buyer = Member.builder().id(2L).signInId("구매자").build(); // 소유자
Member seller = Member.builder().id(1L).signInId("판매자").role(Role.SELLER).build(); // 소유자
Member buyer = Member.builder().id(2L).signInId("구매자").role(Role.BUYER).build(); // 소유자
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

질문: 뒤에 소유자라는 주석은 어떤 의미인가요?!

@minseok-oh minseok-oh merged commit 4ec61a1 into dev Aug 14, 2024
@chhs2131 chhs2131 deleted the feat/117 branch August 14, 2024 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FEATURE 새로운 기능 또는 변경
Projects
None yet
3 participants