feat: pytest settings (#4)

* Add: pytest settings

* Update pyproject.toml

Author: paikend

pyproject.toml

@@ -10,9 +10,22 @@ dependencies = [
     "pydantic>=2.11.7",
 ]
 
+[project.dependency-group]
+test = [
+    "pytest>=8.4.1",
+]
+
+
 [build-system]
 requires = ["hatchling"]
 build-backend = "hatchling.build"
 
 [tool.hatch.build.targets.wheel]
-packages = ["src/openstack_mcp_server"]
+packages = ["src/openstack_mcp_server"]
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]
+
+python_files = ["test_*.py", "*_test.py"]
+python_classes = ["Test*"]
+python_functions = ["test_*"]

tests/__init__.py

@@ -0,0 +1,51 @@
+import pytest
+from unittest.mock import Mock, patch
+
+
+@pytest.fixture
+def mock_openstack_logging():
+    """Mock openstack.enable_logging function."""
+    with patch('openstack_mcp_server.tools.nova_tools.openstack.enable_logging') as mock:
+        yield mock
+
+
+@pytest.fixture
+def mock_openstack_connect():
+    """Mock openstack.connect function."""
+    with patch('openstack_mcp_server.tools.nova_tools.openstack.connect') as mock:
+        yield mock
+
+
+@pytest.fixture
+def mock_openstack_connection():
+    """Mock openstack connection object."""
+    mock_conn = Mock()
+    with patch('openstack_mcp_server.tools.nova_tools.openstack.connect', return_value=mock_conn) as mock_connect:
+        yield mock_conn, mock_connect
+
+
+@pytest.fixture
+def mock_server():
+    """Create a mock server object."""
+    def _create_server(name="test-server", server_id="test-id", status="ACTIVE"):
+        mock = Mock()
+        mock.name = name
+        mock.id = server_id
+        mock.status = status
+        return mock
+    return _create_server
+
+
+@pytest.fixture
+def sample_servers(mock_server):
+    """Create sample server list."""
+    return [
+        mock_server("server1", "id1", "ACTIVE"),
+        mock_server("server2", "id2", "STOPPED")
+    ]
+
+
+@pytest.fixture
+def mock_fastmcp():
+    """Mock FastMCP instance."""
+    return Mock()

tests/conftest.py

@@ -0,0 +1,60 @@
+import pytest
+from openstack_mcp_server.tools.nova_tools import NovaTools
+
+
+class TestNovaTools:
+    def test_register_tools(self, mock_fastmcp):
+        """Test that tools are properly registered with FastMCP."""
+        # Arrange
+        nova_tools = NovaTools()
+
+        # Act
+        nova_tools.register_tools(mock_fastmcp)
+
+        # Assert
+        mock_fastmcp.tool.assert_called_once()
+        mock_fastmcp.tool().assert_called_once_with(nova_tools.get_nova_servers)
+
+    def test_get_nova_servers(self, mock_openstack_logging, mock_openstack_connect, sample_servers):
+        """Test getting Nova servers list."""
+        # Arrange
+        mock_conn = mock_openstack_connect.return_value
+        mock_conn.list_servers.return_value = sample_servers
+        nova_tools = NovaTools()
+
+        # Act
+        result = nova_tools.get_nova_servers()
+
+        # Assert
+        expected = "server1 (id1) - Status: ACTIVE\nserver2 (id2) - Status: STOPPED"
+        assert result == expected
+        mock_openstack_logging.assert_called_once_with(debug=True)
+        mock_openstack_connect.assert_called_once_with(cloud='openstack')
+
+    def test_get_nova_servers_empty_list(self, mock_openstack_logging, mock_openstack_connect):
+        """Test getting Nova servers when no servers exist."""
+        # Arrange
+        mock_conn = mock_openstack_connect.return_value
+        mock_conn.list_servers.return_value = []
+        nova_tools = NovaTools()
+
+        # Act
+        result = nova_tools.get_nova_servers()
+
+        # Assert
+        assert result == ""
+
+    def test_get_nova_servers_single_server(self, mock_openstack_connection, mock_server):
+        """Test getting single Nova server."""
+        # Arrange
+        mock_conn, mock_connect = mock_openstack_connection
+        test_server = mock_server("web-server", "web-001", "BUILDING")
+        mock_conn.list_servers.return_value = [test_server]
+        nova_tools = NovaTools()
+
+        # Act
+        result = nova_tools.get_nova_servers()
+
+        # Assert
+        expected = "web-server (web-001) - Status: BUILDING"
+        assert result == expected