Fix test coverage regression by updating import paths and test files (#12)

* Fix test coverage regression by using improved test files and fixing test assertions

* Improve system_tools test coverage and fix import paths in test files

Author: ipv1337

scripts/tools_coverage.sh

@@ -38,7 +38,7 @@ python -m pytest test_dir/improved/test_tree_tool.py test_dir/test_tree_tool_edg
 
 # System tools
 echo "=== Running system_tools.py tests ==="
-python -m pytest test_dir/test_tools_basic.py -v --cov=src.cli_code.tools.system_tools --cov-append
+python -m pytest test_dir/test_system_tools.py test_dir/test_tools_basic.py::TestSystemTools -v --cov=src.cli_code.tools.system_tools --cov-append
 
 # Base tool class
 echo "=== Running base.py tests ==="

test_dir/test_system_tools.py

@@ -1,9 +1,10 @@
 """
-Tests for system tools.
+Tests for system_tools module to improve code coverage.
 """
-import subprocess
+import os
 import pytest
 from unittest.mock import patch, MagicMock
+import subprocess
 
 # Direct import for coverage tracking
 import src.cli_code.tools.system_tools
@@ -14,7 +15,7 @@ def test_bash_tool_init():
     """Test BashTool initialization."""
     tool = BashTool()
     assert tool.name == "bash"
-    assert tool.description == "Execute a bash command"
+    assert "Execute a bash command" in tool.description
     assert isinstance(tool.BANNED_COMMANDS, list)
     assert len(tool.BANNED_COMMANDS) > 0
 
@@ -79,7 +80,7 @@ def test_bash_tool_timeout(mock_popen):
 
     # Execute command with short timeout
     tool = BashTool()
-    result = tool.execute("sleep 10", timeout=1000)  # 1 second timeout
+    result = tool.execute("sleep 10", timeout=1)  # 1 second timeout
 
     # Verify timeout handling
     assert "Command timed out" in result

test_dir/test_tools_basic.py

@@ -11,13 +11,13 @@
 
 # Import necessary modules safely
 try:
-    from cli_code.tools.base import BaseTool
-    from cli_code.tools.file_tools import ViewTool, EditTool, GrepTool, GlobTool
-    from cli_code.tools.quality_tools import _run_quality_command, LinterCheckerTool, FormatterTool
-    from cli_code.tools.summarizer_tool import SummarizeCodeTool
-    from cli_code.tools.system_tools import BashTool
-    from cli_code.tools.task_complete_tool import TaskCompleteTool
-    from cli_code.tools.tree_tool import TreeTool
+    from src.cli_code.tools.base import BaseTool
+    from src.cli_code.tools.file_tools import ViewTool, EditTool, GrepTool, GlobTool
+    from src.cli_code.tools.quality_tools import _run_quality_command, LinterCheckerTool, FormatterTool
+    from src.cli_code.tools.summarizer_tool import SummarizeCodeTool
+    from src.cli_code.tools.system_tools import BashTool
+    from src.cli_code.tools.task_complete_tool import TaskCompleteTool
+    from src.cli_code.tools.tree_tool import TreeTool
     IMPORTS_AVAILABLE = True
 except ImportError:
     IMPORTS_AVAILABLE = False