test: add create_order tests for validate_symbol data source check (#247)

Add 3 tests covering create_order with validate_symbol:
- rejects when no data source registered for the pair
- accepts when matching data source exists
- default behavior skips validation

Author: MDUYN

tests/app/algorithm/test_validate_symbol.py

@@ -1,5 +1,5 @@
 from investing_algorithm_framework import PortfolioConfiguration, \
-    MarketCredential, OrderSide, DataSource
+    MarketCredential, OrderSide, OrderType, DataSource
 from investing_algorithm_framework.domain import OperationalException
 from tests.resources import TestBase
 from tests.resources.strategies_for_testing import StrategyOne
@@ -170,4 +170,57 @@ def test_error_message_lists_registered_symbols(self):
         self.assertIn("BTC/EUR", error_msg)
         self.assertIn("ETH/EUR", error_msg)
         self.assertIn("validate_symbol=False", error_msg)
-        self.assertIn("validate_symbol=False", error_msg)
+
+    # ── create_order tests ──────────────────────────────────────────
+
+    def test_create_order_rejects_missing_data_source(self):
+        """create_order with validate_symbol=True rejects when no
+        data source is registered for the pair."""
+        self.app.add_strategy(StrategyOne)
+
+        with self.assertRaises(OperationalException) as cm:
+            self.app.context.create_order(
+                target_symbol="BTC",
+                amount=1,
+                price=10,
+                order_type=OrderType.LIMIT,
+                order_side=OrderSide.BUY,
+                validate_symbol=True,
+            )
+
+        error_msg = str(cm.exception)
+        self.assertIn("BTC/EUR", error_msg)
+        self.assertIn("No data source registered", error_msg)
+
+    def test_create_order_accepts_with_matching_data_source(self):
+        """create_order with validate_symbol=True passes when a
+        data source exists for the pair."""
+        self.app.add_strategy(StrategyOne)
+        self._register_data_source("BTC/EUR")
+
+        self.app.context.create_order(
+            target_symbol="BTC",
+            amount=1,
+            price=10,
+            order_type=OrderType.LIMIT,
+            order_side=OrderSide.BUY,
+            validate_symbol=True,
+        )
+        order_repository = self.app.container.order_repository()
+        order = order_repository.find({"target_symbol": "BTC"})
+        self.assertIsNotNone(order)
+
+    def test_create_order_default_skips_validation(self):
+        """create_order without validate_symbol allows any symbol."""
+        self.app.add_strategy(StrategyOne)
+
+        self.app.context.create_order(
+            target_symbol="UNKNOWN",
+            amount=1,
+            price=10,
+            order_type=OrderType.LIMIT,
+            order_side=OrderSide.BUY,
+        )
+        order_repository = self.app.container.order_repository()
+        order = order_repository.find({"target_symbol": "UNKNOWN"})
+        self.assertIsNotNone(order)

tests/resources/backtest_reports_for_testing/test_algorithm_backtest/algorithm_id.json

@@ -1,3 +1,3 @@
 {
-    "algorithm_id": "TestStrategy"
+    "algorithm_id": "BacktestTestStrategy"
 }

tests/resources/backtest_reports_for_testing/test_algorithm_backtest/runs/backtest_EUR_20231201_20231202/run.json

@@ -1 +1 @@
-{"backtest_start_date": "2023-12-01 00:00:00", "backtest_date_range_name": null, "backtest_end_date": "2023-12-02 00:00:00", "trading_symbol": "EUR", "initial_unallocated": 1000.0, "number_of_runs": 1441, "portfolio_snapshots": [{"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-01T00:00:00+00:00", "total_value": 1000.0}, {"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-02T00:00:00+00:00", "total_value": 1000.0}], "trades": [], "orders": [], "positions": [{"symbol": "EUR", "amount": 1000.0, "cost": 1000.0, "portfolio_id": 1}], "created_at": "2026-04-20 19:53:20", "symbols": [], "number_of_days": 0, "number_of_trades": 0, "number_of_trades_closed": 0, "number_of_trades_open": 0, "number_of_orders": 0, "number_of_positions": 0, "metadata": {}, "signals": {}, "signal_events": []}
+{"backtest_start_date": "2023-12-01 00:00:00", "backtest_date_range_name": null, "backtest_end_date": "2023-12-02 00:00:00", "trading_symbol": "EUR", "initial_unallocated": 1000.0, "number_of_runs": 1441, "portfolio_snapshots": [{"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-01T00:00:00+00:00", "total_value": 1000.0}, {"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-02T00:00:00+00:00", "total_value": 1000.0}], "trades": [], "orders": [], "positions": [{"symbol": "EUR", "amount": 1000.0, "cost": 1000.0, "portfolio_id": 1}], "created_at": "2026-04-22 08:04:47", "symbols": [], "number_of_days": 0, "number_of_trades": 0, "number_of_trades_closed": 0, "number_of_trades_open": 0, "number_of_orders": 0, "number_of_positions": 0, "metadata": {}, "signals": {}, "signal_events": []}

tests/resources/backtest_reports_for_testing/test_algorithm_backtest/runs/backtest_EUR_20231202_20231203/run.json

@@ -1 +1 @@
-{"backtest_start_date": "2023-12-02 00:00:00", "backtest_date_range_name": null, "backtest_end_date": "2023-12-03 00:00:00", "trading_symbol": "EUR", "initial_unallocated": 1000.0, "number_of_runs": 1441, "portfolio_snapshots": [{"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-02T00:00:00+00:00", "total_value": 1000.0}, {"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-03T00:00:00+00:00", "total_value": 1000.0}], "trades": [], "orders": [], "positions": [{"symbol": "EUR", "amount": 1000.0, "cost": 1000.0, "portfolio_id": 1}], "created_at": "2026-04-20 19:56:42", "symbols": [], "number_of_days": 0, "number_of_trades": 0, "number_of_trades_closed": 0, "number_of_trades_open": 0, "number_of_orders": 0, "number_of_positions": 0, "metadata": {}, "signals": {}, "signal_events": []}
+{"backtest_start_date": "2023-12-02 00:00:00", "backtest_date_range_name": null, "backtest_end_date": "2023-12-03 00:00:00", "trading_symbol": "EUR", "initial_unallocated": 1000.0, "number_of_runs": 1441, "portfolio_snapshots": [{"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-02T00:00:00+00:00", "total_value": 1000.0}, {"metadata": "MetaData()", "portfolio_id": "1", "trading_symbol": "EUR", "pending_value": 0.0, "unallocated": 1000.0, "total_net_gain": 0.0, "total_revenue": 0.0, "total_cost": 0.0, "cash_flow": 0.0, "net_size": 1000.0, "created_at": "2023-12-03T00:00:00+00:00", "total_value": 1000.0}], "trades": [], "orders": [], "positions": [{"symbol": "EUR", "amount": 1000.0, "cost": 1000.0, "portfolio_id": 1}], "created_at": "2026-04-22 08:04:18", "symbols": [], "number_of_days": 0, "number_of_trades": 0, "number_of_trades_closed": 0, "number_of_trades_open": 0, "number_of_orders": 0, "number_of_positions": 0, "metadata": {}, "signals": {}, "signal_events": []}