camcheck: no nested parentheses in the missing cameras' message

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit 917ef9d29c)
This commit is contained in:
DeeJanuz committed 2026-10-05 13:55:17 -06:00
1 parent 98bd6ce21c
commit 662919babf
2 files changed
+2 -2

No files matched your search

+1 -1
View File
@@ -412,7 +412,7 @@ def check(log=None, proc=True, ring=True, ring_path=None):
have = {c["node"] for c in r["mono"]}
want = state.tracking_nodes() if state else SIDE_NODES + UPPER_NODES
out["ring_missing"] = [n for n in want if n not in have]
status, reason = "degraded", ("ft-camd publishes only %d of %d mono cameras (missing: %s)" % (
status, reason = "degraded", ("ft-camd publishes only %d of %d mono cameras, missing %s" % (
len(r["mono"]), TRACKING, " ".join("video%d" % n for n in out["ring_missing"]) or "?"))
out.update(status=status, reason=reason, evidence=evidence,
summary="ok" if status == "ok" else "%s: %s" % (status, reason))
+1 -1
View File
@@ -293,7 +293,7 @@ class Ring(unittest.TestCase):
r = camcheck.check(log=self.log, proc=False, ring_path=self.ring)
self.assertEqual(r["status"], "degraded")
self.assertEqual(r["ring_missing"], [0, 3])
self.assertIn("missing: video0 video3", r["reason"])
self.assertIn("missing video0 video3", r["reason"])
self.assertTrue(camcheck.is_ring_short(r))
self.assertFalse(camcheck.is_vcint_failure(r))
self.assertEqual(r["map"]["slam_left"]["node"], 0)