Skip to content

Commit 8a6bd07

Browse files
Merge pull request #960 from microsoft/code-quality
fix: code quality issues
2 parents 955faeb + 5047166 commit 8a6bd07

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

src/App/frontend_server.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,7 @@ async def forward_to_client():
130130
try:
131131
await websocket.close()
132132
except Exception:
133+
# Ignore errors during cleanup - websocket may already be closed
133134
pass
134135

135136

src/tests/backend/v4/config/test_settings.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -344,9 +344,9 @@ async def cancel_task():
344344
cancel_task_handle = asyncio.create_task(cancel_task())
345345

346346
with self.assertRaises(asyncio.CancelledError):
347-
_ = await task
347+
await task
348348

349-
await cancel_task_handle
349+
_ = await cancel_task_handle
350350
self.assertTrue(task.cancelled())
351351

352352
async def test_wait_for_clarification_cancelled(self):
@@ -365,9 +365,9 @@ async def cancel_task():
365365
cancel_task_handle = asyncio.create_task(cancel_task())
366366

367367
with self.assertRaises(asyncio.CancelledError):
368-
_ = await task
368+
await task
369369

370-
await cancel_task_handle
370+
_ = await cancel_task_handle
371371
self.assertTrue(task.cancelled())
372372

373373
def test_cleanup_approval(self):

0 commit comments

Comments
 (0)