fix/#66

[songhaechan]

#️⃣연관된 이슈

ex) #66

📝작업 내용

그룹 호스트인지에 따라서 프론트 UI가 변경돼야하기 때문에 서버 측에서 식별필드 추가
참여 요청을 보내고 아직 수락되지 않은 참여자를 식별하는 필드 추가
그룹 내역 조회에서 활성화그룹을 비활성화그룹으로 잘못 응답이 내려가는 중 (isBefore(), isAfter() 함수가 뒤바뀌어있음)
GroupParticipation의 이름에 의미가 잘 드러나지 않아서 도메인 이름 변경 -> ParticipationRequest (그룹 참여 요청자들을 위한 도메인)
그룹 참여 제한 설정 (한 번에 5개 이상의 모각코 활동을 제한)

💬리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요.

[ohksj77]

고생하셨습니다. 궁금한 점들 리뷰로 남겨두었으며, 리뷰 범위 외로도 코드를 보다 궁금해진 점들을 여기에 남겨봅니다.

1. Group이 boolean값을 반환하는 것까지가 아닌, 적합하지 않은 경우 예외를 던지는 로직까지 Group의 책임 아닐까요? 지금은 하나의 기능에 대한 책임이 여기저기에 퍼져있는 느낌입니다. 어떻게 생각하시는지 궁금합니다!
   

2. group-locations 가 이 컨트롤러가 다루는 자원으로 보이는데, groups가 컨트롤러의 공통 경로인 이유가 궁금합니다! groups가 공통 경로인 다른 컨트롤러도 존재하는 것으로 보여서요!
   

[songhaechan]

고생하셨습니다. 궁금한 점들 리뷰로 남겨두었으며, 리뷰 범위 외로도 코드를 보다 궁금해진 점들을 여기에 남겨봅니다.

1. Group이 boolean값을 반환하는 것까지가 아닌, 적합하지 않은 경우 예외를 던지는 로직까지 Group의 책임 아닐까요? 지금은 하나의 기능에 대한 책임이 여기저기에 퍼져있는 느낌입니다. 어떻게 생각하시는지 궁금합니다!
   
2. group-locations 가 이 컨트롤러가 다루는 자원으로 보이는데, groups가 컨트롤러의 공통 경로인 이유가 궁금합니다! groups가 공통 경로인 다른 컨트롤러도 존재하는 것으로 보여서요!
   

1. 좋은 지적 감사합니다. 확인을 해보니 중복된로직이 여기저기 퍼져있었네요. isNotGroupHost()의 경우 내부에서 예외까지 던지도록 했고 네이밍은 validateGroupHost로 변경했습니다. isGroupHost()의 경우엔 실제 boolean 필드자체를 응답해야해서 그대로 두었습니다.

2. group-locations로 url을 변경했습니다!

[github-actions]

📝 테스트 커버리지 리포트입니다

Overall Project	65.75% -3.48%	🍏
Files changed	65.87%	🍏

File	Coverage
GroupApplicationService.java	100%	🍏
GroupLocationController.java	100%	🍏
GroupController.java	100%	🍏
GroupApplication.java	100%	🍏
GroupApplicationStatus.java	100%	🍏
GroupApplicationController.java	100%	🍏
S2CellSearch.java	100%	🍏
Participant.java	88.66% -7.22%	❌
GroupApplicationValidator.java	88.57% -11.43%	🍏
Member.java	88.06% -2.99%	❌
Group.java	74.8% -25.2%	❌
GroupService.java	50% -36.96%	❌
JwtService.java	0% -7.46%	❌

[ohksj77]

고생하셨습니다.