From 89f10dd9a1a29f904ba6e27daf0822fe77d9629f Mon Sep 17 00:00:00 2001 From: Mark Backman Date: Mon, 4 May 2026 16:42:05 -0400 Subject: [PATCH] test: drop webrtc-dependent test, remove webrtc extra from CI --- .github/workflows/tests.yaml | 1 - tests/test_runner_downloads.py | 15 ++------------- 2 files changed, 2 insertions(+), 14 deletions(-) diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml index eb5414a30..04ff89543 100644 --- a/.github/workflows/tests.yaml +++ b/.github/workflows/tests.yaml @@ -49,7 +49,6 @@ jobs: --extra runner \ --extra sagemaker \ --extra tracing \ - --extra webrtc \ --extra websocket - name: Test with pytest diff --git a/tests/test_runner_downloads.py b/tests/test_runner_downloads.py index 4dc4fc25d..5b984dda0 100644 --- a/tests/test_runner_downloads.py +++ b/tests/test_runner_downloads.py @@ -8,12 +8,10 @@ import os import tempfile import unittest from pathlib import Path -from types import SimpleNamespace -from fastapi import FastAPI, HTTPException -from fastapi.testclient import TestClient +from fastapi import HTTPException -from pipecat.runner.run import _resolve_download_path, _setup_webrtc_routes +from pipecat.runner.run import _resolve_download_path class TestRunnerDownloads(unittest.TestCase): @@ -41,15 +39,6 @@ class TestRunnerDownloads(unittest.TestCase): self.assertEqual(context.exception.status_code, 403) - def test_download_file_returns_404_when_folder_not_configured(self): - app = FastAPI() - args = SimpleNamespace(folder=None, esp32=False, host="127.0.0.1") - _setup_webrtc_routes(app, args) - - response = TestClient(app).get("/files/recording.txt") - - self.assertEqual(response.status_code, 404) - def test_resolve_download_path_blocks_decoded_encoded_slashes(self): with tempfile.TemporaryDirectory() as tmpdir: root = Path(tmpdir)