Skip to content

Commit cc8865d

Browse files
Resolved co-pilot comments
1 parent 43265c3 commit cc8865d

3 files changed

Lines changed: 10 additions & 8 deletions

File tree

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -499,9 +499,9 @@ async def get_team_agent(self, team_id, agent_name): return None
499499
assert database.initialized is True
500500
# Raise an exception to test cleanup
501501
raise ValueError("Test exception")
502-
503-
# Even with exception, close should have been called
504-
assert database.closed is True
502+
503+
# Even with exception, close should have been called
504+
assert database.closed is True
505505

506506

507507
class TestDatabaseBaseInheritance:

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ def test_ad_token_provider(self, mock_config):
7676
mock_credential.get_token.assert_called_once_with(mock_config.AZURE_COGNITIVE_SERVICES)
7777

7878

79-
8079
class TestAzureConfigAsync(unittest.IsolatedAsyncioTestCase):
8180
"""Async test cases for AzureConfig class."""
8281

@@ -350,7 +349,7 @@ async def cancel_task():
350349
with self.assertRaises(asyncio.CancelledError):
351350
await task
352351

353-
_ = await cancel_task_handle
352+
await cancel_task_handle
354353

355354
async def test_wait_for_clarification_cancelled(self):
356355
"""Test waiting for clarification when cancelled."""
@@ -370,7 +369,7 @@ async def cancel_task():
370369
with self.assertRaises(asyncio.CancelledError):
371370
await task
372371

373-
_ = await cancel_task_handle
372+
await cancel_task_handle
374373

375374
def test_cleanup_approval(self):
376375
"""Test cleanup approval."""

src/tests/backend/v4/magentic_agents/test_proxy_agent.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,11 @@
5858
# Now import the module under test
5959
import backend.v4.magentic_agents.proxy_agent as proxy_agent_module
6060

61-
# Trivial usage to ensure the import is considered used (and to assert it loaded)
62-
assert proxy_agent_module is not None
61+
62+
def test_module_imports():
63+
"""Ensure the proxy_agent module imports correctly and is referenced in tests."""
64+
assert proxy_agent_module is not None
65+
6366

6467
class TestProxyAgentComplexScenarios:
6568
"""Additional test scenarios to improve coverage."""

0 commit comments

Comments
 (0)