Skip to content

Refactor Modo init: deprecate legacy API, unify caching, added unit tests#76

Merged
ikarenkov merged 6 commits intoreleases/0.11.0from
refactor-remember-counter-and-root
Apr 21, 2026
Merged

Refactor Modo init: deprecate legacy API, unify caching, added unit tests#76
ikarenkov merged 6 commits intoreleases/0.11.0from
refactor-remember-counter-and-root

Conversation

@ikarenkov
Copy link
Copy Markdown
Owner

  • Deprecate getOrCreateRootScreen/save/onRootScreenFinished (use rememberRootScreen, removed in 1.0)
  • Unify all init paths through rootScreens.getOrPut for consistent in-memory caching
  • Rename restoreScreenCounter → restoreScreenCounterIfNeeded; extract isFragmentClosing
  • Add ModoLegacyIntegrationFragment + button in MainScreen for legacy fragment demo
  • Add ModoRootScreenCacheTest covering all 3 init scenarios (first init, backstack, bundle restore)
  • Add MockK 1.13.12 as test dependency

… add unit tests

- Deprecate getOrCreateRootScreen/save/onRootScreenFinished (use rememberRootScreen, removed in 1.0)
- Unify all init paths through rootScreens.getOrPut for consistent in-memory caching
- Rename restoreScreenCounter → restoreScreenCounterIfNeeded; extract isFragmentClosing
- Add ModoLegacyIntegrationFragment + button in MainScreen for legacy fragment demo
- Add ModoRootScreenCacheTest covering all 3 init scenarios (first init, backstack, bundle restore)
- Add MockK 1.13.12 as test dependency
# Conflicts:
#	gradle/libs.versions.toml
#	modo-compose/build.gradle.kts
Comment thread gradle/libs.versions.toml
detektComposeVersion = "0.3.20"
detektVersion = "1.23.6"
junit = "4.13.2"
mockk = "1.13.12"
@ikarenkov ikarenkov force-pushed the refactor-remember-counter-and-root branch from f8cd13e to 42c5aaf Compare April 20, 2026 16:42
@ikarenkov ikarenkov merged commit 74f8992 into releases/0.11.0 Apr 21, 2026
4 checks passed
@ikarenkov ikarenkov deleted the refactor-remember-counter-and-root branch April 21, 2026 05:49
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.

2 participants