From b70f3228ca900a74f0e1b7e47c404e36998643c0 Mon Sep 17 00:00:00 2001 From: didericis Date: Mon, 13 Jul 2026 15:05:55 -0400 Subject: [PATCH] =?UTF-8?q?fix(orchestrator):=20pyright=20=E2=80=94=20pass?= =?UTF-8?q?=20explicit=20LaunchRequest=20in=20the=20docker=20integration?= =?UTF-8?q?=20test=20(#352)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The **kw unpacking put str into slot: int|None. Pass explicit LaunchRequests instead. Co-Authored-By: Claude Opus 4.8 Claude-Session: https://claude.ai/code/session_01WBMWTEtQdJ4W5UrWuLHCck --- tests/integration/test_orchestrator_docker_broker.py | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/integration/test_orchestrator_docker_broker.py b/tests/integration/test_orchestrator_docker_broker.py index d526f79..8c4da42 100644 --- a/tests/integration/test_orchestrator_docker_broker.py +++ b/tests/integration/test_orchestrator_docker_broker.py @@ -40,14 +40,15 @@ class TestDockerBrokerIntegration(unittest.TestCase): ) return self.name in proc.stdout.split() - def _submit(self, op: str, **kw: str) -> None: - req = LaunchRequest(op=op, bottle_id=self.bottle_id, **kw) + def _submit(self, req: LaunchRequest) -> None: self.broker.submit(sign_request(req, self.secret)) def test_launch_creates_then_teardown_removes(self) -> None: - self._submit("launch", image_ref=IMAGE) + self._submit( + LaunchRequest(op="launch", bottle_id=self.bottle_id, image_ref=IMAGE) + ) self.assertTrue(self._exists(), "container should exist after launch") - self._submit("teardown") + self._submit(LaunchRequest(op="teardown", bottle_id=self.bottle_id)) self.assertFalse(self._exists(), "container should be gone after teardown")