Fix unit tests: correct URL validation errors, mock paths, and ValidationError params

Co-authored-by: 0xrinegade <[email protected]>

Author: Copilot

python/tests/unit/test_additional_coverage_boost.py

@@ -169,9 +169,9 @@ def test_agent_registry_entry_edge_cases(self):
 
     def test_service_endpoint_edge_cases(self):
         """Test ServiceEndpoint with different protocols."""
-        # Test explicit protocol setting
+        # Test explicit protocol setting with valid URL
         endpoint = ServiceEndpoint(
-            url="custom://example.com", protocol="custom", auth_type="bearer"
+            url="https://example.com", protocol="custom", auth_type="bearer"
         )
         assert endpoint.protocol == "custom"
         assert endpoint.auth_type == "bearer"
@@ -222,7 +222,7 @@ def test_client_with_custom_commitment(self, mock_async_client):
 class TestAgentRegistryAdditionalCoverage:
     """Additional tests for agent registry functionality."""
 
-    @patch("solana_ai_registries.agent.AsyncClient")
+    @patch("solana_ai_registries.client.AsyncClient")
     def test_agent_registry_initialization(self, mock_async_client):
         """Test AgentRegistry initialization."""
         mock_client = Mock()
@@ -233,7 +233,7 @@ def test_agent_registry_initialization(self, mock_async_client):
 class TestMcpServerRegistryAdditionalCoverage:
     """Additional tests for MCP server registry functionality."""
 
-    @patch("solana_ai_registries.mcp.AsyncClient")
+    @patch("solana_ai_registries.client.AsyncClient")
     def test_mcp_registry_initialization(self, mock_async_client):
         """Test McpServerRegistry initialization."""
         mock_client = Mock()
@@ -244,7 +244,7 @@ def test_mcp_registry_initialization(self, mock_async_client):
 class TestPaymentManagerAdditionalCoverage:
     """Additional tests for payment manager functionality."""
 
-    @patch("solana_ai_registries.payments.AsyncClient")
+    @patch("solana_ai_registries.client.AsyncClient")
     def test_payment_manager_initialization(self, mock_async_client):
         """Test PaymentManager initialization."""
         mock_client = Mock()
@@ -313,7 +313,7 @@ def test_all_validation_scenarios(self):
         # String length validation
         validate_string_length("short", 10, "test")
 
-        with pytest.raises(ValueError, match="test must be at most"):
+        with pytest.raises(ValueError, match="test exceeds maximum length"):
             validate_string_length("very_long_string_that_exceeds_limit", 5, "test")
 
         # URL validation
@@ -335,7 +335,7 @@ def test_all_validation_scenarios(self):
         ]
 
         for url in invalid_urls:
-            with pytest.raises(ValueError, match="must be a valid URL"):
+            with pytest.raises(ValueError, match="must start with one of"):
                 validate_url(url, "test_url")
 
         # Invalid schemes

python/tests/unit/test_coverage_booster.py

@@ -39,10 +39,10 @@ def test_validation_functions(self):
         validate_url("http://example.com", "test_url")
 
         # Invalid cases
-        with pytest.raises(ValueError, match="test_field must be at most"):
+        with pytest.raises(ValueError, match="test_field exceeds maximum length"):
             validate_string_length("too_long_string", 5, "test_field")
 
-        with pytest.raises(ValueError, match="test_url must be a valid URL"):
+        with pytest.raises(ValueError, match="test_url must start with one of"):
             validate_url("not_a_url", "test_url")
 
         with pytest.raises(ValueError, match="test_url must start with one of"):
@@ -97,7 +97,7 @@ def test_exception_hierarchy(self):
         assert agent_error.owner == "owner123"
 
         validation_error = ValidationError(
-            "test_field", "test_constraint", "test_value"
+            "test_field", "test_value", "test_constraint"
         )
         assert validation_error.field == "test_field"
         assert validation_error.constraint == "test_constraint"