🐛 Fix: completedLessons를 완료 기준으로 집계 (진행률과 일관)

[naooung]

연결 Issue

closes #

작업 유형

• 🐛 Fix — 버그 수정

변경 내용

과외 카드에 "강의 1/5" + "0%" 처럼 진행률과 완료 강의 수가 모순되게 표시되던 문제를 수정한다.

원인

GET /tutoring(목록·단건)의 completedLessons 집계가 (isCompleted = true OR currentSlideIdx > 0) 즉 진행중인 강의까지 세고 있었다. 강의를 열기만 하고 끝내지 않으면(currentSlideIdx>0, isCompleted=false) completedLessons=1로 잡히는데, progress(course.progressPercent)는 완료 기준이라 0% → 모순. (#118에서 집계 의도는 "완료 강의 수"였으나 구현이 진행 포함으로 들어감)

수정 (완료만 집계)

• CourseService: countStudiedChapters(ByCourseIds) → countCompletedChapters(ByCourseIds) 로 호출 전환.
  • 단건: 기존 countCompletedByCourseIdAndUserId(isCompleted=true) 재사용.
  • 목록: countCompletedByUserIdAndCourseIdIn(N+1 방지 IN 쿼리) 신규 추가.
• studied 쿼리(진행 포함)는 향후 "수강중" 표시용으로 보존(메인 코드 미사용, 죽은 코드 아님).
• progress와 completedLessons가 둘 다 완료 기준이 돼 일관됨.

테스트 방법

• ./gradlew test --tests "com.swez.backend.domain.course.service.CourseServiceTest"
  • 기존 completedLessons 검증을 completed mock으로 정정
  • 신규 회귀: 진행중만 있고 완료 0이면 completedLessons == 0
• 전체 ./gradlew test BUILD SUCCESSFUL

체크리스트

• develop 브랜치에서 분기했다
• 브랜치 네이밍 규칙(타입/작업내용)을 따랐다
• 커밋 메시지가 {깃모지} {타입}: {내용} 형식을 따른다
• 관련 문서를 동시에 업데이트했다 (필요 시)
• Merge 후 브랜치 삭제 예정이다