nick/amethyst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

feat: make forEachDB action suspending, remove redundant runBlocking

Browse Source 
Change the action lambda to `suspend (EventStore) -> Unit` so callers
can use suspend functions directly. Remove the now-unnecessary
runBlocking wrapper around delay() in ExpirationTest.

https://claude.ai/code/session_01TQRzVTAXBVWRGstcsapA5F
This commit is contained in:
Claude
2026-03-28 16:15:23 +00:00
parent 0889473c15
commit d299b65c8a
2 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
quartz/src/commonTest/kotlin/com/vitorpamplona/quartz/nip01Core/store/sqlite/BaseDBTest.kt
+1 -1
View File
@@ -75,7 +75,7 @@ open class BaseDBTest {
dbs.forEach { it.value.close() } dbs.forEach { it.value.close() }
} }
fun forEachDB(action: (EventStore) -> Unit) = fun forEachDB(action: suspend (EventStore) -> Unit) =
runBlocking { runBlocking {
dbs.forEach { (key, value) -> dbs.forEach { (key, value) ->
launch(Dispatchers.Default) { launch(Dispatchers.Default) {
quartz/src/commonTest/kotlin/com/vitorpamplona/quartz/nip01Core/store/sqlite/ExpirationTest.kt
+1 -4
View File
@@ -27,7 +27,6 @@ import com.vitorpamplona.quartz.nip10Notes.TextNoteEvent
import com.vitorpamplona.quartz.nip40Expiration.expiration import com.vitorpamplona.quartz.nip40Expiration.expiration
import com.vitorpamplona.quartz.utils.TimeUtils import com.vitorpamplona.quartz.utils.TimeUtils
import kotlinx.coroutines.delay import kotlinx.coroutines.delay
import kotlinx.coroutines.runBlocking
import kotlin.test.Test import kotlin.test.Test
import kotlin.test.assertFailsWith import kotlin.test.assertFailsWith
@@ -59,9 +58,7 @@ class ExpirationTest : BaseDBTest() {
db.assertQuery(noteToExpire, Filter(ids = listOf(noteToExpire.id))) db.assertQuery(noteToExpire, Filter(ids = listOf(noteToExpire.id)))
runBlocking { delay(2000)
delay(2000)
}
db.deleteExpiredEvents() db.deleteExpiredEvents()