feat(identity): Add create domain tool(#31)

- Add create_domain tool
- Tests are updated and passing

Author: S0okJu

src/openstack_mcp_server/tools/identity_tools.py

@@ -1,7 +1,7 @@
 from fastmcp import FastMCP
 
 from .base import get_openstack_conn
-from .response.identity import Region, Domain
+from .response.identity import Domain, Region
 
 
 class IdentityTools:
@@ -22,6 +22,7 @@ def register_tools(self, mcp: FastMCP):
 
         mcp.tool()(self.get_domains)
         mcp.tool()(self.get_domain)
+        mcp.tool()(self.create_domain)
 
     def get_regions(self) -> list[Region]:
         """
@@ -133,3 +134,25 @@ def get_domain(self, id: str) -> Domain:
 
         return Domain(id=domain.id, name=domain.name, description=domain.description, is_enabled=domain.is_enabled)
 
+    def create_domain(self, 
+        id: str, 
+        name: str, 
+        description: str = "",
+        is_enabled: bool = False,
+        ) -> Domain:
+        """
+        Create a new domain.
+
+        :param id: The ID of the domain. (required)
+        :param name: The name of the domain. (required)
+        :param description: The description of the domain. (optional)
+        :param is_enabled: Whether the domain is enabled. (optional)
+
+        :return: The created Domain object.
+        """
+
+        conn = get_openstack_conn()
+
+        domain = conn.identity.create_domain(id=id, name=name, description=description, is_enabled=is_enabled)
+
+        return Domain(id=domain.id, name=domain.name, description=domain.description, is_enabled=domain.is_enabled)

tests/tools/test_identity_tools.py

@@ -5,7 +5,7 @@
 from openstack import exceptions
 
 from openstack_mcp_server.tools.identity_tools import IdentityTools
-from openstack_mcp_server.tools.response.identity import Region, Domain
+from openstack_mcp_server.tools.response.identity import Domain, Region
 
 
 class TestIdentityTools:
@@ -428,3 +428,121 @@ def test_get_domain_not_found(self, mock_get_openstack_conn_identity):
 
         # Verify mock calls
         mock_conn.identity.get_domain.assert_called_once_with(domain="domainone")
+
+    def test_create_domain_success(self, mock_get_openstack_conn_identity):
+        """Test creating a identity domain successfully."""
+        mock_conn = mock_get_openstack_conn_identity
+
+        # Create mock domain object
+        mock_domain = Mock()
+        mock_domain.id = "domainone"
+        mock_domain.name = "DomainOne"
+        mock_domain.description = "Domain One description"
+        mock_domain.is_enabled = False
+
+        # Configure mock domain.create_domain()
+        mock_conn.identity.create_domain.return_value = mock_domain
+
+        # Test create_domain()
+        identity_tools = self.get_identity_tools()
+        result = identity_tools.create_domain(id="domainone", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+        # Verify results
+        assert result == Domain(id="domainone", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+        # Verify mock calls
+        mock_conn.identity.create_domain.assert_called_once_with(id="domainone", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+    
+    def test_create_domain_with_empty_id(self, mock_get_openstack_conn_identity):
+        """Test creating a identity domain with an empty ID."""
+        mock_conn = mock_get_openstack_conn_identity
+
+        # Create mock domain object
+        mock_domain = Mock()
+        mock_domain.id = "" 
+        mock_domain.name = "DomainOne"
+        mock_domain.description = "Domain One description"
+        mock_domain.is_enabled = False
+
+        # Configure mock domain.create_domain()
+        mock_conn.identity.create_domain.return_value = mock_domain
+
+        # If id is empty, the id will be generated by the server
+        temp_id = "abcdefghijklmnopqrs12345"
+        mock_domain.id = temp_id
+        
+        # Test create_domain()
+        identity_tools = self.get_identity_tools()
+        result = identity_tools.create_domain(id="", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+        # Verify results
+        assert result == Domain(id=temp_id, name="DomainOne", description="Domain One description", is_enabled=False)
+        
+    def test_create_domain_with_empty_name(self, mock_get_openstack_conn_identity):
+        """Test creating a identity domain with an empty name."""
+        mock_conn = mock_get_openstack_conn_identity
+        
+        # Create mock domain object
+        mock_domain = Mock()
+        mock_domain.id = "domainone"
+        mock_domain.name = ""
+        mock_domain.description = "Domain One description"
+        mock_domain.is_enabled = False
+
+        # Configure mock domain.create_domain()
+        mock_conn.identity.create_domain.return_value = mock_domain
+        
+        # Test create_domain()
+        identity_tools = self.get_identity_tools()
+        result = identity_tools.create_domain(id="domainone", name="", description="Domain One description", is_enabled=False)
+        
+        # Verify results
+        assert result == Domain(id="domainone", name="", description="Domain One description", is_enabled=False)
+    
+    def test_create_domain_with_empty_description(self, mock_get_openstack_conn_identity):
+        """Test creating a identity domain with an empty description."""
+        mock_conn = mock_get_openstack_conn_identity
+        
+        # Create mock domain object
+        mock_domain = Mock()
+        mock_domain.id = "domainone"
+        mock_domain.name = "DomainOne"
+        mock_domain.description = ""
+        mock_domain.is_enabled = False
+
+        # Configure mock domain.create_domain()
+        mock_conn.identity.create_domain.return_value = mock_domain
+        
+        # Test create_domain()
+        identity_tools = self.get_identity_tools()
+        result = identity_tools.create_domain(id="domainone", name="DomainOne", description="", is_enabled=False)
+        
+        # Verify results
+        assert result == Domain(id="domainone", name="DomainOne", description="", is_enabled=False)
+    
+    def test_create_domain_with_empty_is_enabled(self, mock_get_openstack_conn_identity):
+        """Test creating a identity domain with an empty is_enabled."""
+        mock_conn = mock_get_openstack_conn_identity
+        
+        # Create mock domain object
+        mock_domain = Mock()
+        mock_domain.id = "domainone"
+        mock_domain.name = "DomainOne"
+        mock_domain.description = "Domain One description"
+        mock_domain.is_enabled = False
+
+        # Configure mock domain.create_domain()
+        mock_conn.identity.create_domain.return_value = mock_domain
+
+        # Test create_domain()
+        identity_tools = self.get_identity_tools()
+        result = identity_tools.create_domain(id="domainone", name="DomainOne", description="Domain One description")
+        
+        # Verify results
+        assert result == Domain(id="domainone", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+        # Verify mock calls
+        mock_conn.identity.create_domain.assert_called_once_with(id="domainone", name="DomainOne", description="Domain One description", is_enabled=False)
+        
+