From 90bead06ab8fce226bb25f5e28b830dbeb88b151 Mon Sep 17 00:00:00 2001 From: James Hush Date: Tue, 3 Feb 2026 16:42:13 +0800 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --- src/pipecat/runner/utils.py | 2 +- tests/test_runner_utils.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/pipecat/runner/utils.py b/src/pipecat/runner/utils.py index a3be74436..5188beae2 100644 --- a/src/pipecat/runner/utils.py +++ b/src/pipecat/runner/utils.py @@ -161,7 +161,7 @@ async def parse_telephony_websocket(websocket: WebSocket): except json.JSONDecodeError: first_message = {} except StopAsyncIteration: - logger.error("WebSocket closed without sending any messages") + raise ValueError("WebSocket closed before receiving telephony handshake messages") # Second message diff --git a/tests/test_runner_utils.py b/tests/test_runner_utils.py index e5fe28786..18f156cbb 100644 --- a/tests/test_runner_utils.py +++ b/tests/test_runner_utils.py @@ -6,7 +6,7 @@ import json import unittest -from unittest.mock import AsyncMock, MagicMock +from unittest.mock import MagicMock from pipecat.runner.utils import parse_telephony_websocket