Removed the redundant sentinel_task and mock_create_task.return_value assignment

kevinlu310 · kevinlu310 · commit a22e2c73fc52 · 2026-03-05T18:07:29.000-08:00
diff --git a/tests/server/tasks/test_result_aggregator.py b/tests/server/tasks/test_result_aggregator.py
@@ -262,8 +262,6 @@ async def mock_consume_generator():
 
         # Mock _continue_consuming to check if it's called by create_task
         self.aggregator._continue_consuming = AsyncMock()
-        sentinel_task = asyncio.ensure_future(asyncio.sleep(0))
-        mock_create_task.return_value = sentinel_task
         mock_create_task.side_effect = lambda coro: asyncio.ensure_future(coro)
 
         (

Original file line number	Diff line number	Diff line change
`@@ -262,8 +262,6 @@ async def mock_consume_generator():`
`262`	`262`
`263`	`263`	`# Mock _continue_consuming to check if it's called by create_task`
`264`	`264`	`self.aggregator._continue_consuming = AsyncMock()`
`265`		`- sentinel_task = asyncio.ensure_future(asyncio.sleep(0))`
`266`		`- mock_create_task.return_value = sentinel_task`
`267`	`265`	`mock_create_task.side_effect = lambda coro: asyncio.ensure_future(coro)`
`268`	`266`
`269`	`267`	`(`