open-api-mt5 0.7.2__tar.gz → 0.7.3__tar.gz

{open_api_mt5-0.7.2 → open_api_mt5-0.7.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: open-api-mt5
-Version: 0.7.2
+Version: 0.7.3
 Summary: REST and WebSocket API project for MetaTrader 5
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

open_api_mt5-0.7.3/app/services/connection_service.py

@@ -0,0 +1,72 @@
+from __future__ import annotations
+
+from pathlib import PurePath, PureWindowsPath
+from typing import Any
+
+import MetaTrader5 as mt5
+
+
+def _path_parts(value: str) -> PurePath:
+    if "\\" in value or ":" in value:
+        return PureWindowsPath(value)
+    return PurePath(value)
+
+
+def _normalize_terminal_dir(value: str | None) -> str | None:
+    if not value:
+        return None
+
+    cleaned = value.strip().strip('"')
+    if not cleaned:
+        return None
+
+    parsed = _path_parts(cleaned)
+    if parsed.name.lower() in {"terminal.exe", "terminal64.exe"}:
+        parsed = parsed.parent
+    return str(parsed).replace("/", "\\").rstrip("\\").casefold()
+
+
+def _connected_to_requested_terminal(path: str) -> bool:
+    terminal_info = mt5.terminal_info()
+    connected_path = _normalize_terminal_dir(getattr(terminal_info, "path", None))
+    requested_path = _normalize_terminal_dir(path)
+    return connected_path is None or requested_path is None or connected_path == requested_path
+
+
+def initialize_connection(
+    username: str | int | None,
+    password: str | None,
+    server: str | None,
+    path: str | None = None,
+    portable: bool | None = None,
+) -> tuple[bool, str]:
+    init_kwargs: dict[str, Any] = {}
+    if path:
+        init_kwargs["portable"] = True if portable is None else portable
+    elif portable is not None:
+        init_kwargs["portable"] = portable
+    if username not in (None, ""):
+        try:
+            init_kwargs["login"] = int(username)
+        except ValueError:
+            return False, "failed: username/login must be a number"
+    if password:
+        init_kwargs["password"] = password
+    if server:
+        init_kwargs["server"] = server
+
+    initialized = mt5.initialize(path, **init_kwargs) if path else mt5.initialize(**init_kwargs)
+    if initialized:
+        if path and not _connected_to_requested_terminal(path):
+            connected_path = getattr(mt5.terminal_info(), "path", "unknown")
+            mt5.shutdown()
+            return False, f"failed: connected to terminal at '{connected_path}' instead of requested path '{path}'"
+        return True, "connected"
+    return False, f"failed: {mt5.last_error()}"
+
+
+def shutdown_connection() -> tuple[bool, str]:
+    mt5.shutdown()
+    if mt5.terminal_info() is None:
+        return True, "disconnected"
+    return False, "failed: terminal still connected"

{open_api_mt5-0.7.2 → open_api_mt5-0.7.3}/open_api_mt5.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: open-api-mt5
-Version: 0.7.2
+Version: 0.7.3
 Summary: REST and WebSocket API project for MetaTrader 5
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{open_api_mt5-0.7.2 → open_api_mt5-0.7.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "open-api-mt5"
-version = "0.7.2"
+version = "0.7.3"
 description = "REST and WebSocket API project for MetaTrader 5"
 readme = "README.md"
 requires-python = ">=3.11"

open_api_mt5-0.7.3/tests/test_connection_service.py

@@ -0,0 +1,100 @@
+from __future__ import annotations
+
+from types import SimpleNamespace
+
+from app.services import connection_service
+
+
+def test_initialize_connection_uses_portable_mode_with_terminal_path(monkeypatch):
+    captured = {"args": None, "kwargs": None}
+
+    def initialize(*args, **kwargs):
+        captured["args"] = args
+        captured["kwargs"] = kwargs
+        return True
+
+    monkeypatch.setattr(connection_service.mt5, "initialize", initialize, raising=False)
+    monkeypatch.setattr(
+        connection_service.mt5,
+        "terminal_info",
+        lambda: SimpleNamespace(path=r"C:\MT5-A"),
+        raising=False,
+    )
+
+    assert connection_service.initialize_connection(
+        username="123",
+        password="secret",
+        server="Broker-Server",
+        path=r"C:\MT5-A\terminal64.exe",
+    ) == (True, "connected")
+    assert captured["args"] == (r"C:\MT5-A\terminal64.exe",)
+    assert captured["kwargs"] == {
+        "portable": True,
+        "login": 123,
+        "password": "secret",
+        "server": "Broker-Server",
+    }
+
+
+def test_initialize_connection_allows_disabling_portable_mode(monkeypatch):
+    captured = {"args": None, "kwargs": None}
+
+    def initialize(*args, **kwargs):
+        captured["args"] = args
+        captured["kwargs"] = kwargs
+        return True
+
+    monkeypatch.setattr(
+        connection_service.mt5,
+        "initialize",
+        initialize,
+        raising=False,
+    )
+    monkeypatch.setattr(
+        connection_service.mt5,
+        "terminal_info",
+        lambda: SimpleNamespace(path=r"C:\MT5-A"),
+        raising=False,
+    )
+
+    assert connection_service.initialize_connection(
+        username="123",
+        password=None,
+        server=None,
+        path=r"C:\MT5-A\terminal64.exe",
+        portable=False,
+    ) == (True, "connected")
+    assert captured["args"] == (r"C:\MT5-A\terminal64.exe",)
+    assert captured["kwargs"] == {
+        "portable": False,
+        "login": 123,
+    }
+
+
+def test_initialize_connection_rejects_wrong_running_terminal(monkeypatch):
+    shutdown_called = False
+
+    monkeypatch.setattr(connection_service.mt5, "initialize", lambda *_args, **_kwargs: True, raising=False)
+    monkeypatch.setattr(
+        connection_service.mt5,
+        "terminal_info",
+        lambda: SimpleNamespace(path=r"C:\MT5-B"),
+        raising=False,
+    )
+
+    def shutdown():
+        nonlocal shutdown_called
+        shutdown_called = True
+
+    monkeypatch.setattr(connection_service.mt5, "shutdown", shutdown, raising=False)
+
+    ok, message = connection_service.initialize_connection(
+        username="123",
+        password=None,
+        server=None,
+        path=r"C:\MT5-A\terminal64.exe",
+    )
+
+    assert ok is False
+    assert "instead of requested path" in message
+    assert shutdown_called is True

open_api_mt5-0.7.2/app/services/connection_service.py

@@ -1,41 +0,0 @@
-from __future__ import annotations
-
-from typing import Any
-
-import MetaTrader5 as mt5
-
-
-def initialize_connection(
-    username: str | int | None,
-    password: str | None,
-    server: str | None,
-    path: str | None = None,
-    portable: bool | None = None,
-) -> tuple[bool, str]:
-    init_kwargs: dict[str, Any] = {}
-    if path:
-        init_kwargs["path"] = path
-        init_kwargs["portable"] = True if portable is None else portable
-    elif portable is not None:
-        init_kwargs["portable"] = portable
-    if username not in (None, ""):
-        try:
-            init_kwargs["login"] = int(username)
-        except ValueError:
-            return False, "failed: username/login must be a number"
-    if password:
-        init_kwargs["password"] = password
-    if server:
-        init_kwargs["server"] = server
-
-    initialized = mt5.initialize(**init_kwargs)
-    if initialized:
-        return True, "connected"
-    return False, f"failed: {mt5.last_error()}"
-
-
-def shutdown_connection() -> tuple[bool, str]:
-    mt5.shutdown()
-    if mt5.terminal_info() is None:
-        return True, "disconnected"
-    return False, "failed: terminal still connected"

open_api_mt5-0.7.2/tests/test_connection_service.py

@@ -1,50 +0,0 @@
-from __future__ import annotations
-
-from app.services import connection_service
-
-
-def test_initialize_connection_uses_portable_mode_with_terminal_path(monkeypatch):
-    captured = {}
-
-    def initialize(**kwargs):
-        captured.update(kwargs)
-        return True
-
-    monkeypatch.setattr(connection_service.mt5, "initialize", initialize, raising=False)
-
-    assert connection_service.initialize_connection(
-        username="123",
-        password="secret",
-        server="Broker-Server",
-        path=r"C:\MT5-A\terminal64.exe",
-    ) == (True, "connected")
-    assert captured == {
-        "path": r"C:\MT5-A\terminal64.exe",
-        "portable": True,
-        "login": 123,
-        "password": "secret",
-        "server": "Broker-Server",
-    }
-
-
-def test_initialize_connection_allows_disabling_portable_mode(monkeypatch):
-    captured = {}
-    monkeypatch.setattr(
-        connection_service.mt5,
-        "initialize",
-        lambda **kwargs: captured.update(kwargs) or True,
-        raising=False,
-    )
-
-    assert connection_service.initialize_connection(
-        username="123",
-        password=None,
-        server=None,
-        path=r"C:\MT5-A\terminal64.exe",
-        portable=False,
-    ) == (True, "connected")
-    assert captured == {
-        "path": r"C:\MT5-A\terminal64.exe",
-        "portable": False,
-        "login": 123,
-    }