[Fix]: 챌린지 그룹 업데이트 코드 엔티티 책임으로 수정

[kwonssshyeon]

🔎 작업 내용

챌린지 업데이트 챌린지 그룹 엔티티에서 하도록 수정

체크 리스트

• 테스트를 작성했습니다.
• 테스트를 통과했습니다.
• API 변경사항이 존재합니다.
• API 호출을 직접 실시하였고, 해당 데이터가 정상적으로 표시됩니다.
• 기존 코드에 영향을 주는 작업 내용이 존재합니다.
• 향후 추가적인 작업이 필요한 부분이 있습니다.

➕ 관련 이슈

• close 어드민 api 구현 #79

[github-actions]

Unit Test Results

33 tests   33 ✔️  2s ⏱️
11 suites    0 💤
11 files      0 ❌

Results for commit 082c28c.

♻️ This comment has been updated with latest results.

[kwonssshyeon]

챌린지 그룹 -> 업데이트 여부 확인(challenge.updateChallengeIfPresent()) 후 없으면 remove
챌린지 -> 일치하는 챌린지 Id 있으면 업데이트 후 결과 T/F 반환

1. 두 클래스에 메소드 나눠썻는데 괜찮나요 .??
   (메소드 두개가 연결..? 서로 의존적..?이라 해야하나 암무튼 같은 클래스에 두는게 더 나을것 같기도 해서 ,,,)

2. 챌린지에 있는 update는 다른데서도 쓸 수 있을 것 같아서 public으로 뒀습니다 !

[bayy1216]

코드가 요구사항을 만족하기때문에 approve합니다
객체의 책임과 도메인모델에 대한것은 팀 컨벤션을 정하지 않았기 때문에 정답이 없는거 같아요