iii-sdk 0.19.2.dev1__tar.gz → 0.19.3.dev1__tar.gz

{iii_sdk-0.19.2.dev1 → iii_sdk-0.19.3.dev1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iii-sdk
-Version: 0.19.2.dev1
+Version: 0.19.3.dev1
 Summary: III SDK for Python
 Project-URL: Homepage, https://github.com/iii-hq/iii
 Project-URL: Repository, https://github.com/iii-hq/iii
@@ -14,7 +14,7 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.10
-Requires-Dist: iii-observability==0.19.2.dev1
+Requires-Dist: iii-observability==0.19.3.dev1
 Requires-Dist: opentelemetry-api>=1.25
 Requires-Dist: pydantic>=2.0
 Requires-Dist: websockets>=12.0

{iii_sdk-0.19.2.dev1 → iii_sdk-0.19.3.dev1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "iii-sdk"
-version = "0.19.2.dev1"
+version = "0.19.3.dev1"
 description = "III SDK for Python"
 authors = [{ name = "III" }]
 license = { text = "Apache-2.0" }
@@ -23,7 +23,7 @@ dependencies = [
     "websockets>=12.0",
     "pydantic>=2.0",
     "opentelemetry-api>=1.25",
-    "iii-observability==0.19.2.dev1",
+    "iii-observability==0.19.3.dev1",
 ]
 
 [project.urls]

{iii_sdk-0.19.2.dev1 → iii_sdk-0.19.3.dev1}/tests/test_api_triggers.py

@@ -136,6 +136,64 @@ async def test_raw_json_request_body(engine_http_url, iii_client: III):
     trigger.unregister()
 
 
+@pytest.mark.asyncio
+async def test_conflicting_route_structure_is_rejected(engine_http_url, iii_client: III, caplog):
+    """Two routes with identical structure but swapped path-param names must not
+    crash the engine: the first keeps serving and the second is rejected with a
+    logged registration error."""
+    caplog.set_level("ERROR", logger="iii")
+
+    def handler(input_data):
+        return {"status_code": 200, "body": {"ok": True}}
+
+    # First route registers normally.
+    fn_a = iii_client.register_function("test.api.conflict.a.py", handler)
+    trig_a = iii_client.register_trigger(
+        {
+            "type": "http",
+            "function_id": "test.api.conflict.a.py",
+            "config": {
+                "api_path": "test/py/conflict/:listId/:userId",
+                "http_method": "GET",
+            },
+        }
+    )
+
+    # Second route has the same axum shape with swapped param names -> conflict.
+    fn_b = iii_client.register_function("test.api.conflict.b.py", handler)
+    trig_b = iii_client.register_trigger(
+        {
+            "type": "http",
+            "function_id": "test.api.conflict.b.py",
+            "config": {
+                "api_path": "test/py/conflict/:userId/:listId",
+                "http_method": "GET",
+            },
+        }
+    )
+
+    # Give the engine time to process both registrations and reply.
+    time.sleep(0.5)
+
+    # Engine stayed alive and the first route still serves — no panic.
+    async with aiohttp.ClientSession() as session:
+        async with session.get(f"{engine_http_url}/test/py/conflict/list1/user1") as resp:
+            assert resp.status == 200
+            data = await resp.json()
+            assert data["ok"] is True
+
+    # The conflicting registration was surfaced as an error. The engine rejects
+    # whichever route it processes second (wire order is not guaranteed), so assert on
+    # the conflict message rather than a specific route or the random trigger id.
+    messages = [record.getMessage() for record in caplog.records]
+    assert any("conflicts with already-registered route" in m for m in messages), messages
+
+    fn_a.unregister()
+    trig_a.unregister()
+    fn_b.unregister()
+    trig_b.unregister()
+
+
 @pytest.mark.asyncio
 async def test_path_parameters(engine_http_url, iii_client: III):
     """Verify path parameters are extracted correctly."""