chore: improve test isolation and update dependencies

- Update GitHub Actions workflow to use gRPC tokens and increased idle timeouts for Android emulators.
- Adjust API level 37 to '37.0' in the KMP test matrix.
- Bump `androidSqlCipher` to 4.16.0 and `androidxNavigationEvent` to 1.1.0.
- Refactor `JvmSafeRepo` and `JdbcDatabaseHolder` to support database path overrides, enabling better test isolation.
- Implement temporary file-based database storage for JVM tests to ensure a clean state between runs.
- Enhance `NoteUseCaseTest` teardown to explicitly close and delete the database.
- Refine iOS database deletion logic to verify file existence before attempting removal.

Author: softartdev

.github/workflows/kmp.yml

@@ -62,7 +62,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        api-level: [ 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37 ]
+        api-level: [ 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, '37.0' ]
         target: [default]
     steps:
       - name: Check out
@@ -99,7 +99,7 @@ jobs:
           arch: x86_64
           profile: Nexus 6
           force-avd-creation: false
-          emulator-options: -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
+          emulator-options: -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none -grpc-use-token -idle-grpc-timeout 300
           disable-animations: false
           script: echo "Generated AVD snapshot for caching."
       - name: Run UI-tests on emulator
@@ -110,7 +110,7 @@ jobs:
           arch: x86_64
           profile: Nexus 6
           force-avd-creation: false
-          emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none
+          emulator-options: -no-snapshot-save -no-window -gpu swiftshader_indirect -noaudio -no-boot-anim -camera-back none -grpc-use-token -idle-grpc-timeout 300
           disable-animations: true
           script: ./gradlew :app:android:connectedCheck
       - name: Archive ui-tests-output artifacts

core/data/db-sqldelight/src/commonTest/kotlin/com/softartdev/notedelight/NoteUseCaseTest.kt

@@ -42,6 +42,7 @@ class NoteUseCaseTest : BaseTest() {
     @BeforeTest
     fun setUp() = runTest {
         Logger.setLogWriters(PrintLogWriter())
+        deleteDb()
         val noteDB = noteDB()
         noteDAO = safeRepo.noteDAO
         createNoteUseCase = CreateNoteUseCase(noteDAO!!)
@@ -52,7 +53,9 @@ class NoteUseCaseTest : BaseTest() {
 
     @AfterTest
     fun tearDown() = runTest {
-        noteDAO!!.deleteAll()
+        noteDAO?.deleteAll()
+        safeRepo.closeDatabase()
+        deleteDb()
         noteDAO = null
         createNoteUseCase = null
         saveNoteUseCase = null
@@ -130,4 +133,4 @@ class NoteUseCaseTest : BaseTest() {
     ): TestResult = kotlinx.coroutines.test.runTest(context, timeout) {
         mutex.withLock { testBody() }
     }
-}
+}

core/data/db-sqldelight/src/iosMain/kotlin/com/softartdev/notedelight/db/IosCipherUtils.kt

@@ -281,6 +281,7 @@ object IosCipherUtils {
             nsFileManager.createDirectoryAtPath(path, true, null, null)
         }
         val dbPath = getDatabasePath(dbName)
+        val dbFileExisted = nsFileManager.fileExistsAtPath(dbPath)
         val paths = listOf(
             dbPath,
             "$dbPath-wal",
@@ -293,7 +294,7 @@ object IosCipherUtils {
                 deleted = nsFileManager.removeItemAtPath(path, null) && deleted
             }
         }
-        return deleted
+        return dbFileExisted && deleted
     }
 
     fun checkCipherVersion(dbName: String): String? {

core/data/db-sqldelight/src/jvmMain/kotlin/com/softartdev/notedelight/db/JdbcDatabaseHolder.kt

@@ -10,9 +10,11 @@ import java.nio.charset.StandardCharsets
 import java.sql.SQLException
 import java.util.Properties
 
-class JdbcDatabaseHolder(props: Properties = Properties()) : SqlDelightDbHolder {
+class JdbcDatabaseHolder(
+    props: Properties = Properties(),
+    dbPath: String = FilePathResolver().invoke()
+) : SqlDelightDbHolder {
     override val logger = Logger.withTag(this@JdbcDatabaseHolder::class.simpleName.toString())
-    private val dbPath = FilePathResolver().invoke()
 
     val jdbcUrl: String = buildJdbcUrl(dbPath, props)
     override val driver: SqlDriver = createDriver(jdbcUrl, props)

core/data/db-sqldelight/src/jvmMain/kotlin/com/softartdev/notedelight/repository/JvmSafeRepo.kt

@@ -14,6 +14,7 @@ import java.util.Properties
 class JvmSafeRepo(private val coroutineDispatchers: CoroutineDispatchers) : SafeRepo() {
     @Volatile
     private var databaseHolder: JdbcDatabaseHolder? = null
+    private var dbPathOverride: String? = null
 
     override val databaseState: PlatformSQLiteState
         get() = JvmCipherUtils.getDatabaseState(dbPath)
@@ -22,14 +23,18 @@ class JvmSafeRepo(private val coroutineDispatchers: CoroutineDispatchers) : Safe
         get() = SqlDelightNoteDAO({ databaseHolder!!.noteQueries }, coroutineDispatchers)
 
     override val dbPath: String
-        get() = FilePathResolver().invoke()
+        get() = dbPathOverride ?: FilePathResolver().invoke()
+
+    internal fun overrideDbPath(dbPath: String) {
+        dbPathOverride = dbPath
+    }
 
     override suspend fun buildDbIfNeed(passphrase: CharSequence): JdbcDatabaseHolder {
         var instance = databaseHolder
         if (instance == null) {
             val properties = Properties()
             if (passphrase.isNotEmpty()) properties["password"] = StringBuilder(passphrase).toString()
-            instance = JdbcDatabaseHolder(properties)
+            instance = JdbcDatabaseHolder(properties, dbPath)
             instance.createSchema()
             databaseHolder = instance
         }
@@ -78,4 +83,4 @@ class JvmSafeRepo(private val coroutineDispatchers: CoroutineDispatchers) : Safe
         databaseHolder?.close()
         databaseHolder = null
     }
-}
+}

core/data/db-sqldelight/src/jvmTest/kotlin/com/softartdev/notedelight/BaseTest.jvm.kt

@@ -6,16 +6,21 @@ import com.softartdev.notedelight.db.JdbcDatabaseHolder
 import com.softartdev.notedelight.db.NoteDb
 import com.softartdev.notedelight.repository.JvmSafeRepo
 import com.softartdev.notedelight.repository.SafeRepo
+import java.io.File
 import kotlinx.coroutines.ExperimentalCoroutinesApi
 import kotlinx.coroutines.test.UnconfinedTestDispatcher
 
 actual abstract class BaseTest actual constructor() {
 
     private var _noteDb: NoteDb? = null
+    private val testDbFile: File = File.createTempFile("notedelight-test-", ".db").apply {
+        delete()
+        deleteOnExit()
+    }
 
     actual val safeRepo: SafeRepo = JvmSafeRepo(
         coroutineDispatchers = CoroutineDispatchersStub(testDispatcher = UnconfinedTestDispatcher())
-    )
+    ).also { it.overrideDbPath(testDbFile.absolutePath) }
 
     actual suspend fun noteDB(): NoteDb {
         if (_noteDb == null) {
@@ -26,5 +31,8 @@ actual abstract class BaseTest actual constructor() {
         return _noteDb!!
     }
 
-    actual fun deleteDb() {}
-}
+    actual fun deleteDb() {
+        _noteDb = null
+        testDbFile.delete()
+    }
+}

gradle/libs.versions.toml

@@ -11,7 +11,7 @@ sqlDelight = "2.3.2"
 room = "2.8.4"
 androidxSqlite = "2.6.2"
 saferoom = "1.4.0"
-androidSqlCipher = "4.15.0"
+androidSqlCipher = "4.16.0"
 iosSqlCipher = "4.9.0"
 jdbcSQLite = "3.51.2.0"
 koin-bom = "4.2.1"
@@ -28,7 +28,7 @@ androidxBiometric = "1.1.0"
 androidxDatastore = "1.2.1"
 androidxViewModel = "2.10.0"
 androidxNavigation = "2.9.2"
-androidxNavigationEvent = "1.0.1"
+androidxNavigationEvent = "1.1.0"
 androidxActivityCompose = "1.13.0"
 androidxComposeTest = "1.11.1"
 androidxCoreSplashscreen = "1.2.0"