Skip to content

Conversation

@loseminho
Copy link
Collaborator

@loseminho loseminho commented Sep 26, 2025

📌 개요

  • 코드 리뷰로 발견한 불안정한 안정성과 동시성, N+1문제 등을 개선
  • 테스트 코드 생성

🔨 작업 내용

  • DTO 분할 // 타입 안정성을 위해 컨트롤러에서 맵 대신 DTO를 사용
  • 예외 처리 로직 추가 (종료된 방 활성화 불가, 일시정지 불가 등)
  • 유동적으로 하트비트 받을 수 있도록 수정
  • 코드 통일성을 위해 Repository 쿼리DSL 형식으로 수정
  • 룸에 대한 설정값을 외부 설정으로 분리 하여 application.yml의 설정 값으로 사용
  • 비관적 락을 이용한 동시성 제어
  • 단위 테스트 형식의 테스트 코드 생성
  • 스웨거 어노테이션 추가
  • 문서와 다른 api 경로 수정

🔗 관련 이슈

Closes #56

📝 참고 사항

기초 테스트 코드 작성을 성공 한 뒤, 인증 인가나 별도 기능이 업데이트 되어
기존 코드와 테스트 코드에 기초적으로 연결 해볼까? 하다가 연결 못해서 PR이 늦었습니다.
이후 작업부턴 욕심 없이 작은 단위 기능 구현으로 진행 해 올리겠습니다. ..ㅠㅠ
별도 프로젝트에서 테스트 후, 해당 기능을 단위 별로 프로젝트에 합치고 로컬에서 통과 확인 했습니다.

✅ 체크리스트

  • 기능 동작 확인
  • 테스트 코드 작성
  • 문서/주석 추가 및 최신화

@loseminho loseminho self-assigned this Sep 26, 2025
@github-actions github-actions bot changed the title Feat: 스터디룸 로직 보완 및 테스트 코드 작성 Feat: 스터디룸 로직 보완 및 테스트 코드 작성 (#56) Sep 26, 2025
@loseminho loseminho merged commit e6b5fc6 into dev Sep 26, 2025
2 checks passed
@github-actions github-actions bot deleted the feat/56 branch September 26, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants