woowacourse-teams / 2023-festa-go

🎪 페스타고, 대학 축제를 더욱 즐겁게!
71 stars 8 forks source link

[BE] refactor: CreateCommand, UpdateCommand 클래스 빌더 패턴 적용 및 테스트 코드 적용 (#940) #943

Closed seokjin8678 closed 5 months ago

seokjin8678 commented 6 months ago

📌 관련 이슈

✨ PR 세부 내용

CommandService에서 사용하는 CreateCommand, UpdateCommand 클래스에 빌더 패턴을 적용하고 사용했습니다.

또한 테스트 코드에서도 생성자를 호출하는 코드도 전부 빌더를 사용하도록 변경했습니다.

record를 사용하면 필드 순서에 따라 의도치 않게 생성자 파라미터의 위치가 바뀌게 되니, 알아차리기 힘든 버그가 생기네요 😂

게다가 #938 에서 발생한 문제는 SocialMediaCommandServiceTest가 있어도 해당 버그를 잡아내지 못한 것이, Request에서 Command를 반환하기 때문에 실제 환경에 가서야 버그를 확인할 수 있었네요. 😂

앞으로 빌더 패턴을 적극적으로 사용해야 할 것 같습니다.

github-actions[bot] commented 6 months ago

Test Results

226 files  226 suites   27s :stopwatch: 751 tests 751 :white_check_mark: 0 :zzz: 0 :x: 770 runs  770 :white_check_mark: 0 :zzz: 0 :x:

Results for commit ab49b97c.

:recycle: This comment has been updated with latest results.

seokjin8678 commented 5 months ago

리뷰 기간이 지났기에 머지 처리 하도록 하겠습니다!