Skip to content

(fix) : 정산 완료 문서 스니펫 경로 수정#55

Merged
sudhdkso merged 1 commit into
developfrom
fix/payment-request-settlement-detail
May 24, 2026
Merged

(fix) : 정산 완료 문서 스니펫 경로 수정#55
sudhdkso merged 1 commit into
developfrom
fix/payment-request-settlement-detail

Conversation

@sudhdkso
Copy link
Copy Markdown
Contributor

@sudhdkso sudhdkso commented May 24, 2026

#️⃣연관된 이슈

X

🔀반영 브랜치

fix/payment-request-settlement-detail -> develop

🔧변경 사항

정산 수동 완료 API 문서에서 참조하는 REST Docs snippet 이름과 실제 생성되는 snippet 이름이 달라 문서 빌드 시 unresolved directive가 발생하던 문제를 수정했습니다.

💬리뷰 요구사항(선택)

X

체크:

  • 테스트 코드 포함 여부: O
  • 불필요한 로그 제거: O
  • 예외 처리 여부: O

Summary by CodeRabbit

  • Tests
    • 테스트 코드의 문서화 방식을 개선했습니다.

Review Change Stack

@sudhdkso sudhdkso merged commit 0910ca2 into develop May 24, 2026
1 of 2 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 953a13f7-49f2-43bd-b101-a71d977935c1

📥 Commits

Reviewing files that changed from the base of the PR and between 55662c4 and e950eb1.

📒 Files selected for processing (1)
  • src/test/java/com/dnd/moddo/domain/settlement/controller/SettlementControllerTest.java

개요

SettlementControllerTest에서 Spring REST Docs의 정적 import를 추가하고, 문서화 호출을 restDocs.document()에서 document()로 변경하여 코드를 간결하게 리팩토링했습니다.

변경 사항

REST Docs 정적 import 통합

Layer / File(s) 설명
정적 import 추가 및 문서화 호출 변경
src/test/java/com/dnd/moddo/domain/settlement/controller/SettlementControllerTest.java
MockMvcRestDocumentation의 정적 메서드 import를 추가하고, givenExistingSettlement_thenCompleteSettlement 테스트에서 문서화 호출 방식을 정적 import 기반으로 전환합니다.

관련 PR

  • moddo-kr/moddo-backend#53: 본 PR의 SettlementControllerTest 변경 사항이 동일 테스트 범위에서 "정산 수동 완료" 엔드포인트의 문서화 흐름과 연관되어 있습니다.

권장 검토자

  • choyeseol

시인의 한마디

🐰 정적 import의 우아한 춤,
문서화도 간결하게 꿈,
REST Docs와 함께 춤을 추며,
테스트 코드는 더욱 예뻐지네! ✨


🎯 1 (Trivial) | ⏱️ ~3분

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/payment-request-settlement-detail

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

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

There is no coverage information present for the Files changed

Total Project Coverage 71.08%

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.

1 participant