[feat] : 경매글 등록 API 구현 #23

hyun2371 · 2024-02-17T13:28:13Z

close #22

📑 작업 상세 내용

AuctionController 추가
- controller, service DTO 각각 사용
- valid 관련 어노테이션 추가
AuctionService 추가
- AuctionMapper 추가
예외 처리 추가
- GlobalExceptionHandler에 NotFoundException 관련 처리 함수 추가
- AuctionErrorCode 추가
- 엔티티 생성자 내부에 Assert() 추가
Enum 내 String 받고 Enum 변환하는 함수 추가
엔티티 수정
- 아래 사항은 노션 논의사항에 자세히 적어둠
- Auction.product에 cascade 옵션 추가
- ProductCategory.categoryValue 타입 Enum -> String으로 변경

💫 작업 요약

경매글 등록 API 구현 (사진 필드는 추후 구현 예정)

🔍 중점적으로 리뷰 할 부분

예외 처리 로직
엔티티 수정 사항
AuctionMapper

github-actions · 2024-02-17T13:29:37Z

Test Results

3 tests +2 3 ✅ +2 1s ⏱️ -1s
3 suites +2 0 💤 ±0
3 files +2 0 ❌ ±0

Results for commit a8f15ae. ± Comparison against base commit e7c8c1d.

♻️ This comment has been updated with latest results.

ParkJuhan94 · 2024-02-18T11:29:27Z

경매글 등록 작업이 많네요 고생하셨습니다 !
리뷰 할게요~

hyun2371 · 2024-02-18T12:56:18Z

추가 작업 내용

오류 해결

api 모듈에서 testContainerSupport 및 fixture 사용 불가 문제 해결

java-test-fixtures 플러그인 추가
위 파일들 해당 디렉토리로 넣음

@SpringBootTest에서 @SpringBootConfiguration 못찾음

api 모듈에 plain.jar 생성하도록 api 모듈 build.gradle 수정

테스트 추가

controller test 추가
- ApiTestSupport 추가
service test 추가
testFixture 추가

통합 테스트 돌리니 403 에러 떠서 build.gradle에 security 관련 의존성 주석 처리했습니다.

core/src/main/java/dev/handsup/auction/domain/auction_field/PurchaseTime.java

ParkJuhan94 · 2024-02-19T00:49:06Z

api/build.gradle

@@ -12,6 +13,6 @@ dependencies {
    }

    tasks.named('jar') {
-        enabled = false
+        enabled = true


enabled = false 로 되어있으면 jar 파일이 안 생겼나요? 바꾸신 이유가 궁금합니다

bootJar enabled하면 jar가 생기고, jar enabled하면 plain.jar가 생깁니다. 멀티 모듈일 때는 plain.jar 필요해서 true로 설정했습니다.

api/src/test/java/dev/handsup/auction/controller/AuctionControllerTest.java

ParkJuhan94 · 2024-02-19T01:12:49Z

api/src/test/java/dev/handsup/auction/controller/AuctionControllerTest.java

+			.andExpect(jsonPath("$.si").isEmpty())
+			.andExpect(jsonPath("$.gu").isEmpty())
+			.andExpect(jsonPath("$.dong").isEmpty());
+	}


경매글 등록에 실패하면 예외를 던지는 테스트는 필요없을까요?

추가하겠습니다~

ParkJuhan94 · 2024-02-19T01:16:40Z

core/src/main/java/dev/handsup/auction/domain/Auction.java

 	@Id
 	@GeneratedValue(strategy = IDENTITY)
 	@Column(name = "auction_id")
 	private Long id;

 	@ManyToOne(fetch = LAZY)
 	@JoinColumn(name = "seller_id",
-		nullable = false,
+		// nullable = false,


판매자가 null 이 될 수가 있나요??

아직 유저 쪽 구현이 안되어서 일단 null 허용하였습니다.

ParkJuhan94 · 2024-02-19T01:22:32Z

core/src/main/java/dev/handsup/auction/domain/Auction.java

@@ -81,9 +90,10 @@ public class Auction extends TimeBaseEntity {
 	private int bookmarkCount;

 	@Builder
-	public Auction(User seller, String title, Product product, int initPrice, LocalDate endDate,


판매자가 null 이 될 수가 있나요?? 라는 질문과 동일한 부분입니다

ParkJuhan94 · 2024-02-19T01:28:22Z

core/src/main/java/dev/handsup/auction/domain/product/product_category/ProductCategory.java

+		this.categoryValue = categoryValue;
+	}
+
+	public static ProductCategory of(String categoryValue) {


다른 엔티티와 다르게 ProductCategory 에는 빌더를 private 로 막고
팩토리메서드를 쓰신 이유가 있을까요~?

Auction, Product에도 팩토리 메서드 추가하겠습니다!

ParkJuhan94 · 2024-02-19T01:32:50Z

core/src/main/java/dev/handsup/auction/dto/RegisterAuctionRequest.java

+import lombok.Builder;
+
+@Builder
+public record RegisterAuctionRequest(


RegisterAuctionApiRequest 와 요청하는 필드가 똑같은데,
dto 를 분리한 이점이 있는걸까요~?

service가 controller에 의존하지 않게 하고자 dto를 분리하였습니다~ 예전에 슬랙에 보내드린 링크 참고해주시면 될 것 같아요

core/src/main/java/dev/handsup/auction/exception/AuctionErrorCode.java

hyun2371 added 15 commits February 16, 2024 20:16

[feat] : 거래희망 장소 필드 nullable true로 변경

a8e62d6

[feat] : Enum <-> String 간 변환 메서드 추가

b989776

[feat] : 상품 카테고리 필드명 변경 및 생성자 추가

f07469f

[feat] : 경매 에러코드 추가

8cf6dcd

[feat] : 경매 생성 controller 코드 작성 및 request 추가"

607813f

[feat] : Enum 관련 에러 코드 추가

5f0bc83

[Style] : Enum 변수명 변경

0fb8339

[feat] : service dto 및 mapper 추가

2cd4787

[feat] : 전역 예외 처리 NotFoundException 처리 함수 추가

4405285

[feat] auctionService 추가 및 관련 repository 추가

128bc55

[feat] : auctionErrorCode 추가 및 불필요한 코드 삭제

76de873

[feat] : @ valid, @ EnableJpaAuditing 어노테이션 추가

177382f

[feat] : 카테고리 값 Enum -> String 타입 변경

3e0406f

[refactor] : 코드 리포멧팅

76fc961

[feat] : dto validation 어노테이션 추가

a7389ae

hyun2371 added the feat New feature or request label Feb 17, 2024

hyun2371 requested a review from ParkJuhan94 February 17, 2024 13:28

hyun2371 self-assigned this Feb 17, 2024

hyun2371 linked an issue Feb 17, 2024 that may be closed by this pull request

[feat] : 경매글 등록 API 구현 #22

Closed

2 tasks

hyun2371 added 5 commits February 18, 2024 16:53

[feat] auction, product test fixture 추가

92d604f

[chore] : testFixtures 추가해 모듈간 테스트 코드 중복 줄이기

443efd8

[feat] : 경매글 등록 서비스 테스트 코드 추가

9857e40

[fix] : springboottest 컴포넌트 스캔 에러 해결

7945f68

[feat] controller 테스트용 support 추가

9622d47

hyun2371 added 4 commits February 18, 2024 20:41

[style] dto 필드명 변경 및 enum 값 변경

d514c4f

[feat] : 경매 등록 통합 테스트 추가

f65ea07

[refactor] : 코드 리포멧팅

2a1169b

[fix] : 403 에러 해결 위해 security 임시 비활성화

0de31e7

[fix] : jar 파일 생성 안되는 에러 해결

a8f15ae

ParkJuhan94 approved these changes Feb 19, 2024

View reviewed changes

ParkJuhan94 merged commit d7c03af into dev Feb 19, 2024
3 checks passed

ParkJuhan94 reviewed Feb 19, 2024

View reviewed changes

core/src/main/java/dev/handsup/auction/exception/AuctionErrorCode.java Show resolved Hide resolved

hyun2371 deleted the feat/#22/auction-post-api branch February 20, 2024 01:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] : 경매글 등록 API 구현 #23

[feat] : 경매글 등록 API 구현 #23

hyun2371 commented Feb 17, 2024 •

edited

Loading

github-actions bot commented Feb 17, 2024 •

edited

Loading

ParkJuhan94 commented Feb 18, 2024

hyun2371 commented Feb 18, 2024 •

edited

Loading

ParkJuhan94 Feb 19, 2024

hyun2371 Feb 19, 2024

ParkJuhan94 Feb 19, 2024

hyun2371 Feb 19, 2024

ParkJuhan94 Feb 19, 2024

hyun2371 Feb 19, 2024

ParkJuhan94 Feb 19, 2024

ParkJuhan94 Feb 19, 2024

hyun2371 Feb 19, 2024 •

edited

Loading

ParkJuhan94 Feb 19, 2024

hyun2371 Feb 19, 2024

[feat] : 경매글 등록 API 구현 #23

[feat] : 경매글 등록 API 구현 #23

Conversation

hyun2371 commented Feb 17, 2024 • edited Loading

📑 작업 상세 내용

💫 작업 요약

🔍 중점적으로 리뷰 할 부분

github-actions bot commented Feb 17, 2024 • edited Loading

Test Results

ParkJuhan94 commented Feb 18, 2024

hyun2371 commented Feb 18, 2024 • edited Loading

추가 작업 내용

오류 해결

테스트 추가

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyun2371 Feb 19, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyun2371 commented Feb 17, 2024 •

edited

Loading

github-actions bot commented Feb 17, 2024 •

edited

Loading

hyun2371 commented Feb 18, 2024 •

edited

Loading

hyun2371 Feb 19, 2024 •

edited

Loading