Skip to content

Commit b265a38

Browse files
committed
Fix test warnings
1 parent 0eb0254 commit b265a38

3 files changed

Lines changed: 7 additions & 4 deletions

File tree

‎livekit-rtc/livekit/rtc/audio_stream.py‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -248,7 +248,10 @@ def from_track(
248248
)
249249

250250
def __del__(self) -> None:
251-
FfiClient.instance.queue.unsubscribe(self._ffi_queue)
251+
# _ffi_queue is unset if __init__ raised early or the instance was built via __new__
252+
ffi_queue = getattr(self, "_ffi_queue", None)
253+
if ffi_queue is not None:
254+
FfiClient.instance.queue.unsubscribe(ffi_queue)
252255

253256
def _create_owned_stream(self) -> Any:
254257
assert self._track is not None

‎tests/api/test_access_token.py‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
from livekit.protocol.agent_dispatch import RoomAgentDispatch
99

1010
TEST_API_KEY = "myapikey"
11-
TEST_API_SECRET = "thiskeyistotallyunsafe"
11+
TEST_API_SECRET = "thiskeyistotallyunsafe-but-32-bytes-long"
1212

1313

1414
def test_verify_token() -> None:
@@ -81,7 +81,7 @@ def test_agent_config() -> None:
8181
def test_verify_token_invalid() -> None:
8282
token = AccessToken(TEST_API_KEY, TEST_API_SECRET).with_identity("test_identity").to_jwt()
8383

84-
token_verifier = TokenVerifier(TEST_API_KEY, "invalid_secret")
84+
token_verifier = TokenVerifier(TEST_API_KEY, "invalid_secret-padded-to-32-bytes-long")
8585
with pytest.raises(Exception):
8686
token_verifier.verify(token)
8787

‎tests/api/test_webhook.py‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from livekit.api import AccessToken, TokenVerifier, WebhookReceiver
66

77
TEST_API_KEY = "myapikey"
8-
TEST_API_SECRET = "thiskeyistotallyunsafe"
8+
TEST_API_SECRET = "thiskeyistotallyunsafe-but-32-bytes-long"
99
TEST_EVENT = """
1010
{
1111
"event": "room_started",

0 commit comments

Comments
 (0)