Skip to content

Add more tests for RealSqlCipherLoaderTest#8034

Merged
karlenDimla merged 1 commit intodevelopfrom
feature/karl/sqlcipher/add-missing-test
Mar 20, 2026
Merged

Add more tests for RealSqlCipherLoaderTest#8034
karlenDimla merged 1 commit intodevelopfrom
feature/karl/sqlcipher/add-missing-test

Conversation

@karlenDimla
Copy link
Contributor

@karlenDimla karlenDimla commented Mar 20, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1201462763415876/task/1213737428120588?focus=true

Description

Add back removed test for SqlCipher loader

Steps to test this PR

QA-optional


Note

Low Risk
Low risk: only unit test and Gradle test configuration changes, no production logic modifications. Main risk is potential test flakiness or slower test execution due to added Robolectric resources.

Overview
Adds a Robolectric-based test suite for RealSqlCipherLoader.waitForLibraryLoad, covering success, load failure, timeout handling, cancellation propagation, single-initialization behavior, concurrent callers, and early-load triggers via onCreate/onPirProcessCreated.

Updates sqlcipher-loader-impl test setup to support these cases by adding Robolectric/AndroidX test dependencies, enabling includeAndroidResources, and introducing a ShadowLibraryLoader to simulate native library load callbacks without loading real binaries.

Written by Cursor Bugbot for commit 143c446. This will update automatically on new commits. Configure here.

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@karlenDimla karlenDimla merged commit a78317d into develop Mar 20, 2026
17 checks passed
@karlenDimla karlenDimla deleted the feature/karl/sqlcipher/add-missing-test branch March 20, 2026 09:25
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