WISH-441 - src/event-handlers/ refactoring && fix failing tests

[ChoiWheatley]

이벤트 핸들러 모듈 리팩토링 및 실패하는 테스트 수정

변경 사항:

1. 이벤트 핸들러 모듈 구조 개선

   • src/event-handlers 내부의 이벤트 핸들러들을 각 도메인별 하위 디렉토리로 이동 (deposit/, donation/, provisional-donation/)

2. Funding Entity 테스트 코드 개선

   • isClosed 메서드의 테스트에서 과거 날짜를 설정하는 방식 변경
     • 기존: pastDate.setDate(pastDate.getDate() - 1);
     • 변경: new Date('1000-01-01') 사용 → 더욱 명확한 과거 날짜 지정
     • ⚠️ 이유는 모르겠는데 오늘 전체 테스트 하면서 갑자기 안되더라구요. 일단 먼 과거로 보냈을 땐 성공하는데 어제는 안되어야 하는데 이거 관련해서 문제 해결해야 할 것 같습니다.

3. Deposit 및 Donation 관련 이벤트 핸들러 리팩토링

   • DepositDeleteSaga, DepositEventHandler, DonationEventHandler, ProvisionalDonationEventHandler 등 이벤트 핸들러의 의존성 및 모듈 import 정리
   • 이벤트 핸들러에서 NotificationService 호출 횟수 조정 (DonationEventHandler의 Admin 알림 제거)
   • DeleteDepositUseCase 및 DecreaseFundSumUseCase 실행 부분을 주석 처리하고 관련 로직을 DeleteDepositSaga로 이동

4. 테스트 코드 리팩토링 및 버그 수정

   • DepositDeleteSaga 및 관련 테스트에서 불필요한 import 정리
   • MatchDepositUseCase에서 deposit.transition을 Mock 처리하여 테스트 안정성 강화
   • image-query.spec.ts에서 TypeORM query 이벤트 리스너 관련 컴파일 오류 발생 코드 주석 처리 (!FIXME 추가)

변경 이유:

• 모듈 구조를 개선하여 각 도메인의 이벤트 핸들러가 명확하게 구분되도록 정리
• isClosed 테스트 개선을 통해 가독성을 높이고 명확한 테스트를 수행
• 이벤트 핸들러 내 의존성 관리를 최적화하고 불필요한 의존성을 제거
• 기존 테스트 코드에서 발생할 수 있는 컴파일 오류를 방지하고 유지보수성을 높임

검토 요청:

• 이벤트 핸들러 모듈 분리 방식이 적절한지 확인 부탁드립니다.
• DonationEventHandler에서 Admin 알림 제거가 비즈니스 로직상 문제가 없는지 검토 바랍니다.
• funding.entity.spec: ⚠️ 이유는 모르겠는데 오늘 전체 테스트 하면서 갑자기 안되더라구요. 일단 먼 과거로 보냈을 땐 성공하는데 어제는 안되어야 하는데 이거 관련해서 문제 해결해야 할 것 같습니다.

[coding-jjun]

LGTM