[Refactor] useSession 리팩토링

[yiseungyun]

Note

• 작성자: 이승윤
• 작성 날짜: 2024.11.25

✅ 체크리스트

• 코드가 정상적으로 작동하는지 확인했습니다.
• 주요 변경사항에 대한 설명을 작성했습니다.
• 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

• 미디어 스트림, 피어 커넥션 정리 분리
• reaction 분리
• 소켓 이벤트 코드 분리
• 소켓, 닉네임 없을 때 확인하는 함수 분리
• sessionId 없으면 early return

📝 작업 상세 내역

useSession 리팩토링

• 관련 파일은 hooks/session에 넣었습니다.
• 훅에서 훅을 호출하는게 괜찮은가? 하고 정우님이 질문해주셨는데 이런 경우는 많다고 합니다.
• session 폴더에는 현재
  • useMediaDevices
  • useMediaStreamCleanup
  • usePeerConnectionCleanup
  • useReaction
  • usePeerConnection
  • useSocketEvents
  • useSession
  • 위와 같은 훅이 있고, 기존에 있던 mediadevices나 peerconnection 훅도 useSession에서 호출하기 때문에, 같은 폴더로 옮겼습니다.

📌 테스트 및 검증 결과

• 테스트 코드를 실행했을 때 모두 통과했습니다.
• 테스트 케이스 중 세션 아이디가 없을 때 방을 입장하려는 부분 테스트를 제외했습니다.
  • 세션 아이디가 없으면 early return하도록 수정했기 때문에, 이 부분은 테스트를 지웠습니다.
• 테스트 코드 실행 뿐만 아니라 실제 사이트에서 화상회의가 되는지 테스트도 했습니다.

💬 다음 작업 또는 논의 사항

• 이번 주 태스크 진행

[ShipFriend0516]

셀프 검토요청하고 검토하기 ☃️

리팩토링 수고하셨습니다 테스트코드 있으니 어떤가요? 확실히 장점이 체감이 되는지 궁금합니다

훅 안에서 훅 호출을 해도 되는지 몰랐는데 배워갑니다..