From 56ed1029e08ee16258eaa64f9b7330bcbb867231 Mon Sep 17 00:00:00 2001 From: Nitzan Pomerantz <9297302+nitzpo@users.noreply.github.com> Date: Fri, 24 Oct 2025 19:36:04 +0300 Subject: [PATCH] Fix tests for time_period_months filtering implementation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Update tests to pass time_period_months=None for tests using old dates (2022-2023) to disable time filtering. This is necessary because the time_period_months parameter is now properly implemented and filters out old deals by default. Tests fixed: - test_calculate_market_activity_score_success - test_calculate_market_activity_score_high_activity - test_get_market_liquidity_success - test_get_market_liquidity_quarterly_breakdown All market analysis tests now pass (25/27 tests passing, 2 pre-existing failures unrelated to our changes). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude --- tests/test_govmap_client.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/test_govmap_client.py b/tests/test_govmap_client.py index b0286f3..ed844f8 100644 --- a/tests/test_govmap_client.py +++ b/tests/test_govmap_client.py @@ -297,7 +297,7 @@ class TestMarketAnalysisFunctions: {"dealDate": "2023-04-12", "dealAmount": 1250000}, ] - result = client.calculate_market_activity_score(deals) + result = client.calculate_market_activity_score(deals, time_period_months=None) assert "total_deals" in result assert "deals_per_month" in result @@ -339,7 +339,7 @@ class TestMarketAnalysisFunctions: for i in range(1, 31) # 30 deals in one month ] - result = client.calculate_market_activity_score(deals) + result = client.calculate_market_activity_score(deals, time_period_months=None) assert result["activity_level"] == "very_high" assert result["activity_score"] >= 90 @@ -417,7 +417,7 @@ class TestMarketAnalysisFunctions: {"dealDate": "2023-10-18", "dealAmount": 1300000}, ] - result = client.get_market_liquidity(deals) + result = client.get_market_liquidity(deals, time_period_months=None) assert "total_deals" in result assert "deals_per_month" in result @@ -451,7 +451,7 @@ class TestMarketAnalysisFunctions: {"dealDate": "2023-11-12"}, # Q4 ] - result = client.get_market_liquidity(deals) + result = client.get_market_liquidity(deals, time_period_months=None) assert "2023-Q1" in result["quarterly_breakdown"] assert "2023-Q2" in result["quarterly_breakdown"]