woowacourse-teams / 2023-emmsale

18 stars 2 forks source link

Feature/#901 온보딩에서 활동 설정 시 관심태그도 함께 설정 #903

Open amaran-th opened 7 months ago

amaran-th commented 7 months ago

️⃣ 연관된 이슈

close : #901

📝 작업 내용

온보딩에서 직무 활동을 설정할 때 직무활동 name과 같은 이름을 갖는 태그가 관심태그로 설정되도록 했습니다.

예상 소요 시간 및 실제 소요 시간 (일 / 시간 / 분)

예상 소요 시간 : 1/24(수) 실제 소요 시간 : 1/24(수)

💬 리뷰어 요구사항 (선택)

생각보다 건드린 부분이 많네요...! 먼저 기존에는 api가 MemberActivityCommandService의 메서드를 호출하고 있었는데, 이 서비스에서 InterestTag 도메인에 대한 작업을 수행하는 게 부자연스러운 것 같아서 로직 흐름 구조를 아래 사진처럼 변경해주었습니다. 그리고 추가 테스트를 작성하는 과정에서, 온보딩을 하지 않았고 활동, 관심태그가 설정되지 않은 멤버가 필요해서 data-test.sql에 회원 한 명을 추가했습니다. 이로 인해 기존에 memberId 3을 사용하던 몇몇 테스트가 깨져서 그 코드도 수정을 가한 상태입니다.