From f30166f9d5be206510b9be4bdfb1a6379897e3c4 Mon Sep 17 00:00:00 2001 From: Andy Lee Date: Wed, 13 Aug 2025 12:28:48 -0700 Subject: [PATCH] fix: increase CI test timeouts to accommodate model download MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Analysis of recent CI failures shows: - Model download takes ~12 seconds - Embedding server startup + first search takes additional ~78 seconds - Total time needed: ~90-100 seconds Updated timeouts: - test_readme_basic_example: 90s -> 180s - test_backend_options: 60s -> 150s - test_llm_config_simulated: 75s -> 150s Root cause: Initial model download from huggingface.co in CI environment is slower than local development, causing legitimate timeouts rather than actual hanging processes. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- tests/test_readme_examples.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/test_readme_examples.py b/tests/test_readme_examples.py index 7c483c2..4defcf9 100644 --- a/tests/test_readme_examples.py +++ b/tests/test_readme_examples.py @@ -12,7 +12,7 @@ from test_timeout import ci_timeout @pytest.mark.parametrize("backend_name", ["hnsw", "diskann"]) -@ci_timeout(90) # 90 second timeout for this comprehensive test +@ci_timeout(180) # 180 second timeout to allow for model download and initialization def test_readme_basic_example(backend_name): """Test the basic example from README.md with both backends.""" # Skip on macOS CI due to MPS environment issues with all-MiniLM-L6-v2 @@ -81,7 +81,7 @@ def test_readme_imports(): assert callable(LeannChat) -@ci_timeout(60) # 60 second timeout +@ci_timeout(150) # 150 second timeout to allow for model download def test_backend_options(): """Test different backend options mentioned in documentation.""" # Skip on macOS CI due to MPS environment issues with all-MiniLM-L6-v2 @@ -118,7 +118,7 @@ def test_backend_options(): @pytest.mark.parametrize("backend_name", ["hnsw", "diskann"]) -@ci_timeout(75) # 75 second timeout for LLM tests +@ci_timeout(150) # 150 second timeout to allow for model download def test_llm_config_simulated(backend_name): """Test simulated LLM configuration option with both backends.""" # Skip on macOS CI due to MPS environment issues with all-MiniLM-L6-v2