AdrianoDev
4d9db750be
- pyproject.toml: ruff target-version py311 → py313 (auto-fix 42 lint warnings via UP rules); aggiunto consider_namespace_packages = true che risolve la collisione conftest tra servizi e permette di lanciare pytest sull'intera suite cross-servizio. - mcp_common.audit: nuovo helper audit_write_op() con logger dedicato mcp.audit. Wirato su tutti i write endpoint di deribit, bybit, alpaca e hyperliquid (place_order, place_combo_order, cancel_*, set_*, close_*, transfer_*, switch_*, amend_*) con principal + target + payload non-sensibile + result summarizzato. - mcp_common.app_factory: ExchangeAppSpec + run_exchange_main() centralizza il boilerplate dei __main__.py (configure_root_logging, fail_fast_if_missing, summarize, load creds, resolve_environment, load token store, uvicorn). I 4 __main__.py exchange ridotti da ~60 LOC ognuno a ~25 LOC dichiarativi. mcp_common.env_validation promosso da mcp_deribit (mantenuto re-export shim per back-compat test_env_validation). - 8 test nuovi (4 audit + 4 app_factory). Suite full: 450/450 verdi. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
51 lines
1.6 KiB
Python
51 lines
1.6 KiB
Python
from __future__ import annotations
|
|
|
|
import pytest
|
|
from fastapi import HTTPException
|
|
from mcp_bybit.leverage_cap import enforce_leverage, get_max_leverage
|
|
|
|
|
|
def test_get_max_leverage_returns_creds_value():
|
|
creds = {"max_leverage": 5}
|
|
assert get_max_leverage(creds) == 5
|
|
|
|
|
|
def test_get_max_leverage_default_when_missing():
|
|
"""Default 1 (cash) se il secret non ha max_leverage."""
|
|
assert get_max_leverage({}) == 1
|
|
|
|
|
|
def test_enforce_leverage_pass_under_cap():
|
|
creds = {"max_leverage": 3}
|
|
enforce_leverage(2, creds=creds, exchange="bybit") # no raise
|
|
|
|
|
|
def test_enforce_leverage_pass_at_cap():
|
|
creds = {"max_leverage": 3}
|
|
enforce_leverage(3, creds=creds, exchange="bybit") # no raise
|
|
|
|
|
|
def test_enforce_leverage_reject_over_cap():
|
|
creds = {"max_leverage": 3}
|
|
with pytest.raises(HTTPException) as exc:
|
|
enforce_leverage(10, creds=creds, exchange="bybit")
|
|
assert exc.value.status_code == 403
|
|
assert exc.value.detail["error"] == "LEVERAGE_CAP_EXCEEDED"
|
|
assert exc.value.detail["exchange"] == "bybit"
|
|
assert exc.value.detail["requested"] == 10
|
|
assert exc.value.detail["max"] == 3
|
|
|
|
|
|
def test_enforce_leverage_reject_when_below_one():
|
|
creds = {"max_leverage": 3}
|
|
with pytest.raises(HTTPException) as exc:
|
|
enforce_leverage(0, creds=creds, exchange="bybit")
|
|
assert exc.value.status_code == 403
|
|
|
|
|
|
def test_enforce_leverage_default_when_none():
|
|
"""Se requested è None, applica il cap come default."""
|
|
creds = {"max_leverage": 3}
|
|
result = enforce_leverage(None, creds=creds, exchange="bybit")
|
|
assert result == 3
|