From 4b3cc52afe052d5d900379548aee04d813c46d4c Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Sun, 6 Jul 2025 10:02:47 -0500 Subject: [PATCH] preen --- tests/integration/test_scheduler_heap_stress.py | 8 -------- tests/integration/test_scheduler_rapid_cancellation.py | 4 ++-- .../integration/test_scheduler_simultaneous_callbacks.py | 2 +- tests/integration/test_scheduler_string_lifetime.py | 5 +---- 4 files changed, 4 insertions(+), 15 deletions(-) diff --git a/tests/integration/test_scheduler_heap_stress.py b/tests/integration/test_scheduler_heap_stress.py index d5f03462fd..4add431f7c 100644 --- a/tests/integration/test_scheduler_heap_stress.py +++ b/tests/integration/test_scheduler_heap_stress.py @@ -133,14 +133,6 @@ async def test_scheduler_heap_stress( assert len(indices) == 100, ( f"Thread {thread_id} executed {len(indices)} callbacks, expected 100" ) - - # Verify that we executed a reasonable number of callbacks - assert timeout_count > 0, ( - f"Expected some timeout callbacks but got {timeout_count}" - ) - assert interval_count > 0, ( - f"Expected some interval callbacks but got {interval_count}" - ) # Total should be 1000 callbacks total_callbacks = timeout_count + interval_count assert total_callbacks == 1000, ( diff --git a/tests/integration/test_scheduler_rapid_cancellation.py b/tests/integration/test_scheduler_rapid_cancellation.py index 9c5ed4bb6e..f38c5ebb57 100644 --- a/tests/integration/test_scheduler_rapid_cancellation.py +++ b/tests/integration/test_scheduler_rapid_cancellation.py @@ -46,8 +46,8 @@ async def test_scheduler_rapid_cancellation( # Count log lines test_stats["log_count"] += 1 - # Check for errors - if "ERROR" in line or "WARN" in line: + # Check for errors (only ERROR level, not WARN) + if "ERROR" in line: test_stats["errors"].append(line) # Parse summary statistics diff --git a/tests/integration/test_scheduler_simultaneous_callbacks.py b/tests/integration/test_scheduler_simultaneous_callbacks.py index de5ea601d6..60b87c3cfd 100644 --- a/tests/integration/test_scheduler_simultaneous_callbacks.py +++ b/tests/integration/test_scheduler_simultaneous_callbacks.py @@ -46,7 +46,7 @@ async def test_scheduler_simultaneous_callbacks( test_stats["scheduled"] += 1 elif "Callback executed" in line: test_stats["executed"] += 1 - elif "ERROR" in line or "WARN" in line: + elif "ERROR" in line: test_stats["errors"].append(line) # Check for crash indicators diff --git a/tests/integration/test_scheduler_string_lifetime.py b/tests/integration/test_scheduler_string_lifetime.py index 3b79fc8b70..720b75fd40 100644 --- a/tests/integration/test_scheduler_string_lifetime.py +++ b/tests/integration/test_scheduler_string_lifetime.py @@ -121,10 +121,7 @@ async def test_scheduler_string_lifetime( # Check for any errors assert test_stats["tests_failed"] == 0, f"Tests failed: {test_stats['errors']}" - # Verify we had the expected number of passing tests and no failures + # Verify we had the expected number of passing tests assert test_stats["tests_passed"] == 30, ( f"Expected exactly 30 tests to pass, but got {test_stats['tests_passed']}" ) - assert test_stats["tests_failed"] == 0, ( - f"Expected no test failures, but got {test_stats['tests_failed']} failures" - )