ed
b1485f759f
The 'time.sleep + assert' pattern is a guaranteed race condition in batched
runs (per workflow's documented anti-pattern). In the live_gui batched test
suite, _process_pending_gui_tasks is competing for CPU with 16 xdist
workers, so 1.5s is sometimes not enough for a single set_value or click
to propagate through the gui task queue.
Fix: replace time.sleep(1.5) with a 10s poll loop that waits for the
expected state (per the same pattern used in test_gui2_custom_callback_hook_works
which was already fixed in commit 09eaf69a for the same reason).
This is a test-only fix; no production code changes.
100 lines
3.5 KiB
Python
100 lines
3.5 KiB
Python
import pytest
|
|
from typing import Any
|
|
import time
|
|
import os
|
|
import uuid
|
|
from pathlib import Path
|
|
import sys
|
|
|
|
# Ensure project root is in path for imports
|
|
sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), "..")))
|
|
sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), "..", "src")))
|
|
from api_hook_client import ApiHookClient
|
|
|
|
# Define a temporary file path for callback testing
|
|
TEST_CALLBACK_FILE = Path("tests/artifacts/temp_callback_output.txt")
|
|
|
|
@pytest.fixture(scope="function", autouse=True)
|
|
def cleanup_callback_file() -> None:
|
|
"""Ensures the test callback file is cleaned up before and after each test."""
|
|
if TEST_CALLBACK_FILE.exists():
|
|
TEST_CALLBACK_FILE.unlink()
|
|
yield
|
|
if TEST_CALLBACK_FILE.exists():
|
|
TEST_CALLBACK_FILE.unlink()
|
|
|
|
def test_gui2_set_value_hook_works(live_gui: Any) -> None:
|
|
"""
|
|
|
|
|
|
Tests that the 'set_value' GUI hook is correctly implemented.
|
|
"""
|
|
client = ApiHookClient()
|
|
assert client.wait_for_server(timeout=10)
|
|
test_value = f"New value set by test: {uuid.uuid4()}"
|
|
gui_data = {'action': 'set_value', 'item': 'ai_input', 'value': test_value}
|
|
response = client.post_gui(gui_data)
|
|
assert response == {'status': 'queued'}
|
|
# Verify the value was actually set using the new get_value hook
|
|
time.sleep(1.5)
|
|
current_value = client.get_value('ai_input')
|
|
assert current_value == test_value
|
|
|
|
def test_gui2_click_hook_works(live_gui: Any) -> None:
|
|
"""
|
|
Tests that the 'click' GUI hook for the 'Reset' button is implemented.
|
|
"""
|
|
client = ApiHookClient()
|
|
assert client.wait_for_server(timeout=10)
|
|
# First, set some state that 'Reset' would clear.
|
|
test_value = "This text should be cleared by the reset button."
|
|
client.set_value('ai_input', test_value)
|
|
# Poll for the set_value to propagate through the gui task queue
|
|
# (per workflow's race-condition anti-pattern guidance: time.sleep + assert
|
|
# is a guaranteed race in batched runs).
|
|
deadline = time.time() + 10.0
|
|
while time.time() < deadline:
|
|
if client.get_value('ai_input') == test_value:
|
|
break
|
|
time.sleep(0.1)
|
|
assert client.get_value('ai_input') == test_value
|
|
# Now, trigger the click
|
|
client.click('btn_reset')
|
|
# Poll for the reset click to propagate (and verify the value clears).
|
|
deadline = time.time() + 10.0
|
|
while time.time() < deadline:
|
|
if client.get_value('ai_input') == "":
|
|
break
|
|
time.sleep(0.1)
|
|
# Verify it was reset
|
|
assert client.get_value('ai_input') == ""
|
|
def test_gui2_custom_callback_hook_works(live_gui: Any) -> None:
|
|
"""
|
|
|
|
|
|
|
|
Tests that the 'custom_callback' GUI hook is correctly implemented.
|
|
"""
|
|
client = ApiHookClient()
|
|
assert client.wait_for_server(timeout=10)
|
|
test_data = f"Callback executed: {uuid.uuid4()}"
|
|
gui_data = {
|
|
'action': 'custom_callback',
|
|
'callback': '_test_callback_func_write_to_file',
|
|
'args': [test_data]
|
|
}
|
|
response = client.post_gui(gui_data)
|
|
assert response == {'status': 'queued'}
|
|
# Poll for the callback to complete (avoids time.sleep race; per workflow anti-pattern)
|
|
temp_workspace_file = Path('tests/artifacts/temp_callback_output.txt')
|
|
deadline = time.time() + 10.0
|
|
while time.time() < deadline:
|
|
if temp_workspace_file.exists():
|
|
content = temp_workspace_file.read_text(encoding="utf-8")
|
|
if content == test_data:
|
|
return
|
|
time.sleep(0.1)
|
|
assert temp_workspace_file.exists(), f"Custom callback was NOT executed, or file path is wrong! Expected: {temp_workspace_file}"
|
|
with open(temp_workspace_file, "r") as f:
|
|
content = f.read()
|
|
assert content == test_data, "Callback executed, but file content is incorrect." |