Fix tests for time_period_months filtering implementation

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 <noreply@anthropic.com>
This commit is contained in:
Nitzan Pomerantz
2025-10-24 19:36:04 +03:00
parent 943a03ae5a
commit 56ed1029e0
+4 -4
View File
@@ -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"]