Files
app/backend/tests/flow/test_connector.py
T
stroblme f52d9c3077 Service node ends its call when the loop cancels every task; test order and timing leaks
_serve_once treated a cancelled call future as finished, so an event loop
cancelling all tasks at once (shutdown) never sent the worker a cancel and the
call's thread kept asyncio.run waiting for SILENCE_S. The cameras audio test
now removes its voice flow and device, the unpaired-frame test waits for its
first frame before the unpair, and test_connector no longer inherits the
controller an app test bound to demand.
2026-10-27 21:23:40 +01:00

337 lines
10 KiB
Python

"""The connector contract: polling, deduplication, health and discovery."""
import asyncio
from collections.abc import Callable
from types import SimpleNamespace
from typing import Any
import pytest
from fluksio.flow.artifacts import ArtifactStore
from fluksio.flow.connector import CONTRACT_VERSION, ConnectorNode
from fluksio.flow.controller import NODE_TYPES
from fluksio.flow.messages import DType, MessageSpec
from fluksio.flow.nodes import Node
from fluksio.flow.pipeline import Pipeline
from fluksio.flow.plugins import load_plugins
from fluksio.flow.supervision import Supervisor
class Sensor(ConnectorNode):
contract = CONTRACT_VERSION
title = "Test sensor"
description = "Reads whatever it is told to."
class Params(ConnectorNode.Params):
secret_token: str | None = None
def __init__(self, readings: list[Any] | None = None, **kwargs: Any) -> None:
super().__init__(**kwargs)
self._readings = list(readings or [])
self.polls = 0
async def poll(self) -> dict[str, Any] | None:
self.polls += 1
if not self._readings:
return None
value = self._readings.pop(0)
if isinstance(value, Exception):
raise value
return {"reading": value}
def a_sensor(readings: list[Any], **params: Any) -> Sensor:
node = Sensor(
readings=readings,
provides=[MessageSpec(name="reading", dtype=DType.FLOAT)],
params={"poll_interval": 0.01, **params},
)
node.assign_flow("demo", "sensor")
return node
def run_briefly(
node: ConnectorNode,
seconds: float = 0.12,
app: Any = None,
until: Callable[[], bool] | None = None,
) -> None:
"""Start the poll loop, let it tick a few times, stop it.
With ``until``, stop once that holds instead, giving up after 5 s: a fixed
pause is too short when the machine is busy. The caller's asserts then say
what never happened.
"""
async def cycle() -> None:
await node.start(app)
if until is None:
await asyncio.sleep(seconds)
else:
for _ in range(500):
if until():
break
await asyncio.sleep(0.01)
await node.stop()
asyncio.run(cycle())
def test_polling_publishes_what_it_reads():
node = a_sensor([21.5])
pipeline = Pipeline(nodes=[node])
run_briefly(node)
assert pipeline.state["demo.reading"] == 21.5
def test_an_unchanged_reading_is_not_republished():
node = a_sensor([21.5, 21.5, 21.5])
consumer_ran: list[float] = []
def consume(reading, params):
consumer_ran.append(reading)
return None
consumer = Node(
f=consume,
requires=[MessageSpec(name="reading", dtype=DType.FLOAT)],
name="consumer",
)
consumer.assign_flow("demo", "consumer")
Pipeline(nodes=[node, consumer])
run_briefly(node)
# Polled repeatedly, but the value never changed, so downstream ran once.
assert node.polls > 1
assert consumer_ran == [21.5]
def test_a_failing_poll_reports_down_and_keeps_going():
health: list[tuple[str, str | None]] = []
node = a_sensor([RuntimeError("device unplugged"), 21.5])
node._on_health = lambda _node, status, detail: health.append((status, detail))
Pipeline(nodes=[node])
# The first poll fails, so an "ok" after it is the recovery.
run_briefly(node, until=lambda: len(health) > 1 and health[-1][0] == "ok")
assert ("down", "RuntimeError: device unplugged") in health
# It recovered rather than giving up.
assert health[-1][0] == "ok"
def test_an_undeclared_port_keeps_failing_until_the_node_declares_it():
"""A publication that raised is retried, not remembered as published.
The loop remembers what it published. If it remembered what it read, a
value the node cannot publish would be skipped on the next poll, the poll
would succeed, and the node would go back to reporting itself healthy with
its port still dark.
"""
class Chatty(Sensor):
"""Reads a port it never declared."""
async def poll(self) -> dict[str, Any]:
self.polls += 1
return {"reading": 21.5, "lat": 48.1}
health: list[tuple[str, str | None]] = []
node = Chatty(
provides=[MessageSpec(name="reading", dtype=DType.FLOAT)],
params={"poll_interval": 0.01},
)
node.assign_flow("demo", "sensor")
node._on_health = lambda _node, status, detail: health.append((status, detail))
pipeline = Pipeline(nodes=[node])
run_briefly(node)
assert pipeline.state.get("demo.reading") is None
assert health[-1][0] == "down"
assert "NodeOutputError" in (health[-1][1] or "")
# Still failing on the last poll, not just the first.
assert len([entry for entry in health if entry[0] == "down"]) > 1
def test_a_push_session_that_fails_before_it_listens_reports_down():
"""A crashed loop is node health, not only a crash event."""
class Push(Sensor):
async def start(self, app: Any = None) -> None:
self._run_supervised("push", self._session)
async def _session(self) -> None:
raise ConnectionError("bridge unreachable")
health: list[tuple[str, str | None]] = []
node = Push(provides=[MessageSpec(name="reading", dtype=DType.FLOAT)])
node.assign_flow("demo", "push")
node._on_health = lambda _node, status, detail: health.append((status, detail))
async def scenario() -> None:
node.supervisor = Supervisor()
await node.start()
for _ in range(500):
if health:
break
await asyncio.sleep(0.01)
await node.supervisor.cancel_all()
asyncio.run(scenario())
assert health == [("down", "ConnectionError: bridge unreachable")]
class Actuator(ConnectorNode):
"""A connector that commands something instead of reading it."""
contract = CONTRACT_VERSION
title = "Test actuator"
def __init__(self, **kwargs: Any) -> None:
super().__init__(**kwargs)
self.commands: list[dict[str, Any]] = []
def write(self, **ports: Any) -> None:
self.commands.append(ports)
return None
def test_an_incoming_message_reaches_a_connector_that_writes():
node = Actuator(requires=[MessageSpec(name="level", dtype=DType.INT)])
node.assign_flow("demo", "actuator")
assert node.execute({"demo.level": 255}) is None
assert node.commands == [{"level": 255}]
def test_a_read_only_connector_ignores_what_reaches_it():
node = a_sensor([])
assert node.execute({}) is None
def test_a_credential_param_is_marked_for_the_editor():
schema = Sensor.Params.model_json_schema()
assert schema["properties"]["poll_interval"]["default"] == 0
assert "secret_token" in schema["properties"]
def test_a_connector_is_discovered_from_its_entry_point(monkeypatch):
class FakeDist:
name = "fluksio-connector-test"
version = "0.1.0"
class FakeEntry:
name = "test_sensor"
dist = FakeDist()
def load(self):
return Sensor
monkeypatch.setattr(
"fluksio.flow.plugins.entry_points", lambda group: [FakeEntry()]
)
try:
assert load_plugins() == ["test_sensor"]
assert NODE_TYPES["test_sensor"].plugin == "fluksio-connector-test 0.1.0"
assert NODE_TYPES["test_sensor"].title == "Test sensor"
finally:
NODE_TYPES.pop("test_sensor", None)
def test_a_connector_written_for_another_contract_is_refused(monkeypatch):
class Outdated(ConnectorNode):
contract = CONTRACT_VERSION + 1
class FakeEntry:
name = "outdated"
dist = None
def load(self):
return Outdated
monkeypatch.setattr(
"fluksio.flow.plugins.entry_points", lambda group: [FakeEntry()]
)
assert load_plugins() == []
assert "outdated" not in NODE_TYPES
def test_a_connector_may_not_take_over_a_built_in_type(monkeypatch):
class FakeEntry:
name = "mqtt"
dist = None
def load(self): # pragma: no cover - never reached
raise AssertionError("should not be loaded")
monkeypatch.setattr(
"fluksio.flow.plugins.entry_points", lambda group: [FakeEntry()]
)
assert load_plugins() == []
assert NODE_TYPES["mqtt"].plugin is None
def test_a_connector_publishes_bytes_as_a_media_reference(tmp_path):
"""A camera's reading is bytes, and bytes never travel as a message."""
class Camera(ConnectorNode):
contract = CONTRACT_VERSION
async def poll(self) -> dict[str, Any] | None:
return {"frame": self.save_artifact(b"\x89PNG...", "f.png", "image/png")}
node = Camera(
provides=[MessageSpec(name="frame", dtype=DType.IMAGE)],
params={"poll_interval": 0.01},
)
node.assign_flow("demo", "camera")
pipeline = Pipeline(nodes=[node])
# Before it starts there is no store to write to, and saying so beats an
# AttributeError from inside somebody's connector.
with pytest.raises(RuntimeError, match="artifact store"):
node.save_artifact(b"x")
store = ArtifactStore(tmp_path / "artifacts")
app = SimpleNamespace(state=SimpleNamespace(artifact_store=store))
run_briefly(node, app=app)
reference = pipeline.state["demo.frame"]
# It typechecks against the port it was published on, which is the whole
# point of a media dtype.
MessageSpec(name="demo.frame", dtype=DType.IMAGE).check(reference)
assert store.path(reference["digest"]).read_bytes() == b"\x89PNG..."
def test_a_port_is_wanted_while_watched_or_read(monkeypatch):
"""A camera fetches frames only while a screen shows them or a flow reads them."""
from fluksio.flow.demand import demand
node = a_sensor([])
# A worker process binds no controller and cannot see who watches. The
# one an app test bound stays on the process-wide `demand` after its stop.
monkeypatch.setattr(demand, "_controller", None)
assert node.wanted("reading")
reads: set[str] = set()
monkeypatch.setattr(
demand, "_controller", SimpleNamespace(consumed_messages=lambda: reads)
)
assert not node.wanted("reading")
reads.add("demo.reading")
assert node.wanted("reading")
reads.clear()
demand.want("a viewer", {"demo.reading"})
try:
assert node.wanted("reading")
finally:
demand.drop("a viewer")
assert not node.wanted("reading")