item: devices-read-side — get_push_devices and get_push_device, sync and async, 37 -> 39 routes

Author: Domantas

docs/hub/push.md

@@ -80,6 +80,28 @@ client.hub.notifications.register_device(
 )
 ```
 
+## Listing registered devices
+
+`get_push_devices` returns the devices registered in the project, each with
+the user it belongs to. Narrow it with `userId`, `deviceKey` (the provider
+token) or `platform` (`ios`, `android`, `chrome`, `safari`, `expo`); a word
+outside that list is refused rather than answered with an empty page.
+
+```python
+devices = client.hub.notifications.get_push_devices(platform="ios")
+```
+
+Devices are stored inside their user, so a page is a page of **users** and
+carries every matching device those users hold. Follow `hasMore` rather than
+stopping at the first short page.
+
+`get_push_device` takes one device id and answers with the device and its
+owner:
+
+```python
+device = client.hub.notifications.get_push_device("pnd_123")
+```
+
 ## Known gaps
 
 | what | why |

src/norbix_python/hub/notifications.py

@@ -919,6 +919,36 @@ def register_device(self, *, timeout: float | None = None, bearer_token: str | N
             bearer_token=bearer_token,
         )
 
+    def get_push_devices(self, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
+        """GET /{version}/notifications/push/devices
+
+        Narrow the list with userId, deviceKey (the provider token) or
+        platform (ios, android, chrome, safari, expo).
+        """
+        return self._transport.send(
+            target="hub",
+            path="/{version}/notifications/push/devices",
+            method="GET",
+            path_params={},
+            request=request,
+            scope="project",
+            timeout=timeout,
+            bearer_token=bearer_token,
+        )
+
+    def get_push_device(self, id: str, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
+        """GET /{version}/notifications/push/devices/{id}"""
+        return self._transport.send(
+            target="hub",
+            path="/{version}/notifications/push/devices/{id}",
+            method="GET",
+            path_params={"id": id},
+            request=request,
+            scope="project",
+            timeout=timeout,
+            bearer_token=bearer_token,
+        )
+
     def get_all_contacts(self, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
         """GET /{version}/notifications/contacts"""
         return self._transport.send(
@@ -2627,6 +2657,36 @@ async def register_device(self, *, timeout: float | None = None, bearer_token: s
             bearer_token=bearer_token,
         )
 
+    async def get_push_devices(self, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
+        """GET /{version}/notifications/push/devices
+
+        Narrow the list with userId, deviceKey (the provider token) or
+        platform (ios, android, chrome, safari, expo).
+        """
+        return await self._transport.send(
+            target="hub",
+            path="/{version}/notifications/push/devices",
+            method="GET",
+            path_params={},
+            request=request,
+            scope="project",
+            timeout=timeout,
+            bearer_token=bearer_token,
+        )
+
+    async def get_push_device(self, id: str, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
+        """GET /{version}/notifications/push/devices/{id}"""
+        return await self._transport.send(
+            target="hub",
+            path="/{version}/notifications/push/devices/{id}",
+            method="GET",
+            path_params={"id": id},
+            request=request,
+            scope="project",
+            timeout=timeout,
+            bearer_token=bearer_token,
+        )
+
     async def get_all_contacts(self, *, timeout: float | None = None, bearer_token: str | None = None, **request: Any) -> Any:
         """GET /{version}/notifications/contacts"""
         return await self._transport.send(

tests/hub/test_push.py

@@ -3,6 +3,7 @@
 import json
 from collections.abc import Callable
 from typing import Any
+from urllib.parse import parse_qs, urlparse
 
 import pytest
 
@@ -12,7 +13,7 @@
 
 The generated test file asserts each push method's verb and that the URL is
 https. This file adds the two things it cannot see: the fully-resolved path of
-every one of the 37 push routes, and the polymorphic bodies that the server
+every one of the 39 push routes, and the polymorphic bodies that the server
 routes on.
 
 Only the Fake provider is used for a send path — it is the sandbox that accepts
@@ -25,6 +26,7 @@
 NOTIFICATION_ID = "notif_1"
 TEMPLATE_ID = "tpl_1"
 INTEGRATION_ID = "int_1"
+DEVICE_ID = "pnd_1"
 
 BASE = "/v2/notifications/push"
 
@@ -201,6 +203,13 @@
     ),
     # devices
     ("register_device", "POST", f"{BASE}/devices", lambda m: m.register_device()),
+    ("get_push_devices", "GET", f"{BASE}/devices", lambda m: m.get_push_devices()),
+    (
+        "get_push_device",
+        "GET",
+        f"{BASE}/devices/{DEVICE_ID}",
+        lambda m: m.get_push_device(DEVICE_ID),
+    ),
 ]
 
 
@@ -225,8 +234,8 @@ def test_push_endpoint_hits_the_expected_route(
 
 
 def test_push_surface_size() -> None:
-    """37 live push routes. A new one changes this count, so it cannot arrive untested."""
-    assert len(PUSH_CASES) == 37
+    """39 live push routes. A new one changes this count, so it cannot arrive untested."""
+    assert len(PUSH_CASES) == 39
 
 
 def test_push_call_sends_auth_and_project_headers() -> None:
@@ -382,6 +391,36 @@ def test_register_device_sends_the_device_shape() -> None:
     assert body["userId"] == "user_1"
 
 
+def test_get_push_devices_narrows_by_user_token_and_platform() -> None:
+    """Gateway Hub.Push/Devices/GetAll.cs: the three filters travel as query fields."""
+    client, transport = make_client()
+    client.hub.notifications.get_push_devices(
+        userId="user_1", deviceKey="device_1", platform="ios"
+    )
+
+    assert transport.last_request is not None
+    assert transport.last_request["method"] == "GET"
+
+    # A GET carries its filters in the query string, not a body.
+    url = urlparse(transport.last_request["url"])
+    assert url.path == f"{BASE}/devices"
+    assert parse_qs(url.query) == {
+        "userId": ["user_1"],
+        "deviceKey": ["device_1"],
+        "platform": ["ios"],
+    }
+
+
+def test_get_push_device_fills_the_route_token() -> None:
+    """The device id is a route token, not a body field."""
+    client, transport = make_client()
+    client.hub.notifications.get_push_device(DEVICE_ID)
+
+    assert transport.last_request is not None
+    assert transport.last_request["method"] == "GET"
+    assert urlparse(transport.last_request["url"]).path == f"{BASE}/devices/{DEVICE_ID}"
+
+
 def test_async_module_exposes_the_same_push_surface() -> None:
     """Every sync push method must have an async twin."""
     from norbix_python.hub.notifications import AsyncNotificationsModule, NotificationsModule