Address auto-generated code-quality review comments on PR #145

1. tests/durabletask/test_orchestration_e2e.py (test_suspend_and_resume):
   Drop the unused `state =` assignment around the expected-timeout
   `wait_for_orchestration_completion` call. The value is never read
   because the next line asserts False and the only non-failing path
   raises TimeoutError; `state` is reassigned a few lines down. Silences
   the "variable defined multiple times" warning that CodeQL flagged
   because this previously-untouched line was pulled into the diff by
   the indent change.

2. tests/durabletask/test_client.py
   (test_sync_client_context_manager_propagates_exception_and_calls_close):
   Replace the nested `with pytest.raises(...): with client: raise ...`
   pattern with an explicit try/except so CodeQL no longer reports the
   post-block assertions as unreachable. Test intent (exception
   propagation + cleanup verification) is preserved.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: Bernd Verst

tests/durabletask/test_client.py

@@ -791,9 +791,14 @@ def test_sync_client_context_manager_propagates_exception_and_calls_close():
         patch("durabletask.client.stubs.TaskHubSidecarServiceStub", return_value=MagicMock()),
     ):
         client = TaskHubGrpcClient(host_address=HOST_ADDRESS)
-        with pytest.raises(RuntimeError, match="boom"):
+        raised = False
+        try:
             with client:
                 raise RuntimeError("boom")
+        except RuntimeError as exc:
+            raised = True
+            assert str(exc) == "boom"
+        assert raised, "RuntimeError raised inside the with block must propagate"
 
     assert client._closing is True
     channel.close.assert_called_once_with()

tests/durabletask/test_orchestration_e2e.py

@@ -299,7 +299,7 @@ def orchestrator(ctx: task.OrchestrationContext, _):
             # Raise an event to the orchestration and confirm that it does NOT complete
             task_hub_client.raise_orchestration_event(id, "my_event", data=42)
             try:
-                state = task_hub_client.wait_for_orchestration_completion(id, timeout=3)
+                task_hub_client.wait_for_orchestration_completion(id, timeout=3)
                 assert False, "Orchestration should not have completed"
             except TimeoutError:
                 pass