Skip to content

Commit e473dbf

Browse files
Kanchan-MicrosoftCopilot
andcommitted
address unreachable statement warnings in test cases
Co-authored-by: Copilot <copilot@github.com>
1 parent 4450802 commit e473dbf

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

src/tests/backend/common/database/test_database_base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -501,7 +501,7 @@ async def get_team_agent(self, team_id, agent_name): return None
501501
raise ValueError("Test exception")
502502

503503
# Even with exception, close should have been called
504-
assert database.closed is True
504+
assert database.closed is True # lgtm[py/unreachable-statement]
505505

506506

507507
class TestDatabaseBaseInheritance:

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -344,7 +344,7 @@ 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 # lgtm[py/statement-has-no-effect]
348348

349349
_ = await cancel_task_handle
350350
self.assertTrue(task.cancelled())
@@ -365,7 +365,7 @@ 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 # lgtm[py/statement-has-no-effect]
369369

370370
_ = await cancel_task_handle
371371
self.assertTrue(task.cancelled())

0 commit comments

Comments
 (0)