From 59b53f0a2313fbafef3c189f4f6911bd3dbe32db Mon Sep 17 00:00:00 2001 From: dmahan93 Date: Mon, 9 Mar 2026 23:37:32 -0500 Subject: [PATCH] fix: skip tests when atroposlib/minisweagent unavailable in CI - test_agent_loop_tool_calling.py: import atroposlib at module level to trigger skip (environments.agent_loop is now importable without atroposlib due to __init__.py graceful fallback) - test_modal_sandbox_fixes.py: skip TestToolResolution tests when minisweagent not installed --- tests/test_agent_loop_tool_calling.py | 1 + tests/tools/test_modal_sandbox_fixes.py | 11 +++++++++++ 2 files changed, 12 insertions(+) diff --git a/tests/test_agent_loop_tool_calling.py b/tests/test_agent_loop_tool_calling.py index b07fdefeb..857be5fa0 100644 --- a/tests/test_agent_loop_tool_calling.py +++ b/tests/test_agent_loop_tool_calling.py @@ -35,6 +35,7 @@ if str(_repo_root) not in sys.path: try: from environments.agent_loop import AgentResult, HermesAgentLoop + from atroposlib.envs.server_handling.openai_server import OpenAIServer # noqa: F401 except ImportError: pytest.skip("atroposlib not installed", allow_module_level=True) diff --git a/tests/tools/test_modal_sandbox_fixes.py b/tests/tools/test_modal_sandbox_fixes.py index b2d7fe022..6da25216b 100644 --- a/tests/tools/test_modal_sandbox_fixes.py +++ b/tests/tools/test_modal_sandbox_fixes.py @@ -36,8 +36,17 @@ except ImportError: class TestToolResolution: """Verify get_tool_definitions returns all expected tools for eval.""" + def _has_minisweagent(self): + try: + import minisweagent # noqa: F401 + return True + except ImportError: + return False + def test_terminal_and_file_toolsets_resolve_all_tools(self): """enabled_toolsets=['terminal', 'file'] should produce 6 tools.""" + if not self._has_minisweagent(): + pytest.skip("minisweagent not installed (git submodule update --init)") from model_tools import get_tool_definitions tools = get_tool_definitions( enabled_toolsets=["terminal", "file"], @@ -49,6 +58,8 @@ class TestToolResolution: def test_terminal_tool_present(self): """The terminal tool must be present (not silently dropped).""" + if not self._has_minisweagent(): + pytest.skip("minisweagent not installed (git submodule update --init)") from model_tools import get_tool_definitions tools = get_tool_definitions( enabled_toolsets=["terminal", "file"],