Skip to content

[Fix]: 온보딩 테스트 컴파일 에러 및 ktlint 위반 수정 - #163

Merged
oungsi2000 merged 1 commit into
developfrom
fix/onboarding-test-compile-error
Aug 24, 2026
Merged

[Fix]: 온보딩 테스트 컴파일 에러 및 ktlint 위반 수정#163
oungsi2000 merged 1 commit into
developfrom
fix/onboarding-test-compile-error

Conversation

@oungsi2000

Copy link
Copy Markdown
Collaborator

관련 이슈

작업 내용

develop 브랜치의 CI(lint-and-test)가 실패하고 있던 문제 수정

변경사항 / 상세

  • OnboardingState.isSubmitting 필드가 submitState: OnboardingSubmitState?로 바뀐 이후 OnboardingViewModelTest가 갱신되지 않아 컴파일 에러(No parameter with name 'isSubmitting' found)로 테스트 빌드가 실패하던 문제 수정
    • isSubmitting = true/false 참조를 submitState = OnboardingSubmitState.Loading/Success/Failure로 교체
    • 성공 경로 테스트에 누락되어 있던 PermissionRequest 사이드이펙트 검증 추가 (turbine Unconsumed events 방지)
  • OnboardingViewModel.ktktlintMainSourceSetCheck 위반(줄바꿈/라인 길이) 정리

중점 리뷰사항

  • 테스트에서 재구성한 submitState/step 전이가 OnboardingViewModel.confirmComplete의 실제 로직과 일치하는지

스크린샷 (선택)

🤖 Generated with Claude Code

OnboardingState의 isSubmitting 필드가 submitState(OnboardingSubmitState)로
바뀐 이후 OnboardingViewModelTest가 갱신되지 않아 develop 빌드의
lint-and-test가 컴파일 에러로 실패하고 있었다. 테스트를 현재 상태 모델에
맞게 수정하고, 성공 경로에서 함께 발생하는 PermissionRequest 사이드이펙트
검증을 추가했다. 같은 파일의 ktlintMainSourceSetCheck 위반도 함께 정리.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015do5t4ThrkmdyfYb4gkbCq
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 598ad865-3efb-48ea-b2b4-c5a3345c8b57

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@oungsi2000 oungsi2000 changed the title fix: 온보딩 테스트 컴파일 에러 및 ktlint 위반 수정 [Fix]: 온보딩 테스트 컴파일 에러 및 ktlint 위반 수정 Aug 24, 2026
@oungsi2000 oungsi2000 self-assigned this Aug 24, 2026
@oungsi2000
oungsi2000 merged commit f31f474 into develop Aug 24, 2026
3 checks 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.

1 participant