Merge pull request #2795 from vitorpamplona/claude/fix-deprecated-mutex-lock-bRzWb

Rename lock fields to reflect their specific purposes
This commit is contained in:
Vitor Pamplona
2026-05-08 16:38:03 -04:00
committed by GitHub
@@ -53,7 +53,7 @@ class QlogObserverTest {
runBlocking { runBlocking {
val client = handshakedClient(QlogObserver.NoOp) val client = handshakedClient(QlogObserver.NoOp)
// Drive a tiny operation post-handshake. // Drive a tiny operation post-handshake.
client.lock.withLock { client.lifecycleLock.withLock {
// No-op — just confirm we can still take the lock. // No-op — just confirm we can still take the lock.
assertEquals(QuicConnection.Status.CONNECTED, client.status) assertEquals(QuicConnection.Status.CONNECTED, client.status)
} }
@@ -141,7 +141,7 @@ class QlogObserverTest {
0x00, // pn byte 0x00, // pn byte
0x00, // payload byte (will fail AEAD) 0x00, // payload byte (will fail AEAD)
) )
client.lock.withLock { client.streamsLock.withLock {
feedDatagram(client, garbage, nowMillis = 1L) feedDatagram(client, garbage, nowMillis = 1L)
} }
val drops = recorder.events.filter { it.name == "packetDropped" } val drops = recorder.events.filter { it.name == "packetDropped" }