Skip to content

fix: goal BC computed fields 하드코딩 문서화 및 통합 테스트 추가 (#28)#31

Merged
robinjoon merged 1 commit intomainfrom
ticket/#28
Mar 7, 2026
Merged

fix: goal BC computed fields 하드코딩 문서화 및 통합 테스트 추가 (#28)#31
robinjoon merged 1 commit intomainfrom
ticket/#28

Conversation

@robinjoon
Copy link
Member

Summary

  • GoalDataFetcher의 hardcoded computed fields(totalTaskCount, completedTaskCount, achievementRate)가 task BC의 @DgsData resolver에 의해 항상 오버라이드됨을 확인
  • DGS Codegen 제약으로 placeholder 값 제거 불가 → 주석 문서화로 대응
  • Presentation 통합 테스트 3개 추가: DataLoader를 통한 실제 값 반환 검증

Test plan

  • 전체 테스트 통과 (./gradlew test)
  • goal BC 파일만 수정, 다른 BC 미수정
  • 통합 테스트로 computed fields가 DataLoader에 의해 올바르게 반환됨 검증

Closes #28

🤖 Generated with Claude Code

- GoalDataFetcher.toGraphql()의 placeholder 값에 @DgsData 오버라이드 주석 추가
- GoalDataFetcherTest 통합 테스트 3개 추가 (DataLoader 기반 실제 값 검증)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@robinjoon robinjoon merged commit 6851503 into main Mar 7, 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.

goal BC: computed fields 하드코딩 값 반환 수정

1 participant