Skip to content

test: mindrecord androidTest를 JVM으로 이동 (#1090) - #1353

Merged
OnlyReadOneBook merged 4 commits into
fix/964-write-failure-telemetryfrom
test/1090-mindrecord-jvm-tests
Aug 28, 2026
Merged

OnlyReadOneBook merged 4 commits into
fix/964-write-failure-telemetryfrom
test/1090-mindrecord-jvm-tests

Conversation

@OnlyReadOneBook

Copy link
Copy Markdown
Collaborator

📌 Issues

📎 Work Description

  • MindRecordFlowAndroidTest·MindRecordLifecycleAndroidTest의 7개 Compose 시나리오를 mindrecord presentation 모듈 JVM src/test로 이동했습니다.
  • AndroidJUnit4를 Robolectric SDK 35·Native graphics·Compose v2 rule로 전환하고 instrumentation 전용 FailureArtifactRule을 제거했습니다.
  • PR #1278의 RecordingErrorReporter stage 검증과 ViewModel 생성자 계약, 기존 payload·repository 호출·UI assertion·5초 timeout을 보존했습니다.
  • 원본 app helper의 strict UserRepository allowlist를 feature-local helper로 복원해 예기치 않은 repository 호출은 계속 실패하도록 했습니다.
  • 필터 바텀시트는 w360dp-h800dp-xhdpi viewport에서 원본 performClick()을 그대로 실행하고 상태 재구성 완료를 명시적으로 기다립니다.

스택 관계

🧪 CI Test Plan

{
  "androidTest": {
    "mode": "selected",
    "reason": "mindrecord feature-local Compose 계측 소스를 JVM으로 이동했으므로, app에 남는 mindrecord home의 로딩·재시도·성공 경계를 API 30에서 선택 검증합니다.",
    "tests": [
      {
        "path": "app/src/androidTest/java/com/afternote/afternote_fe/TimeLetterMindRecordCompletionAndroidTest.kt",
        "selector": "com.afternote.afternote_fe.TimeLetterMindRecordCompletionAndroidTest#mindRecordHome_dailyQuestionLoadingEmptyAndErrorRetrySuccess_areRendered",
        "device": "api30"
      }
    ]
  }
}

검증:

  • :feature:mindrecord:presentation:testDebugUnitTest (7/7, failures/errors/skipped 0)
  • :feature:mindrecord:presentation:ktlintCheck
  • :feature:mindrecord:presentation:koverXmlReportCi
  • :konsist:test
  • :app:compileDebugAndroidTestKotlin -x :app:processDebugGoogleServices
  • git diff --check

📷 Screenshot

  • UI 변경 없음

💬 To Reviewers

  • TimeLetterMindRecordCompletionAndroidTest는 timeletter 3개·mindrecord 5개로 분리 가능하지만, 현재 PR #1340과 #1278이 서로 다른 스택에서 같은 파일을 수정 중이므로 이 PR에서는 #1090에 명시된 두 파일만 이동했습니다.
  • 두 선행 스택이 합류되면 남은 completion 파일도 각 feature JVM 테스트로 물리적 분할해야 합니다.

@github-actions github-actions Bot added the android-test PR head에서 Android Managed Device 계측 테스트 실행 label Aug 28, 2026
@OnlyReadOneBook
OnlyReadOneBook merged commit a9ca273 into fix/964-write-failure-telemetry Aug 28, 2026
23 checks passed
@OnlyReadOneBook
OnlyReadOneBook deleted the test/1090-mindrecord-jvm-tests branch August 28, 2026 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android-test PR head에서 Android Managed Device 계측 테스트 실행

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants