From 3eaea44e61b4e7203f981a4bf1b1b946cf6a6eb7 Mon Sep 17 00:00:00 2001 From: ChengLong GU <88359731+gcl-coder@users.noreply.github.com> Date: Thu, 5 Nov 2026 10:27:32 +0800 Subject: [PATCH] fix(tests): spell the steward evidence sender as a def `tests/test_manager_context_roundtrip.py` assigned a lambda to `sender`, which the `ruff check tests` step in `kernel-static-checks` rejects with E731. The rule is in ruff's default E7 set, so the lane went red for every pull request from that commit onward: `checks` depends on `kernel-static-checks`, and `pytest`, `stage2c-correctness-e2e` and `merge-gate` all depend on `checks`. The queued-sender callback only returns one fixed receipt, so a nested def says the same thing without the lint exemption. `ruff check --select E4,E7,E9,F` on the file now passes. Signed-off-by: ChengLong GU <88359731+gcl-coder@users.noreply.github.com> --- tests/test_manager_context_roundtrip.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/test_manager_context_roundtrip.py b/tests/test_manager_context_roundtrip.py index d2895c4bda..181d8654c6 100644 --- a/tests/test_manager_context_roundtrip.py +++ b/tests/test_manager_context_roundtrip.py @@ -286,7 +286,9 @@ def read(): return page["rows"][0] remaining = {"summary": "Measure the original revision sequence", "resume_when": "The original evidence is available"} - sender = lambda *_: {"ok": True, "reply_verified": True} + def sender(*_): + return {"ok": True, "reply_verified": True} + report(root, "research", "worker", rid, "conclusion", "An initial analysis is available.", remaining_work=remaining) assert read()["remaining_work"] == remaining assert drain(root, registry, store, sender) == 1