fix: remove redundant asyncio.sleep calls in E2E helper tests (#470)

* fix: remove redundant asyncio.sleep calls in E2E helper tests

Remove three instances of `await asyncio.sleep(5)` that were immediately
followed by `wait_for_entity_registration()` calls. These sleeps were
completely redundant because:

1. wait_for_entity_registration() already polls with a 20-second timeout
2. The fixed 5-second sleep wastes time even when entities are ready
3. The sleeps don't improve reliability - polling handles timing naturally

This change saves ~15 seconds per test run and addresses part of issue #366
regarding flaky E2E tests and technical debt from fixed sleep patterns.

Also removes unused asyncio import and fixes a pre-existing f-string lint
warning (string without placeholders).

Relates to #366

* refactor: remove redundant comments per code review

Remove explanatory comments that duplicate what the self-documenting
function name `wait_for_entity_registration` already communicates.

Addresses Gemini Code Assist review feedback on PR #470.

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: kingpanther13

tests/src/e2e/workflows/config/test_helper_crud.py

@@ -7,7 +7,6 @@
 - Type-specific parameter validation
 """
 
-import asyncio
 import logging
 
 import pytest
@@ -242,19 +241,7 @@ async def test_input_number_full_lifecycle(self, mcp_client, cleanup_tracker):
         cleanup_tracker.track("input_number", entity_id)
         logger.info(f"Created input_number: {entity_id}")
 
-        # Give HA a moment to process entity registration before polling
-
-
-        await asyncio.sleep(5)
-
-
-
-        # Wait for entity to be registered (existence only, not specific state)
-
-
         entity_ready = await wait_for_entity_registration(mcp_client, entity_id)
-
-
         assert entity_ready, f"Entity {entity_id} not registered within timeout"
 
         # VERIFY via state
@@ -347,19 +334,7 @@ async def test_input_select_full_lifecycle(self, mcp_client, cleanup_tracker):
         cleanup_tracker.track("input_select", entity_id)
         logger.info(f"Created input_select: {entity_id}")
 
-        # Give HA a moment to process entity registration before polling
-
-
-        await asyncio.sleep(5)
-
-
-
-        # Wait for entity to be registered (existence only, not specific state)
-
-
         entity_ready = await wait_for_entity_registration(mcp_client, entity_id)
-
-
         assert entity_ready, f"Entity {entity_id} not registered within timeout"
 
         # VERIFY via state
@@ -445,19 +420,7 @@ async def test_input_text_full_lifecycle(self, mcp_client, cleanup_tracker):
         cleanup_tracker.track("input_text", entity_id)
         logger.info(f"Created input_text: {entity_id}")
 
-        # Give HA a moment to process entity registration before polling
-
-
-        await asyncio.sleep(5)
-
-
-
-        # Wait for entity to be registered (existence only, not specific state)
-
-
         entity_ready = await wait_for_entity_registration(mcp_client, entity_id)
-
-
         assert entity_ready, f"Entity {entity_id} not registered within timeout"
 
         # DELETE
@@ -853,7 +816,7 @@ async def test_timer_full_lifecycle(self, mcp_client, cleanup_tracker):
             mcp_client, entity_id, "idle", timeout=10
         )
         assert state_reached, f"Timer {entity_id} not registered in idle state within timeout"
-        logger.info(f"Timer initial state: idle")
+        logger.info("Timer initial state: idle")
 
         # START timer
         start_result = await mcp_client.call_tool(