Skip to content

Wish 430 create mock funding with gifts#233

Merged
coding-jjun merged 4 commits intomainfrom
WISH-430-createMockFundingWithGifts
Feb 6, 2025
Merged

Wish 430 create mock funding with gifts#233
coding-jjun merged 4 commits intomainfrom
WISH-430-createMockFundingWithGifts

Conversation

@ChoiWheatley
Copy link
Collaborator

Gift 엔터티의 관계 추가 및 관련 테스트 코드 반영

변경 사항

  1. Gift 엔터티에 OneToOne 관계 추가:

    • Gift 엔터티에 image: Image 필드를 추가하여 추후 ImageInstanceManager에서 활용될 수 있도록 만듦.
  2. Deposit 기능의 E2E 테스트 수정:

    • giftRepo를 추가하여 Gift 관련 데이터를 관리할 수 있도록 수정
    • mockFunding.gifts가 정상적으로 생성되었는지 검증하는 테스트 케이스 추가
  3. Mock 데이터 생성 관련 수정:

    • createMockGift 함수를 추가하여 Gift 객체를 생성할 수 있도록 변경
    • createMockFundingWithRelations 함수에서 Gift 데이터를 생성할 수 있도록 로직 추가
    • CreateMockFundingWithRelationsDelegate 인터페이스에 giftRepo를 추가

영향 범위

  • Deposit 관련 E2E 테스트에서 Gift가 생성되는지 확인하는 검증이 추가됨
  • Mock 데이터 생성 시 Gift 객체도 함께 생성할 수 있도록 변경됨

테스트

  • Deposit API E2E Test가 정상적으로 동작하는지 확인
  • mock-factory.tscreateMockGiftcreateMockFundingWithRelations 함수가 정상적으로 실행되는지 확인

추가 고려 사항

  • Gift 엔터티에 추가된 OneToOne 관계가 기존 데이터와 충돌하지 않는지 확인 필요
  • Gift가 생성될 때 Image와의 관계가 올바르게 매핑되는지 검증하는 추가 테스트 필요

리뷰 및 피드백 부탁드립니다! 😊

Copy link
Owner

@coding-jjun coding-jjun left a comment

Choose a reason for hiding this comment

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

LGTM

@coding-jjun coding-jjun merged commit fd3825a into main Feb 6, 2025
@ChoiWheatley ChoiWheatley deleted the WISH-430-createMockFundingWithGifts branch February 9, 2025 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants