test: align vendor construction with typed pydantic constructors (mypy)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: plutoless

tests/custom/test_agentkit_vendors.py

@@ -89,7 +89,7 @@ def test_anam_avatar_serializes_avatar_id() -> None:
 
 def test_anam_avatar_requires_avatar_id() -> None:
     with pytest.raises(ValidationError):
-        AnamAvatar(api_key="anam-key")
+        AnamAvatar(api_key="anam-key")  # type: ignore[call-arg]
 
 
 def test_vertex_ai_explicit_fields_override_additional_params():

tests/custom/test_sensetime_avatar.py

@@ -42,8 +42,8 @@ def _scene_list() -> List[Dict[str, Any]]:
 
 def test_sensetime_avatar_accepts_snake_case() -> None:
     from agora_agent.agentkit.vendors.cn import SenseTimeAvatar
-    camel = SenseTimeAvatar(agora_uid="2", appId="app", app_key="key").to_config()
     snake = SenseTimeAvatar(agora_uid="2", app_id="app", app_key="key").to_config()
+    camel = SenseTimeAvatar(**{"agora_uid": "2", "appId": "app", "app_key": "key"}).to_config()
     assert camel == snake
     assert snake["params"]["appId"] == "app"
 
@@ -52,9 +52,9 @@ def test_sensetime_avatar_to_config_shape() -> None:
     config = SenseTimeAvatar(
         agora_token="avatar-token",
         agora_uid="2",
-        appId="sensetime-app-id",
+        app_id="sensetime-app-id",
         app_key="sensetime-app-key",
-        sceneList=_scene_list(),
+        scene_list=_scene_list(),
     ).to_config()
 
     assert config == {
@@ -75,7 +75,7 @@ def test_sensetime_avatar_to_config_omits_token_when_not_provided() -> None:
     config = SenseTimeAvatar(
         agora_uid="2",
         app_key="sensetime-app-key",
-        sceneList=_scene_list(),
+        scene_list=_scene_list(),
     ).to_config()
 
     assert "agora_token" not in config["params"]
@@ -142,7 +142,7 @@ def test_sensetime_avatar_session_validation_and_token_passthrough() -> None:
             agora_token="avatar-token",
             agora_uid="2",
             app_key="sensetime-app-key",
-            sceneList=_scene_list(),
+            scene_list=_scene_list(),
         )
     )
     session = _session(agent)
@@ -164,7 +164,7 @@ def test_sensetime_avatar_enrichment_generates_token() -> None:
         SenseTimeAvatar(
             agora_uid="2",
             app_key="sensetime-app-key",
-            sceneList=_scene_list(),
+            scene_list=_scene_list(),
         )
     )
     session = _session(agent)
@@ -187,7 +187,7 @@ def test_sensetime_avatar_user_token_is_not_overwritten() -> None:
             agora_uid="2",
             agora_token="user-token",
             app_key="sensetime-app-key",
-            sceneList=_scene_list(),
+            scene_list=_scene_list(),
         )
     )
     session = _session(agent)

tests/custom/test_vendor_collapse_golden.py

@@ -85,9 +85,10 @@ def test_aliyun_llm_pins_vendor_golden() -> None:
 
 
 def test_sensetime_avatar_camelcase_golden() -> None:
-    # SenseTimeAvatarOptions uses alias "appId" for the app_id field;
-    # pydantic v2 requires the alias keyword in the constructor.
-    cfg = SenseTimeAvatar(agora_uid="2", appId="app", app_key="key").to_config()
+    # Verifies that the camelCase alias "appId" still works as constructor input
+    # (populate_by_name=True means both the field name and alias are accepted at runtime).
+    # Dict-unpack is used so mypy does not call-arg-check the alias keyword.
+    cfg = SenseTimeAvatar(**{"agora_uid": "2", "appId": "app", "app_key": "key"}).to_config()
     assert cfg["vendor"] == "sensetime"
     assert cfg["params"]["appId"] == "app"
 
@@ -104,5 +105,5 @@ def test_fengming_rejects_kwargs() -> None:
     import pytest
     from pydantic import ValidationError
     with pytest.raises(ValidationError):
-        FengmingSTT(unexpected="x")
+        FengmingSTT(unexpected="x")  # type: ignore[call-arg]
     assert FengmingSTT().to_config() == {"vendor": "fengming"}