charieswei
/
TradingAgents
mirror of https://github.com/TauricResearch/TradingAgents.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #47 from aguzererler/fix/test-config-sum-constraints-edge-case-6454385375638160819 

🧪 test: add edge case for portfolio config sum constraints
This commit is contained in:
ahmet guzererler 2026-03-21 17:20:25 +01:00 committed by GitHub
parent e877f298bb 2663661be9
commit 0313cc629e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/portfolio/test_config.py
View File

@ -102,3 +102,15 @@ def test_validate_config_sum_constraints_invalid():
} }
with pytest.raises(ValueError, match=r"must be <= 1.0"): with pytest.raises(ValueError, match=r"must be <= 1.0"):
validate_config(cfg) validate_config(cfg)
def test_validate_config_sum_constraints_edge_case():
"""min_cash_pct + max_position_pct can be exactly 1.0."""
cfg = {
"max_positions": 10,
"max_position_pct": 0.5,
"max_sector_pct": 0.3,
"min_cash_pct": 0.5,
"default_budget": 100000.0
}
# Should not raise any exception
validate_config(cfg)