23.10.12

1. 리팩토링 미션 1단계를 완성해서 제출했다

https://github.com/woowacourse/jwp-refactoring/pull/510

[레거시 코드 리팩토링 미션 - 1단계] 오잉(이하늘) 미션 제출합니다. by hanueleee · Pull Request #510 ·

ㅋㅋ 오늘까지인줄 모르고 플젝 하다가..

6시 제출인데 1시반?2시?엔가 시작해서.. 4시간의 전사로 완성 ^^

ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ쩝 그래서인가 코드가 좀 부끄럽다..

 

2. [펀잇] 비동기 이벤트에 대한 테스트 코드를 작성했다

나름 열심히 찾아보고 쓰긴 했는데.. 이렇게 하는게 맞는건지는 모르겠다 ><

일단 작성한 코드는 이곳에 있다

 

이벤트 테스트 

- @RecordApplicationEvents와 ApplicationEvents로 이벤트가 몇번 발생했는지 셀 수 있다

비동기 테스트

- 방법1) 비동기 로직을 테스트코드에서 대기

- 방법2) 테스트 코드에서만 동기로 작동하게

이벤트 리스너 테스트 (@RecordApplicationEvents)

spring 비동기 테스트 코드 작성법

허브 블로그 비동기 처리 관련 테스트

여우네 이벤트 테스트 코드

 

관련

- 리뷰 삭제 성공시 204 반환 (오늘 아침에 있던 이슈처럼 204처리를 따로 해야하나? 근데 GET일 때만 프론트에서 따로 처리필요하다했으니 괜찮을 것 같기도)

- 이미지 삭제 실패시 처리 추후에 이슈 새로 파서 보완예정 (db에 저장해두고 나중에 스케줄러로 삭제?)

- updateProductImage도 이벤트 처리하는거 어떨까 (productService의 역할이 아닐까)

- 패키지 위치??