tigrbl_tests 0.3.10.dev1__tar.gz → 0.3.10.dev2__tar.gz

{tigrbl_tests-0.3.10.dev1 → tigrbl_tests-0.3.10.dev2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tigrbl_tests
-Version: 0.3.10.dev1
+Version: 0.3.10.dev2
 Summary: Test suite and fixtures for the Tigrbl framework.
 License-Expression: Apache-2.0
 License-File: LICENSE

{tigrbl_tests-0.3.10.dev1 → tigrbl_tests-0.3.10.dev2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "tigrbl_tests"
-version = "0.3.10.dev1"
+version = "0.3.10.dev2"
 description = "Test suite and fixtures for the Tigrbl framework."
 license = "Apache-2.0"
 readme = "README.md"

{tigrbl_tests-0.3.10.dev1 → tigrbl_tests-0.3.10.dev2}/tests/i9n/test_mountable_lens_uvicorn.py

@@ -26,7 +26,7 @@ async def test_lens_mountable_on_tigrbl_app_uvicorn():
             in response.text
         )
         assert (
-            'href="https://esm.sh/@tigrbljs/tigrbl-lens@latest/dist/tigrbl-lens.css"'
+            'href="https://esm.sh/@tigrbljs/tigrbl-lens@latest/dist/tigrbl-lens.css?css"'
             in response.text
         )
         assert 'import { createRoot } from "react-dom/client";' in response.text

{tigrbl_tests-0.3.10.dev1 → tigrbl_tests-0.3.10.dev2}/tests/i9n/test_v3_default_rpc_ops.py

@@ -140,6 +140,96 @@ async def test_bulk_methods_absent(client_and_model):
         assert not hasattr(Gadget.rpc, name)
 
 
+@pytest_asyncio.fixture()
+async def wrapper_field_client_and_model():
+    Base3.metadata.clear()
+    Base3.registry.dispose()
+
+    class WrapperNamed(Base3):
+        __tablename__ = "wrapper_named"
+        __allow_unmapped__ = True
+
+        id: Mapped[int] = acol(
+            storage=S(type_=Integer, primary_key=True, autoincrement=True)
+        )
+        data: Mapped[str] = acol(
+            storage=S(type_=String, nullable=False),
+            field=F(required_in=("create",)),
+            io=IO(in_verbs=("create", "update"), out_verbs=("read", "list")),
+        )
+
+        __tigrbl_cols__ = {"id": id, "data": data}
+
+    app = Tigrblv3()
+    api = Tigrblv3(engine=mem())
+    api.include_model(WrapperNamed, prefix="")
+    api.mount_jsonrpc(prefix="/rpc")
+    await api.initialize()
+    app.include_router(api.router)
+    transport = ASGITransport(app=app)
+    client = AsyncClient(transport=transport, base_url="http://test")
+    try:
+        yield client, WrapperNamed
+    finally:
+        await client.aclose()
+
+
+@pytest.mark.i9n
+@pytest.mark.asyncio
+async def test_rpc_create_allows_schema_defined_wrapper_like_fields(
+    wrapper_field_client_and_model,
+):
+    client, _ = wrapper_field_client_and_model
+    payload = {
+        "jsonrpc": "2.0",
+        "method": "WrapperNamed.create",
+        "params": {"data": "legit"},
+        "id": 99,
+    }
+
+    response = await client.post("/rpc", json=payload)
+
+    assert response.status_code == 200
+    body = response.json()
+    assert "error" not in body
+    assert body["result"]["data"] == "legit"
+
+
+@pytest.mark.i9n
+@pytest.mark.asyncio
+@pytest.mark.parametrize("wrapper_key", ["data", "payload", "body", "item"])
+async def test_rpc_create_rejects_wrapper_object(wrapper_key, client_and_model):
+    client, _ = client_and_model
+    payload = {
+        "jsonrpc": "2.0",
+        "method": "Gadget.create",
+        "params": {wrapper_key: {"name": "Wrapped", "age": 7}},
+        "id": 11,
+    }
+
+    response = await client.post("/rpc", json=payload)
+
+    assert response.status_code == 200
+    error = response.json()["error"]
+    assert error["code"] == -32602
+    assert error["message"] == "Invalid params"
+    assert error["data"]["disallowed_keys"] == [wrapper_key]
+
+
+@pytest.mark.i9n
+@pytest.mark.asyncio
+@pytest.mark.parametrize("wrapper_key", ["data", "payload", "body", "item"])
+async def test_rest_create_rejects_wrapper_object(wrapper_key, client_and_model):
+    client, _ = client_and_model
+
+    response = await client.post(
+        "/gadget",
+        json={wrapper_key: {"name": "Wrapped", "age": 7}},
+    )
+
+    assert response.status_code == 422
+
+
 @pytest_asyncio.fixture()
 async def bulk_client_and_model():
     Base3.metadata.clear()
@@ -184,6 +274,31 @@ async def bulk_client_and_model():
         await client.aclose()
 
 
+@pytest.mark.i9n
+@pytest.mark.asyncio
+@pytest.mark.parametrize("wrapper_key", ["data", "payload", "body", "item"])
+async def test_rpc_bulk_create_rejects_wrapper_object_items(
+    wrapper_key,
+    bulk_client_and_model,
+):
+    client, _ = bulk_client_and_model
+
+    payload = {
+        "jsonrpc": "2.0",
+        "method": "Gadget.bulk_create",
+        "params": [{wrapper_key: {"name": "Wrapped", "age": 7}}],
+        "id": 20,
+    }
+
+    response = await client.post("/rpc", json=payload)
+
+    assert response.status_code == 200
+    error = response.json()["error"]
+    assert error["code"] == -32602
+    assert error["message"] == "Invalid params"
+    assert error["data"]["disallowed_keys"] == [wrapper_key]
+
+
 @pytest.mark.i9n
 @pytest.mark.asyncio
 async def test_rpc_bulk_ops(bulk_client_and_model):

tigrbl_tests-0.3.10.dev2/tests/unit/test_jsonrpc_openrpc.py

@@ -0,0 +1,122 @@
+from httpx import ASGITransport, Client
+from sqlalchemy import Column, String
+from tigrbl import Base, TigrblApi, TigrblApp
+from tigrbl.engine.shortcuts import mem
+from tigrbl.orm.mixins import GUIDPk
+
+
+def _build_app():
+    class Widget(Base, GUIDPk):
+        __tablename__ = "widgets_openrpc"
+        name = Column(String, nullable=False)
+
+    app = TigrblApp(engine=mem(async_=False))
+    app.include_model(Widget)
+    app.initialize()
+    app.mount_jsonrpc()
+    return app, Widget
+
+
+def test_openrpc_endpoint_exposed():
+    app, _ = _build_app()
+    transport = ASGITransport(app=app)
+    with Client(transport=transport, base_url="http://test") as client:
+        response = client.get("/openrpc.json")
+
+        assert response.status_code == 200
+        payload = response.json()
+        assert payload["openrpc"] == "1.2.6"
+        assert payload["servers"] == [{"name": app.title, "url": "/rpc"}]
+        assert "methods" in payload
+
+
+def test_openrpc_server_url_respects_jsonrpc_prefix():
+    app, _ = _build_app()
+    app.jsonrpc_prefix = "/rpc/custom"
+    transport = ASGITransport(app=app)
+
+    with Client(transport=transport, base_url="http://test") as client:
+        payload = client.get("/openrpc.json").json()
+
+    assert payload["servers"] == [{"name": app.title, "url": "/rpc/custom"}]
+
+
+def test_openrpc_includes_method_schema():
+    app, model = _build_app()
+    transport = ASGITransport(app=app)
+    with Client(transport=transport, base_url="http://test") as client:
+        payload = client.get("/openrpc.json").json()
+        methods = {method["name"]: method for method in payload["methods"]}
+
+        create_method = methods[f"{model.__name__}.create"]
+        assert create_method["paramStructure"] == "by-name"
+
+        params = create_method["params"][0]["schema"]
+        assert params["title"].startswith(model.__name__)
+        assert "Create" in params["title"]
+
+        result = create_method["result"]["schema"]
+        assert result["title"].startswith(model.__name__)
+        assert "Response" in result["title"]
+
+
+def test_mount_jsonrpc_updates_openrpc_server_url() -> None:
+    app, _ = _build_app()
+    app.mount_jsonrpc(prefix="/jsonrpc")
+    transport = ASGITransport(app=app)
+
+    with Client(transport=transport, base_url="http://test") as client:
+        payload = client.get("/openrpc.json").json()
+
+    assert payload["servers"] == [{"name": app.title, "url": "/jsonrpc"}]
+
+
+def test_jsonrpc_create_accepts_nested_params_mapping() -> None:
+    app, model = _build_app()
+    transport = ASGITransport(app=app)
+
+    request_payload = {
+        "jsonrpc": "2.0",
+        "method": f"{model.__name__}.create",
+        "params": {"params": {"name": "New Widget"}},
+        "id": 1,
+    }
+
+    with Client(transport=transport, base_url="http://test") as client:
+        response = client.post("/rpc", json=request_payload)
+
+    assert response.status_code == 200
+    data = response.json()
+    assert data["id"] == 1
+    assert "error" not in data
+    assert data["result"]["name"] == "New Widget"
+
+
+def test_mount_lens_uses_latest_openrpc_path_by_default() -> None:
+    app, _ = _build_app()
+    app.mount_openrpc(path="/schema/openrpc.json")
+    app.mount_lens(path="/lens-custom")
+
+    transport = ASGITransport(app=app)
+    with Client(transport=transport, base_url="http://test") as client:
+        html = client.get("/lens-custom").text
+
+    assert "/schema/openrpc.json" in html
+
+
+def test_openrpc_server_url_respects_api_mount_jsonrpc_prefix_argument():
+    class Widget(Base, GUIDPk):
+        __tablename__ = "widgets_openrpc_api_mount_prefix"
+        name = Column(String, nullable=False)
+
+    api = TigrblApi(engine=mem(async_=False))
+    api.include_model(Widget)
+    api.initialize()
+    api.mount_jsonrpc(prefix="/jsonrpc")
+    api.mount_openrpc()
+
+    transport = ASGITransport(app=api)
+    with Client(transport=transport, base_url="http://test") as client:
+        payload = client.get("/openrpc.json").json()
+
+    assert payload["servers"] == [{"name": api.title, "url": "/jsonrpc"}]

{tigrbl_tests-0.3.10.dev1 → tigrbl_tests-0.3.10.dev2}/tests/unit/test_openapi_openrpc_schema_separation.py

@@ -48,3 +48,24 @@ def test_openrpc_schema_excludes_openapi_endpoint() -> None:
     method_names = {method["name"].lower() for method in payload["methods"]}
     assert all("openapi" not in name for name in method_names)
     assert all("openrpc" not in name for name in method_names)
+
+
+def test_openapi_schema_excludes_openrpc_even_if_route_is_schema_visible() -> None:
+    app = _build_app()
+
+    def openrpc_override(_request):
+        return {"openrpc": "1.2.6"}
+
+    app.add_api_route(
+        "/openrpc.json",
+        openrpc_override,
+        methods=["GET"],
+        name="openrpc_json",
+        include_in_schema=True,
+    )
+
+    transport = ASGITransport(app=app)
+    with Client(transport=transport, base_url="http://test") as client:
+        payload = client.get("/openapi.json").json()
+
+    assert "/openrpc.json" not in payload["paths"]

tigrbl_tests-0.3.10.dev1/tests/unit/test_jsonrpc_openrpc.py

@@ -1,60 +0,0 @@
-from httpx import ASGITransport, Client
-from sqlalchemy import Column, String
-from tigrbl import Base, TigrblApp
-from tigrbl.engine.shortcuts import mem
-from tigrbl.orm.mixins import GUIDPk
-
-
-def _build_app():
-    class Widget(Base, GUIDPk):
-        __tablename__ = "widgets_openrpc"
-        name = Column(String, nullable=False)
-
-    app = TigrblApp(engine=mem(async_=False))
-    app.include_model(Widget)
-    app.initialize()
-    app.mount_jsonrpc()
-    return app, Widget
-
-
-def test_openrpc_endpoint_exposed():
-    app, _ = _build_app()
-    transport = ASGITransport(app=app)
-    with Client(transport=transport, base_url="http://test") as client:
-        response = client.get("/openrpc.json")
-
-        assert response.status_code == 200
-        payload = response.json()
-        assert payload["openrpc"] == "1.2.6"
-        assert payload["servers"] == [{"name": app.title, "url": "/rpc"}]
-        assert "methods" in payload
-
-
-def test_openrpc_server_url_respects_jsonrpc_prefix():
-    app, _ = _build_app()
-    app.jsonrpc_prefix = "/rpc/custom"
-    transport = ASGITransport(app=app)
-
-    with Client(transport=transport, base_url="http://test") as client:
-        payload = client.get("/openrpc.json").json()
-
-    assert payload["servers"] == [{"name": app.title, "url": "/rpc/custom"}]
-
-
-def test_openrpc_includes_method_schema():
-    app, model = _build_app()
-    transport = ASGITransport(app=app)
-    with Client(transport=transport, base_url="http://test") as client:
-        payload = client.get("/openrpc.json").json()
-        methods = {method["name"]: method for method in payload["methods"]}
-
-        create_method = methods[f"{model.__name__}.create"]
-        assert create_method["paramStructure"] == "by-name"
-
-        params = create_method["params"][0]["schema"]
-        assert params["title"].startswith(model.__name__)
-        assert "Create" in params["title"]
-
-        result = create_method["result"]["schema"]
-        assert result["title"].startswith(model.__name__)
-        assert "Response" in result["title"]