test: require deterministic trade identity
This commit is contained in:
@@ -106,6 +106,9 @@ def test_research_artifact_projects_versioned_queryable_fact_tables() -> None:
|
|||||||
result.schedule.signal_to_execution.iloc[0].date()
|
result.schedule.signal_to_execution.iloc[0].date()
|
||||||
]
|
]
|
||||||
assert set(artifact.trades["side"]) == {"buy", "sell"}
|
assert set(artifact.trades["side"]) == {"buy", "sell"}
|
||||||
|
assert artifact.trades["trade_id"].is_unique
|
||||||
|
assert artifact.trades["trade_id"].str.startswith("run-20260105-a:").all()
|
||||||
|
assert artifact.trades["signal_id"].str.startswith("run-20260105-a:signal:").all()
|
||||||
assert {"security", "cash"}.issubset(set(artifact.positions["asset_type"]))
|
assert {"security", "cash"}.issubset(set(artifact.positions["asset_type"]))
|
||||||
assert artifact.positions.groupby("trade_date")["weight"].sum().tolist() == pytest.approx(
|
assert artifact.positions.groupby("trade_date")["weight"].sum().tolist() == pytest.approx(
|
||||||
[1.0, 1.0, 1.0, 1.0]
|
[1.0, 1.0, 1.0, 1.0]
|
||||||
|
|||||||
Reference in New Issue
Block a user