From 2d8cccaeaa516971bb6b73a56a8c5261b4745dc9 Mon Sep 17 00:00:00 2001 From: Jeroen Schweitzer Date: Tue, 14 Jul 2026 11:16:27 +0200 Subject: [PATCH] test: align volatile TTL expectations with doubled namespace TTLs The TTLs were doubled in 4cfad2e (v1.7.2) to survive missed scheduler runs, but the unit tests kept the old expectations and have been failing since. Update weather (7200), financial (600), and sports (120) assertions to the current defaults. Co-Authored-By: Claude Fable 5 --- tests/test_volatile.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/test_volatile.py b/tests/test_volatile.py index 73bdf37..3162d11 100644 --- a/tests/test_volatile.py +++ b/tests/test_volatile.py @@ -98,15 +98,15 @@ class TestVolatileNamespaces: def test_weather_default_ttl(self): """Test weather namespace default TTL.""" - assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.WEATHER] == 3600 # 1 hour (current conditions) + assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.WEATHER] == 7200 # 2 hours (hourly refresh) def test_financial_default_ttl(self): """Test financial namespace default TTL.""" - assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.FINANCIAL] == 300 # 5 min + assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.FINANCIAL] == 600 # 10 min (5 min refresh) def test_sports_default_ttl(self): """Test sports namespace default TTL (fast updates).""" - assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.SPORTS] == 60 # 1 min + assert NAMESPACE_DEFAULT_TTL[VolatileNamespace.SPORTS] == 120 # 2 min (1 min refresh) def test_namespace_count(self): """Test we have the expected number of namespaces.""" @@ -274,7 +274,7 @@ class TestVolatileService: def test_get_default_ttl_known_namespace(self, volatile_service): """Test default TTL for known namespace.""" ttl = volatile_service._get_default_ttl("weather") - assert ttl == 3600 # Weather namespace default (1 hour) + assert ttl == 7200 # Weather namespace default (2 hours) def test_get_default_ttl_unknown_namespace(self, volatile_service): """Test default TTL for unknown namespace.""" @@ -366,7 +366,7 @@ class TestVolatileService: ttl=None # Not specified ) - assert result.ttl == 3600 # Weather default (1 hour) + assert result.ttl == 7200 # Weather default (2 hours) @pytest.mark.asyncio async def test_search_empty_collection(self, volatile_service, mock_qdrant, mock_ollama):