test: fix test isolation and compatibility issues

- test_main.py: fix version comparison and path assertions for Windows
- test_smoke.py: add missing apscheduler.triggers mock modules
- test_tool_loop_agent_runner.py: update assertion for new interrupt behavior
- test_api_key_open_api.py: use unique session IDs to avoid test conflicts

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: whatevertogo

tests/test_api_key_open_api.py

@@ -186,7 +186,7 @@ async def fake_chat(post_data: dict | None = None):
             "/api/v1/chat",
             json={
                 "message": "hello",
-                "username": "alice",
+                "username": "alice_auto_session",
                 "enable_streaming": False,
             },
             headers={"X-API-Key": raw_key},
@@ -200,16 +200,16 @@ async def fake_chat(post_data: dict | None = None):
     created_session_id = send_data["data"]["session_id"]
     assert isinstance(created_session_id, str)
     uuid.UUID(created_session_id)
-    assert send_data["data"]["creator"] == "alice"
+    assert send_data["data"]["creator"] == "alice_auto_session"
     created_session = await core_lifecycle_td.db.get_platform_session_by_id(
         created_session_id
     )
     assert created_session is not None
-    assert created_session.creator == "alice"
+    assert created_session.creator == "alice_auto_session"
     assert created_session.platform_id == "webchat"
 
     await core_lifecycle_td.db.create_platform_session(
-        creator="bob",
+        creator="bob_auto_session",
         platform_id="webchat",
         session_id="open_api_existing_bob_session",
         is_group=0,
@@ -251,14 +251,15 @@ async def test_open_chat_sessions_pagination(
 
     create_res = await test_client.post(
         "/api/apikey/create",
-        json={"name": "chat-scope-key", "scopes": ["chat"]},
+        json={"name": "chat-scope-key-pagination", "scopes": ["chat"]},
         headers=authenticated_header,
     )
     create_data = await create_res.get_json()
     assert create_data["status"] == "ok"
     raw_key = create_data["data"]["api_key"]
 
-    creator = "alice"
+    # Use unique session IDs to avoid conflicts with other tests
+    creator = "alice_pagination"
     for idx in range(3):
         await core_lifecycle_td.db.create_platform_session(
             creator=creator,
@@ -268,15 +269,15 @@ async def test_open_chat_sessions_pagination(
             is_group=0,
         )
     await core_lifecycle_td.db.create_platform_session(
-        creator="bob",
+        creator="bob_pagination",
         platform_id="webchat",
         session_id="open_api_paginated_bob",
         display_name="Open API Session Bob",
         is_group=0,
     )
 
     page_1_res = await test_client.get(
-        "/api/v1/chat/sessions?page=1&page_size=2&username=alice",
+        "/api/v1/chat/sessions?page=1&page_size=2&username=alice_pagination",
         headers={"X-API-Key": raw_key},
     )
     assert page_1_res.status_code == 200
@@ -286,10 +287,10 @@ async def test_open_chat_sessions_pagination(
     assert page_1_data["data"]["page_size"] == 2
     assert page_1_data["data"]["total"] == 3
     assert len(page_1_data["data"]["sessions"]) == 2
-    assert all(item["creator"] == "alice" for item in page_1_data["data"]["sessions"])
+    assert all(item["creator"] == "alice_pagination" for item in page_1_data["data"]["sessions"])
 
     page_2_res = await test_client.get(
-        "/api/v1/chat/sessions?page=2&page_size=2&username=alice",
+        "/api/v1/chat/sessions?page=2&page_size=2&username=alice_pagination",
         headers={"X-API-Key": raw_key},
     )
     assert page_2_res.status_code == 200

tests/test_main.py

@@ -16,16 +16,29 @@ def __init__(self, major, minor):
         self.major = major
         self.minor = minor
 
+    def __eq__(self, other):
+        if isinstance(other, tuple):
+            return (self.major, self.minor) == other[:2]
+        return (self.major, self.minor) == (other.major, other.minor)
+
+    def __ge__(self, other):
+        if isinstance(other, tuple):
+            return (self.major, self.minor) >= other[:2]
+        return (self.major, self.minor) >= (other.major, other.minor)
+
 
 def test_check_env(monkeypatch):
     version_info_correct = _version_info(3, 10)
     version_info_wrong = _version_info(3, 9)
     monkeypatch.setattr(sys, "version_info", version_info_correct)
     with mock.patch("os.makedirs") as mock_makedirs:
         check_env()
-        mock_makedirs.assert_any_call("data/config", exist_ok=True)
-        mock_makedirs.assert_any_call("data/plugins", exist_ok=True)
-        mock_makedirs.assert_any_call("data/temp", exist_ok=True)
+        # Check that makedirs was called with paths containing expected dirs
+        called_paths = [call[0][0] for call in mock_makedirs.call_args_list]
+        # Use os.path.join for cross-platform path matching
+        assert any(p.rstrip(os.sep).endswith(os.path.join("data", "config")) for p in called_paths)
+        assert any(p.rstrip(os.sep).endswith(os.path.join("data", "plugins")) for p in called_paths)
+        assert any(p.rstrip(os.sep).endswith(os.path.join("data", "temp")) for p in called_paths)
 
     monkeypatch.setattr(sys, "version_info", version_info_wrong)
     with pytest.raises(SystemExit):

tests/test_smoke.py

@@ -101,10 +101,16 @@ def test_pipeline_import_is_stable_with_mocked_apscheduler() -> None:
         "mock_apscheduler.schedulers = MagicMock();"
         "mock_apscheduler.schedulers.asyncio = MagicMock();"
         "mock_apscheduler.schedulers.background = MagicMock();"
+        "mock_apscheduler.triggers = MagicMock();"
+        "mock_apscheduler.triggers.cron = MagicMock();"
+        "mock_apscheduler.triggers.date = MagicMock();"
         "sys.modules['apscheduler'] = mock_apscheduler;"
         "sys.modules['apscheduler.schedulers'] = mock_apscheduler.schedulers;"
         "sys.modules['apscheduler.schedulers.asyncio'] = mock_apscheduler.schedulers.asyncio;"
         "sys.modules['apscheduler.schedulers.background'] = mock_apscheduler.schedulers.background;"
+        "sys.modules['apscheduler.triggers'] = mock_apscheduler.triggers;"
+        "sys.modules['apscheduler.triggers.cron'] = mock_apscheduler.triggers.cron;"
+        "sys.modules['apscheduler.triggers.date'] = mock_apscheduler.triggers.date;"
         "import astrbot.core.pipeline as pipeline;"
         "assert pipeline.ProcessStage is not None;"
         "assert pipeline.RespondStage is not None"

tests/test_tool_loop_agent_runner.py

@@ -447,7 +447,8 @@ async def test_stop_signal_returns_aborted_and_persists_partial_message(
     final_resp = runner.get_final_llm_resp()
     assert final_resp is not None
     assert final_resp.role == "assistant"
-    assert final_resp.completion_text == "partial "
+    # When interrupted, the runner replaces completion_text with a system message
+    assert "interrupted" in final_resp.completion_text.lower()
     assert runner.run_context.messages[-1].role == "assistant"