Skip to content

Jacoco 업데이트#101

Merged
LeeSG-0114 merged 1 commit into
developfrom
jaehoon0321-patch-1
Apr 30, 2026
Merged

Jacoco 업데이트#101
LeeSG-0114 merged 1 commit into
developfrom
jaehoon0321-patch-1

Conversation

@jaehoon0321
Copy link
Copy Markdown
Contributor

사진 넣을 예정

💡 PR 제목

예: feat: [KHG] 로그인 API 구현


🔗 관련 이슈

진행 중이면 Refs #123, 완전 해결이면 Closes #123 (완전히 끝났을 때만)

  • Refs/Closes: #

📌 작업 내용 요약

  • 주요 변경 1
  • 주요 변경 2
  • 주요 변경 3

🧱 변경 범위 (Scope)

Backend

  • API 추가/수정
  • Validation/예외처리
  • 인증/인가
  • DB 스키마/쿼리
  • 기타:

Frontend

  • UI/라우팅
  • 상태관리/비동기 로직
  • 입력값/검증
  • 기타:

🧪 테스트 내역

Backend

  • 로컬에서 애플리케이션 실행 확인
  • 단위 테스트 통과
  • API 수동 테스트 (Postman / Swagger 등)

Frontend

  • npm run dev 로컬 실행 확인
  • 주요 화면/기능 수동 테스트

테스트 상세(필수)

테스트한 API/페이지/케이스를 간단히 적어주세요.

  • 예) 로그인 성공/실패, 회원가입, 목록 조회 등

🖼️ 화면 캡처 / 시연 영상 (Frontend 변경 시 권장)

  • before:
  • after:

⚠️ 영향도 / 리스크 / 롤백

리뷰어가 “머지해도 되는지” 판단하기 위한 섹션입니다.

  • 영향도: (예: 기존 API 응답 변경 없음 / 있음)
  • 리스크: (예: 권한 로직 수정, 쿼리 성능, 마이그레이션 필요 등)
  • 롤백 방법: (예: revert 커밋 / 기능 플래그 off / 마이그레이션 롤백 불가 등)

✅ 리뷰어 체크 포인트 (선택)

리뷰어가 집중해서 봐줬으면 하는 부분이 있으면 적어주세요.

  • 예) 트랜잭션 범위, 권한 체크, N+1, 상태관리 로직 등

사진 넣을 예정
Copy link
Copy Markdown
Contributor

@LeeSG-0114 LeeSG-0114 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README.md to include a section on JaCoCo test coverage, documenting the improvement from 53% to 79% coverage. A review comment points out a markdown formatting issue where a preceding code block was not closed, causing the new content to render incorrectly as code, and suggests cleaning up excessive whitespace for better readability.

Comment thread README.md
Comment on lines +235 to 253
## 🤖 JaCoCo 테스트 커버리지

서비스의 안정성을 검증하기 위해 JaCoCo를 사용하여 테스트 커버리지를 측정하고 관리하였습니다.

### 📊 1차 테스트 결과 (초기 단계)
> 초기 단계에서는 핵심 비즈니스 로직(Service) 위주의 테스트를 진행하였습니다.



---

### 🚀 2차 테스트 결과 (현재)
> **전체 커버리지 53% → 79% 달성!**
> 모든 Controller 레이어 테스트 100% 완료 및 주요 도메인 로직 검증을 강화했습니다.




```
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

프로젝트 구조를 설명하는 마크다운 코드 블록(```)이 닫히지 않은 상태에서 JaCoCo 섹션이 추가되었습니다. 이로 인해 235번 라인부터의 내용이 모두 코드 블록 내부의 텍스트로 취급되어 마크다운 렌더링이 정상적으로 이루어지지 않습니다.

235번 라인 시작 부분에 닫는 기호(```)를 추가하여 이전 코드 블록을 종료해 주십시오. 또한, 253번 라인의 중복된 닫는 기호와 섹션 사이의 과도한 빈 줄(예: 241-243, 249-252번 라인)을 제거하여 문서의 가독성을 높여주시기 바랍니다.

@LeeSG-0114 LeeSG-0114 merged commit 75eaf31 into develop Apr 30, 2026
1 check passed
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.

2 participants