[FEATURE] : [Auction 도메인 필터 기반 검색 기능 추가]#44
Merged
polyglot-k merged 33 commits intoTaskSprints:developfrom Sep 13, 2024
Merged
Conversation
1. 경매 종료가 임박한 진행중인 경매 목록 조회 추가 미흡 : 성능 개선 못함, /auction으로 엔드포인트 합쳐야 하는데 못했음, 테스트쪽 리뷰 받아야 함.
코드 포맷터 빌드
1. 마감이 얼마 안 남은 경매 목록 조회 메서드의 파라미터가 없던 것에서 now, next24Hours 추가
1. 테스트 케이스의 LocalDateTime을 고정 2. 테스트 대상이 아닌 파라미터는 테스트 픽스처에 넣음 3. assertAll로 컬렉션인 경우 끝까지 assertion하고 결과를 보여주도록 함
1. createAuction 중복 코드 리스트로 수정 2. LocalDateTime.now()를 고정 시간으로 수정 3. AuctionServiceImplTest 중복 코드 삭제
1: /auction에서 검색 조건별로 경매 목록을 조회한다. 미흡 : 파라미터를 SearchCondition이라는 DTO로 받아서 기존 테스트 다 깨져서 수정해야 함, 변경점에 대한 테스트 코드 없음.
[FEATURE] : [경매 종료가 임박한 진행중인 경매 목록 조회]
- 고정 Clock 이 아닌 경우 TZ로 인해서 오류 발생 가능
7 tasks
polyglot-k
changed the title
thornscript
reviewed
Sep 10, 2024
polyglot-k
changed the title
- 불필요한 함수 제거
- 추 후, 자동화 및 강제화 할 것임.
- 추가 enum을 통해 안정성있게 변경
- 추가 enum을 통해 안정성있게 변경
na0th
reviewed
Sep 13, 2024
na0th
reviewed
Sep 13, 2024
Comment on lines
+60
to
+69
| private OrderSpecifier<?> getSortOrder(AuctionRequest.SearchCondition condition, QAuction auction) { | ||
| if (condition.getSortBy() != null) { | ||
| return switch (condition.getSortBy()) { | ||
| case "bidsAsc" -> auction.bids.size().asc(); | ||
| case "bidsDesc" -> auction.bids.size().desc(); | ||
| case "endingSoon" -> auction.endTime.asc(); | ||
| default -> null; | ||
| }; | ||
| } | ||
| return null; |
Contributor
There was a problem hiding this comment.
정렬 선택을 안했을 때, 기본 정렬을 정해두는 건 어떻게 생각하시나요??
Member
Author
There was a problem hiding this comment.
그 부분을 어떤걸 기본으로 할지 고민 중에 있어서 일단 설정을 안했습니다
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
name: Pull Request
about: '풀 리퀘스트를 제출합니다.'
title: "[FEATURE] - [기능명] 또는 [FIX] - [버그 설명]"
labels: ''
assignees: ''
진행 중인 사항
🆕 기능 추가 / 🔧 버그 수정
📋 변경 사항
@RequestBody가 아니라QueryString을 통해서 접근하도록 변경OrderSpecifier을 통해서 정렬 기준 설정하여 QueryDSL 진행🔍 테스트 사항
📄 관련 문서
📝 추가 사항