Skip to content

feat(session): add database preparation boundary PR 1 - #5158

Open
MohamadJaara wants to merge 1 commit into
developfrom
mo/async-db-migration/01-foundation
Open

feat(session): add database preparation boundary PR 1#5158
MohamadJaara wants to merge 1 commit into
developfrom
mo/async-db-migration/01-foundation

Conversation

@MohamadJaara

Copy link
Copy Markdown
Member

Goal

Add the Android boundary for Kalium's explicit user-session database preparation lifecycle.

Changes

  • update the Kalium pointer to the preparation API implementation
  • add a single Android preparation gate and explicit success/failure mapping
  • cover shared preparation, scope reuse, and retryability with focused tests

Impact

This is a foundation-only change. It introduces no screen or consumer behavior yet; later PRs use this boundary before accessing a user database.

Stack

  • Epic branch: mo/epic/async-db-migration
  • PR 1 of 7

@MohamadJaara
MohamadJaara changed the base branch from mo/epic/async-db-migration to develop August 18, 2026 12:27
@MohamadJaara
MohamadJaara force-pushed the mo/async-db-migration/01-foundation branch from 960b004 to c6f2da8 Compare August 18, 2026 12:42
@MohamadJaara
MohamadJaara force-pushed the mo/async-db-migration/01-foundation branch from c6f2da8 to 0f4d4ce Compare August 18, 2026 12:45
@github-actions

Copy link
Copy Markdown
Contributor

Test Results

381 tests   - 1 884   381 ✅  - 1 884   20s ⏱️ - 2m 4s
 40 suites  -   210     0 💤 ±    0 
 40 files    -   210     0 ❌ ±    0 

Results for commit 0f4d4ce. ± Comparison against base commit 96e3744.

This pull request removes 1884 tests.
com.wire.android.GlobalObserversManagerTest ‑ given app not visible and valid session, when handling ephemeral messages, then do not call deleteEphemeralMessageEndDate()
com.wire.android.GlobalObserversManagerTest ‑ given app not visible and valid session, when observing app open, then do not retry sending pending messages()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and invalid session, when app opens, then do not retry sending pending messages()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and invalid session, when handling ephemeral messages, then do not call deleteEphemeralMessageEndDate()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and no session, when app opens, then do not retry sending pending messages()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and no session, when handling ephemeral messages, then do not call deleteEphemeralMessageEndDate()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and session failure, when app opens, then do not retry sending pending messages()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and session failure, when handling ephemeral messages, then do not call deleteEphemeralMessageEndDate()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and valid session, when app opens, then retry sending pending messages()
com.wire.android.GlobalObserversManagerTest ‑ given app visible and valid session, when handling ephemeral messages, then call deleteEphemeralMessageEndDate()
…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants