feat(network): rename port to security group (#87)

Author: platanus-kr

src/openstack_mcp_server/tools/network_tools.py

@@ -45,7 +45,7 @@ def register_tools(self, mcp: FastMCP):
         mcp.tool()(self.get_port_allowed_address_pairs)
         mcp.tool()(self.set_port_binding)
         mcp.tool()(self.add_port_to_security_group)
-        mcp.tool()(self.remove_security_group_from_port)
+        mcp.tool()(self.remove_port_from_security_group)
         mcp.tool()(self.get_floating_ips)
         mcp.tool()(self.create_floating_ip)
         mcp.tool()(self.delete_floating_ip)
@@ -544,7 +544,7 @@ def add_port_to_security_group(
         )
         return self._convert_to_port_model(updated)
 
-    def remove_security_group_from_port(
+    def remove_port_from_security_group(
         self,
         port_id: str,
         security_group_id: str,

tests/tools/test_network_tools.py

@@ -937,15 +937,15 @@ def test_add_remove_security_group_on_port(
         updated_remove.security_group_ids = ["sg-1", "sg-2"]
         mock_conn.network.update_port.return_value = updated_remove
 
-        res_remove = tools.remove_security_group_from_port("port-1", "sg-9")
+        res_remove = tools.remove_port_from_security_group("port-1", "sg-9")
         assert isinstance(res_remove, Port)
         mock_conn.network.update_port.assert_called_with(
             "port-1", security_groups=["sg-1", "sg-2"]
         )
 
         # idempotent remove when sg not present
         mock_conn.network.get_port.return_value = updated_remove
-        res_remove_again = tools.remove_security_group_from_port(
+        res_remove_again = tools.remove_port_from_security_group(
             "port-1", "sg-9"
         )
         assert isinstance(res_remove_again, Port)