diff options
| author | Christophe Besson <cbesson@gmail.com> | 2026-09-24 12:32:56 +0200 |
|---|---|---|
| committer | Christophe Besson <cbesson@gmail.com> | 2026-09-24 16:45:38 +0200 |
| commit | db400a438853036cfd3518873ea47beabe304718 (patch) | |
| tree | ac46807429fb67cdafa95c9f60768ff2e770f094 /packages/meshbay-node/tests/test_dispatch_golden.py | |
| parent | 76c67f4ad43ddb30ac25868554638c1e0aac83bd (diff) | |
| download | meshbay-db400a438853036cfd3518873ea47beabe304718.tar.gz | |
test(node): the dispatch golden covers a type field that is not a string
Missing, None, an int, a list, a dict, bytes: recorded on the elif chain
before it becomes a table, since a dict lookup would raise on the
unhashable ones where the chain found no match.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Diffstat (limited to 'packages/meshbay-node/tests/test_dispatch_golden.py')
| -rw-r--r-- | packages/meshbay-node/tests/test_dispatch_golden.py | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/packages/meshbay-node/tests/test_dispatch_golden.py b/packages/meshbay-node/tests/test_dispatch_golden.py index b31b63b..c917208 100644 --- a/packages/meshbay-node/tests/test_dispatch_golden.py +++ b/packages/meshbay-node/tests/test_dispatch_golden.py @@ -144,6 +144,20 @@ def _message_types() -> list[str]: return sorted(names | {"client_diag", "no_such_type"}) +def _odd_types() -> dict[str, dict]: + """A `type` field that is not a string, or no `type` at all. msgpack will + carry any value there, and a dispatch keyed by type has to treat each of + these as the chain of comparisons did — an unhashable one included.""" + return { + "<missing>": {}, + "<none>": {"type": None}, + "<int>": {"type": 7}, + "<list>": {"type": [MNP.PING]}, + "<dict>": {"type": {"t": MNP.PING}}, + "<bytes>": {"type": MNP.PING.encode()}, + } + + def _shapes(keys: list[str]) -> dict[str, dict]: return { "bare": {}, @@ -165,6 +179,10 @@ def _record(keys: list[str]) -> dict: for shape, fields in _shapes(keys).items(): msg = {**fields, "type": mtype, "req_id": REQ_ID} out["dispatch"][f"{mtype} | {state} | {shape}"] = _run(_session(state), msg) + for label, fields in _odd_types().items(): + for state in ("fresh", "challenged", "member", "operator"): + msg = {**fields, "req_id": REQ_ID} + out["dispatch"][f"{label} | {state} | bare"] = _run(_session(state), msg) for op in _admin_ops() + ["no_such_op"]: s = _session("operator") s._admin_ops["the-op"] = {"op": op, "subject": "subject", "nonce": b"\x05" * 32, |