Update test suite to match new API interfaces

- Update mock factories to include unban_player method
- Refactor test assertions to use new method signatures (_make_get_request, _make_post_request)
- Update test expectations to match new player object parameter pattern
- Ensure test coverage for enhanced player action interfaces

Author: kevinnio

test/support/mock_factories.py

@@ -4,16 +4,16 @@
 from unittest.mock import MagicMock
 
 
-def create_mock_http_response(status_code=200, content=b'{"success": true}', 
+def create_mock_http_response(status_code=200, content=b'{"success": true}',
                               json_data=None, raise_for_status=None):
     """Create a mock HTTP response object.
-    
+
     Args:
         status_code: HTTP status code (default: 200)
         content: Response content bytes (default: b'{"success": true}')
         json_data: JSON data to return from json() method (default: {"success": True})
         raise_for_status: Exception to raise from raise_for_status() (default: None)
-    
+
     Returns:
         MagicMock configured as an HTTP response
     """
@@ -27,12 +27,12 @@ def create_mock_http_response(status_code=200, content=b'{"success": true}',
 
 def create_mock_rcon_console(command_response=None, close_return=None):
     """Create a mock RCON console object.
-    
+
     Args:
         command_response: Response to return from command() method
                          (default: "name,playerid,userid\\nPlayer1,pid1,uid1")
         close_return: Return value for close() method (default: None)
-    
+
     Returns:
         MagicMock configured as an RCON console
     """
@@ -44,7 +44,7 @@ def create_mock_rcon_console(command_response=None, close_return=None):
 
 def create_mock_api_client():
     """Create a mock API client with default return values.
-    
+
     Returns:
         MagicMock configured as an API client
     """
@@ -53,5 +53,5 @@ def create_mock_api_client():
     mock.get_player_names.return_value = []
     mock.kick_player.return_value = True
     mock.ban_player.return_value = True
+    mock.unban_player.return_value = True
     return mock
-