chore(identity): ruff format(#58)

Author: S0okJu

src/openstack_mcp_server/tools/identity_tools.py

@@ -25,7 +25,7 @@ def register_tools(self, mcp: FastMCP):
         mcp.tool()(self.create_domain)
         mcp.tool()(self.delete_domain)
         mcp.tool()(self.update_domain)
-        
+
         mcp.tool()(self.get_projects)
         mcp.tool()(self.get_project)
 
@@ -231,7 +231,7 @@ def get_projects(self) -> list[Project]:
         :return: A list of Project objects representing the projects.
         """
         conn = get_openstack_conn()
-        
+
         project_list = []
         for project in conn.identity.projects():
             project_list.append(
@@ -246,7 +246,7 @@ def get_projects(self) -> list[Project]:
             )
 
         return project_list
-    
+
     def get_project(self, name: str) -> Project:
         """
         Get a project.
@@ -257,7 +257,9 @@ def get_project(self, name: str) -> Project:
         """
         conn = get_openstack_conn()
 
-        project = conn.identity.find_project(name_or_id=name, ignore_missing=False)
+        project = conn.identity.find_project(
+            name_or_id=name, ignore_missing=False
+        )
 
         return Project(
             id=project.id,
@@ -267,4 +269,3 @@ def get_project(self, name: str) -> Project:
             domain_id=project.domain_id,
             parent_id=project.parent_id,
         )
-    

src/openstack_mcp_server/tools/response/identity.py

@@ -14,10 +14,11 @@ class Domain(BaseModel):
     description: str | None = None
     is_enabled: bool | None = None
 
+
 class Project(BaseModel):
     id: str
     name: str
     description: str | None = None
     is_enabled: bool | None = None
     domain_id: str | None = None
-    parent_id: str | None = None
+    parent_id: str | None = None

tests/tools/test_identity_tools.py

@@ -6,7 +6,11 @@
 from openstack import exceptions
 
 from openstack_mcp_server.tools.identity_tools import IdentityTools
-from openstack_mcp_server.tools.response.identity import Domain, Region, Project
+from openstack_mcp_server.tools.response.identity import (
+    Domain,
+    Region,
+    Project,
+)
 
 
 class TestIdentityTools:
@@ -736,14 +740,17 @@ def test_get_projects_success(self, mock_get_openstack_conn_identity):
         mock_project2.is_enabled = False
         mock_project2.domain_id = "domain22222222222222222222222222"
         mock_project2.parent_id = "default"
-        
+
         # Configure mock project.projects()
-        mock_conn.identity.projects.return_value = [mock_project1, mock_project2]
-        
+        mock_conn.identity.projects.return_value = [
+            mock_project1,
+            mock_project2,
+        ]
+
         # Test get_projects()
         identity_tools = self.get_identity_tools()
         result = identity_tools.get_projects()
-        
+
         # Verify results
         assert result == [
             Project(
@@ -763,29 +770,27 @@ def test_get_projects_success(self, mock_get_openstack_conn_identity):
                 parent_id="default",
             ),
         ]
-        
+
         # Verify mock calls
         mock_conn.identity.projects.assert_called_once()
-        
-        
-    
+
     def test_get_projects_empty_list(self, mock_get_openstack_conn_identity):
         """Test getting identity projects when there are no projects."""
         mock_conn = mock_get_openstack_conn_identity
 
         # Empty project list
         mock_conn.identity.projects.return_value = []
-        
+
         # Test get_projects()
         identity_tools = self.get_identity_tools()
         result = identity_tools.get_projects()
-        
+
         # Verify results
         assert result == []
-        
+
         # Verify mock calls
         mock_conn.identity.projects.assert_called_once()
-    
+
     def test_get_project_success(self, mock_get_openstack_conn_identity):
         """Test getting a identity project successfully."""
         mock_conn = mock_get_openstack_conn_identity
@@ -805,7 +810,7 @@ def test_get_project_success(self, mock_get_openstack_conn_identity):
         # Test get_project()
         identity_tools = self.get_identity_tools()
         result = identity_tools.get_project(name="ProjectOne")
-        
+
         # Verify results
         assert result == Project(
             id="project1111111111111111111111111",
@@ -815,13 +820,13 @@ def test_get_project_success(self, mock_get_openstack_conn_identity):
             domain_id="domain1111111111111111111111111",
             parent_id="parentproject1111111111111111111",
         )
-        
+
         # Verify mock calls
         mock_conn.identity.find_project.assert_called_once_with(
             name_or_id="ProjectOne",
             ignore_missing=False,
         )
-    
+
     def test_get_project_not_found(self, mock_get_openstack_conn_identity):
         """Test getting a identity project that does not exist."""
         mock_conn = mock_get_openstack_conn_identity
@@ -835,22 +840,16 @@ def test_get_project_not_found(self, mock_get_openstack_conn_identity):
 
         # Test get_project()
         identity_tools = self.get_identity_tools()
-        
+
         # Verify exception is raised
         with pytest.raises(
             exceptions.NotFoundException,
             match="Project 'ProjectOne' not found",
         ):
             identity_tools.get_project(name="ProjectOne")
-            
+
         # Verify mock calls
         mock_conn.identity.find_project.assert_called_once_with(
             name_or_id="ProjectOne",
             ignore_missing=False,
         )
-        
-        
-        
-        
-        
-