[client-python] Add tests multi-tenant

Author: Megafredo

test/bdd/constraints/multi_tenant_api_routing_constraint.feature

@@ -0,0 +1,12 @@
+Feature: URL normalization in OpenAEV client
+
+  Scenario Outline: URL normalization combines base_url and path correctly
+    Given an OpenAEV client with base_url "<base_url>"
+    When I build the URL for "<path>"
+    Then the resulting URL should be "<expected>"
+
+    Examples:
+      | base_url    | path       | expected              |
+      | base_url    | path       | base_url/api/path    |
+      | base_url/   | /path      | base_url/api/path    |
+      | base_url//  | //path     | base_url/api/path    |

test/bdd/constraints/test_multi_tenant_api_routing_constraint.py

@@ -0,0 +1,38 @@
+import pytest
+
+from pyoaev import OpenAEV
+
+
+@pytest.mark.parametrize(
+    "base_url, input_path, expected",
+    [
+        (
+            "base_url",
+            "path",
+            "base_url/api/path",
+        ),
+        (
+            "base_url/",
+            "/path",
+            "base_url/api/path",
+        ),
+        (
+            "base_url//",
+            "//path",
+            "base_url/api/path",
+        ),
+    ],
+    ids=[
+        "clean-base-url-and-relative-path",
+        "base-url-trailing-slash",
+        "base-url-double-slash-and-path-double-slash",
+    ],
+)
+def test_url_normalization(base_url, input_path, expected):
+    client = OpenAEV(
+        url=base_url,
+        token="token",
+        tenant_id=None,
+    )
+    result = client._build_url(input_path)
+    assert result == expected

test/bdd/features/multi_tenant_api_routing.feature

@@ -0,0 +1,17 @@
+Feature: Multi-tenant API routing in OpenAEV client
+
+  Scenario: Full URL bypasses tenant routing
+    Given an OpenAEV client with tenant_id "2cffad3a-0001-4078-b0e2-ef74274022c3"
+    When I build the URL for "https://external.service/api/path"
+    Then the resulting URL should be "https://external.service/api/path"
+
+  Scenario Outline: Relative path routing behavior
+    Given an OpenAEV client with tenant_id "<tenant_id>"
+    When I build the URL for "/path"
+    Then the resulting URL should be "<output>"
+
+    Examples:
+      | tenant_id                             | output                                                          |
+      | None                                  | base_url/api/path                                               |
+      | 2cffad3a-0001-4078-b0e2-ef74274022c3  | base_url/api/tenants/2cffad3a-0001-4078-b0e2-ef74274022c3/path  |
+

test/bdd/features/multi_tenant_base_daemon_propagation.feature

@@ -0,0 +1,12 @@
+Feature: Tenant propagation in BaseDaemon API client initialization
+
+  Scenario Outline: BaseDaemon propagates tenant_id correctly from configuration
+    Given a daemon configuration with <tenant_id>
+    When the BaseDaemon is initialized
+    Then the API client should be created with tenant_id "<expected_tenant_id>"
+
+    Examples:
+      | tenant_id                               | expected_tenant_id                     |
+      | Missing tenant key                      | None                                   |
+      | None                                    | None                                   |
+      | 2cffad3a-0001-4078-b0e2-ef74274022c3    | 2cffad3a-0001-4078-b0e2-ef74274022c3   |

test/bdd/features/multi_tenant_endpoint_search_targets.feature

@@ -0,0 +1,12 @@
+Feature: searchTargets API routing with and without tenant_id
+
+  Scenario Outline: searchTargets routing behavior
+    Given an OpenAEV client with tenant_id "<tenant_id>"
+    And a valid SearchPaginationInput
+    When I call searchTargets on endpoint
+    Then the request URL should be "<expected_url>"
+
+    Examples:
+      | tenant_id | expected_url |
+      | None | url/api/endpoints/targets |
+      | 2cffad3a-0001-4078-b0e2-ef74274022c3 | url/api/tenants/2cffad3a-0001-4078-b0e2-ef74274022c3/endpoints/targets |

test/bdd/features/test_multi_tenant_api_routing.py

@@ -0,0 +1,46 @@
+from uuid import UUID
+
+import pytest
+
+from pyoaev import OpenAEV
+
+
+@pytest.mark.parametrize(
+    "tenant_id, path, expected",
+    [
+        (
+            None,
+            "/path",
+            "base_url/api/path",
+        ),
+        (
+            UUID("2cffad3a-0001-4078-b0e2-ef74274022c3"),
+            "/path",
+            "base_url/api/tenants/2cffad3a-0001-4078-b0e2-ef74274022c3/path",
+        ),
+        (
+            None,
+            "https://external.service/api/path",
+            "https://external.service/api/path",
+        ),
+        (
+            UUID("2cffad3a-0001-4078-b0e2-ef74274022c3"),
+            "https://external.service/api/path",
+            "https://external.service/api/path",
+        ),
+    ],
+    ids=[
+        "legacy-relative-path",
+        "tenant-relative-path",
+        "legacy-full-url-bypass",
+        "tenant-full-url-bypass",
+    ],
+)
+def test_build_url_behavior(tenant_id, path, expected):
+    client = OpenAEV(
+        "base_url",
+        "token",
+        tenant_id=tenant_id,
+    )
+    result = client._build_url(path)
+    assert result == expected

test/bdd/features/test_multi_tenant_base_daemon_propagation.py

@@ -0,0 +1,72 @@
+from unittest.mock import MagicMock
+from uuid import UUID
+
+import pytest
+
+from pyoaev.daemons.base_daemon import BaseDaemon
+
+
+class DummyDaemon(BaseDaemon):
+    def _setup(self):
+        pass
+
+    def _start_loop(self):
+        pass
+
+
+@pytest.mark.parametrize(
+    "config_map, expected_tenant",
+    [
+        (
+            {
+                "openaev_url": "url",
+                "openaev_token": "token",
+            },
+            None,
+        ),
+        (
+            {
+                "openaev_url": "url",
+                "openaev_token": "token",
+                "openaev_tenant_id": None,
+            },
+            None,
+        ),
+        (
+            {
+                "openaev_url": "url",
+                "openaev_token": "token",
+                "openaev_tenant_id": UUID("2cffad3a-0001-4078-b0e2-ef74274022c3"),
+            },
+            UUID("2cffad3a-0001-4078-b0e2-ef74274022c3"),
+        ),
+    ],
+    ids=[
+        "missing_tenant_key",
+        "explicit_none_tenant",
+        "valid_uuid_tenant",
+    ],
+)
+def test_default_api_client_propagates_tenant_id(
+    monkeypatch, config_map, expected_tenant
+):
+    captured = {}
+
+    def fake_client(url, token, tenant_id=None):
+        captured["url"] = url
+        captured["token"] = token
+        captured["tenant_id"] = tenant_id
+        return MagicMock()
+
+    monkeypatch.setattr("pyoaev.daemons.base_daemon.OpenAEV", fake_client)
+
+    config = MagicMock()
+    config.get.side_effect = lambda key: config_map.get(key)
+
+    daemon = DummyDaemon(configuration=config)
+    assert daemon.api is not None
+
+    assert "openaev_tenant_id" in config_map or expected_tenant is None
+    assert captured["url"] == "url"
+    assert captured["token"] == "token"
+    assert captured["tenant_id"] == expected_tenant

test/bdd/features/test_multi_tenant_endpoint_search_targets.py

@@ -0,0 +1,73 @@
+from unittest.mock import MagicMock
+from uuid import UUID
+
+import pytest
+
+from pyoaev import OpenAEV
+from pyoaev.apis.inputs.search import Filter, FilterGroup, SearchPaginationInput
+
+
+class MockResponse:
+    def __init__(self, json_data=None, status_code=200):
+        self._json_data = json_data
+        self.status_code = status_code
+        self.history = None
+        self.content = None
+        self.headers = {"Content-Type": "application/json"}
+
+    def json(self):
+        return self._json_data or {}
+
+
+def build_search_input():
+    return SearchPaginationInput(
+        0,
+        20,
+        FilterGroup(
+            "or",
+            [
+                Filter(
+                    "targets",
+                    "and",
+                    "eq",
+                    ["target_1", "target_2", "target_3"],
+                )
+            ],
+        ),
+        None,
+        None,
+    )
+
+
+@pytest.mark.parametrize(
+    "tenant_id, expected_url",
+    [
+        (
+            None,
+            "url/api/endpoints/targets",
+        ),
+        (
+            UUID("2cffad3a-0001-4078-b0e2-ef74274022c3"),
+            "url/api/tenants/2cffad3a-0001-4078-b0e2-ef74274022c3/endpoints/targets",
+        ),
+    ],
+    ids=[
+        "legacy_routing_no_tenant",
+        "tenant_routing_enabled",
+    ],
+)
+def test_search_input_correctly_serialised(monkeypatch, tenant_id, expected_url):
+    mock_request = MagicMock(return_value=MockResponse())
+    monkeypatch.setattr("requests.Session.request", mock_request)
+
+    api_client = OpenAEV("url", "token", tenant_id=tenant_id)
+    search_input = build_search_input()
+    expected_json = search_input.to_dict()
+    api_client.endpoint.searchTargets(search_input)
+
+    assert mock_request.call_count == 1
+    _, kwargs = mock_request.call_args
+
+    assert kwargs["method"] == "post"
+    assert kwargs["json"] == expected_json
+    assert kwargs["url"] == expected_url