test: 进一步提高测试覆盖率

Author: he0119

src/utils/validation/models.py

@@ -76,11 +76,11 @@ def prevent_duplication(
         project_link = values.get("project_link")
 
         context = info.context
-        if context is None:
-            raise ValueError("未获取到验证上下文")
+        if context is None:  # pragma: no cover
+            raise PydanticCustomError("validation_context", "未获取到验证上下文")
         data = context.get("previous_data")
         if data is None:
-            raise ValueError("未获取到数据列表")
+            raise PydanticCustomError("previous_data", "未获取到数据列表")
 
         if (
             module_name
@@ -121,8 +121,8 @@ def collect_valid_values(
         cls, v: Any, handler: ValidatorFunctionWrapHandler, info: ValidationInfo
     ):
         context = info.context
-        if context is None:
-            raise ValueError("未获取到验证上下文")
+        if context is None:  # pragma: no cover
+            raise PydanticCustomError("validation_context", "未获取到验证上下文")
 
         result = handler(v)
         context["valid_data"][info.field_name] = result
@@ -180,8 +180,8 @@ def supported_adapters_validator(
         info: ValidationInfo,
     ) -> list[str] | None:
         context = info.context
-        if context is None:
-            raise ValueError("未获取到验证上下文")
+        if context is None:  # pragma: no cover
+            raise PydanticCustomError("validation_context", "未获取到验证上下文")
 
         skip_plugin_test = context.get("skip_plugin_test")
         # 如果是从 issue 中获取的数据，需要先解码

tests/utils/validation/fields/test_pypi.py

@@ -99,3 +99,38 @@ async def test_name_duplication(mocked_api: MockRouter) -> None:
 
     assert not mocked_api["project_link1"].called
     assert not mocked_api["homepage"].called
+
+
+async def test_name_duplication_previos_data_missing(mocked_api: MockRouter) -> None:
+    """没有提供 previos_data 的情况"""
+    from src.utils.validation import PublishType, validate_info
+
+    data = generate_adapter_data(
+        module_name="module_name1",
+        project_link="project_link1",
+        previous_data=None,
+    )
+
+    result = validate_info(PublishType.ADAPTER, data)
+
+    assert not result["valid"]
+    assert not result["data"]
+    assert result["errors"] == [
+        {
+            "type": "previous_data",
+            "loc": (),
+            "msg": "未获取到数据列表",
+            "input": {
+                "name": "name",
+                "desc": "desc",
+                "author": "author",
+                "module_name": "module_name1",
+                "project_link": "project_link1",
+                "homepage": "https://nonebot.dev",
+                "tags": '[{"label": "test", "color": "#ffffff"}]',
+            },
+        }
+    ]
+
+    assert not mocked_api["project_link1"].called
+    assert not mocked_api["homepage"].called