feat: 경매는 진행 중 상태가 아니면 사용자의 요청을 처리할 수 없다. #85
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 진행 중인 경매가 아닌 경우 A016 에러를 반환한다. - 경매 상태 비교를 Service에서 하지 않고, 경매상태(AuctionStatus)에게 직접 물어보도록 수정한다. - 경매 상태에 의존적인 '입찰 성공' 테스트를 일시적으로 Disabled 한다.
…uction-bid-check-auction-progress # Conflicts: # src/main/java/com/wootecam/luckyvickyauction/core/auction/domain/AuctionStatus.java # src/main/java/com/wootecam/luckyvickyauction/core/auction/dto/AuctionInfo.java # src/main/java/com/wootecam/luckyvickyauction/core/auction/service/AuctionService.java # src/main/java/com/wootecam/luckyvickyauction/global/exception/ErrorCode.java
- TODO 임시로직: 추후 스케줄러 등 믿을 수 있는 경매 관리자가 상태를 업데이트 하도록 수정 필요 - 현재는 현재 시간 기준과 재고를 기준으로 판단하는 로직을 활용한다.
- 진행 중인 경매가 아닌 경우 A016 에러를 반환한다. - 경매 상태 비교를 Service에서 하지 않고, 경매상태(AuctionStatus)에게 직접 물어보도록 수정한다.
- 전달되는 AuctionStatus가 Null인 경우 기본값으로 AuctionStatus.WAITING을 사용한다.
- AuctionRepository 인터페이스를 구현하여 활용한다.
yudonggeun
previously approved these changes
Aug 13, 2024
Comment on lines
49
to
53
| if (auctionStatus == null) { | ||
| this.status = AuctionStatus.WAITING; | ||
| } else { | ||
| this.status = auctionStatus; | ||
| } |
There was a problem hiding this comment.
초기에 auctionStatus를 의도하지 않은 상태를 주입한 경우를 대비한 검증 로직이 있으면 좋을 것 같아요!
There was a problem hiding this comment.
좋습니당ㅎㅎ 관련해서 고민하던 사항인데용.
Auction에서는 외부에서 주입(e.g. 스케줄러)해주는 값을 믿고 가는 느낌으로 구성했는데, 여기에서도 같이 검증해주는게 좋을까요?
There was a problem hiding this comment.
주입을 못받게 생성자의 AuctionStatus 인자를 지워버렸습니다 !
|
|
||
| // then | ||
| // expect | ||
| System.out.println(auction); |
Comment on lines
-17
to
+18
| private int maximumPurchaseLimitCount; | ||
| private long maximumPurchaseLimitCount; |
…uction-bid-check-auction-progress
- AuctionStatus는 WAITING을 기본값으로 가진다. - Auction의 ID는 setter를 통해 지정할 수 있도록 수정한다.
yudonggeun
approved these changes
Aug 13, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📄 Summary
🙋🏻 More
close #63