Skip to content

[FSSDK-12510] fix old test issues with mockito versions#527

Open
jaeopt wants to merge 11 commits intomasterfrom
jae/fr
Open

[FSSDK-12510] fix old test issues with mockito versions#527
jaeopt wants to merge 11 commits intomasterfrom
jae/fr

Conversation

@jaeopt
Copy link
Copy Markdown
Contributor

@jaeopt jaeopt commented May 6, 2026

Fix instrumentation tests on Android API 28+

Problem

  • TransactionTooLargeException crashed the android-sdk test process when 262+ test results were bundled into a single IPC parcel exceeding Android's 1MB limit
  • Mockito cannot mock this class failures on API 28+ because the default mockito-android mock maker uses a hidden/restricted API (BaseDexClassLoader#addDexPath()) blocked since API 28
  • Root cause of Mockito failures: PR Upgrade targetSDK to 35 #516 (targetSDK 35 upgrade) downgraded mockito_ver_sdk_module from 3.6.282.28.2, which is too old to work with dexmaker-mockito-inline

Changes

  • Revert mockito_ver_sdk_module back to 3.6.28
  • Add dexmaker-mockito-inline:2.28.3 to all 6 modules — uses JVMTI bytecode rewriting instead of restricted APIs, works on API 28+
  • Add Android Test Orchestrator to android-sdk module — runs each test class in its own process, preventing parcel overflow as the test suite grows
  • Pin objenesis:2.6 to maintain API 21 compatibility (objenesis 3.x requires API 26+)
  • Set testCoverageEnabled false in all modules to avoid JVMTI agent conflict between Jacoco and dexmaker

Test plan

Existing tests will cover the changes.

Issues

  • FSSDK-12510

@jaeopt jaeopt changed the title [FSSDK-12510] fix old test build issues (mockito versions) [FSSDK-12510] fix old test issues with mockito versions May 6, 2026
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