From 100aa624ad3f8ad466a95f9da8af30f31f77cc9c Mon Sep 17 00:00:00 2001 From: TheSiahxyz <164138827+TheSiahxyz@users.noreply.github.com> Date: Wed, 1 Apr 2026 16:24:30 +0900 Subject: fix: resolve lint issues and final integration fixes - Fix ambiguous variable name in binance_rest.py - Remove unused volumes variable in volume_profile_strategy.py - Fix import ordering in backtester main.py and test_metrics.py - Auto-format all files with ruff --- services/order-executor/tests/test_executor.py | 1 + services/order-executor/tests/test_risk_manager.py | 8 +++----- 2 files changed, 4 insertions(+), 5 deletions(-) (limited to 'services/order-executor/tests') diff --git a/services/order-executor/tests/test_executor.py b/services/order-executor/tests/test_executor.py index 4836ffb..e64b6c0 100644 --- a/services/order-executor/tests/test_executor.py +++ b/services/order-executor/tests/test_executor.py @@ -1,4 +1,5 @@ """Tests for OrderExecutor.""" + from decimal import Decimal from unittest.mock import AsyncMock, MagicMock diff --git a/services/order-executor/tests/test_risk_manager.py b/services/order-executor/tests/test_risk_manager.py index f6b5545..a122d16 100644 --- a/services/order-executor/tests/test_risk_manager.py +++ b/services/order-executor/tests/test_risk_manager.py @@ -1,9 +1,9 @@ """Tests for RiskManager.""" + from decimal import Decimal -import pytest -from shared.models import OrderSide, Position, Signal +from shared.models import OrderSide, Signal from order_executor.risk_manager import RiskManager @@ -55,9 +55,7 @@ def test_risk_check_rejects_daily_loss_exceeded(): """Daily PnL of -1100 on 10000 balance = -11%, exceeding -10% limit.""" rm = make_risk_manager(daily_loss_limit_pct="10.0") signal = make_signal(side=OrderSide.BUY, price="100", quantity="0.1") - result = rm.check( - signal, balance=Decimal("10000"), positions={}, daily_pnl=Decimal("-1100") - ) + result = rm.check(signal, balance=Decimal("10000"), positions={}, daily_pnl=Decimal("-1100")) assert result.allowed is False assert result.reason == "Daily loss limit exceeded" -- cgit v1.2.3