✨ 支持git更新（github与aliyun codeup），插件商店支持aliyun codeup (#1999)

* ✨ feat(env): 支持git更新

* ✨ feat(aliyun): 更新阿里云URL构建逻辑，支持组织名称并优化令牌解码处理

* ✨ feat(config): 修改错误提示信息，更新基础配置文件名称为.env.example

* ⚡ 插件商店支持aliyun

* ✨ feat(store): 优化插件数据获取逻辑，合并插件列表和额外插件列表

* 🐛 修复非git仓库的初始化更新

* ✨ feat(update): 增强更新提示信息，添加非git源的变更文件说明

* 🎨 代码格式化

* ✨ webui与resources支持git更新

* ✨ feat(update): 更新webui路径处理逻辑

* Fix/test_runwork (#2001)

* fix(test): 修复测试工作流

- 修改自动更新模块中的导入路径
- 更新插件商店模块中的插件信息获取逻辑
- 优化插件添加、更新和移除流程
- 统一插件相关错误信息的格式
- 调整测试用例以适应新的插件管理逻辑

* test(builtin_plugins): 重构插件商店相关测试

- 移除 jsd 相关测试用例，只保留 gh（GitHub）的测试
- 删除了 test_plugin_store.py 文件，清理了插件商店的测试
- 更新了 test_search_plugin.py 中的插件版本号
- 调整了 test_update_plugin.py 中的已加载插件版本
- 移除了 StoreManager 类中的 is_external 变量
- 更新了 RepoFileManager 类中的文件获取逻辑，优先使用 GitHub

* ✨ feat(submodule): 添加子模块管理功能，支持子模块的初始化、更新和信息获取

* ✨ feat(update): 移除资源管理器，重构更新逻辑，支持通过ZhenxunRepoManager进行资源和Web UI的更新

* test(auto_update): 修改更新检测消息格式 (#2003)

- 移除了不必要的版本号后缀（如 "-e6f17c4"）
- 统一了版本更新消息的格式，删除了冗余信息

* 🐛 修复web zip更新路径问题

* ⚡ 文件获取优化使用ali

* Fix/test (#2008)

* test: 修复bot测试

- 在 test_check_update.py 中跳过两个测试函数
- 移除 test_check.py 中的 mocked_api 参数和相关调用
- 删除 test_add_plugin.py 中的多个测试函数
- 移除 test_remove_plugin.py 中的 mocked_api 参数和相关调用
- 删除 test_search_plugin.py 中的多个测试函数
- 移除 test_update_all_plugin.py 和 test_update_plugin.py 中的 mocked_api 参数和相关调用

* 🚨 auto fix by pre-commit hooks

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* 修复res zip更新路径问题

* 🐛 修复zhenxun更新zip占用问题

* ✨ feat(update): 优化资源更新逻辑，调整更新路径和消息处理

---------

Co-authored-by: molanp <104612722+molanp@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

tests/builtin_plugins/plugin_store/test_add_plugin.py

@@ -6,23 +6,17 @@ from nonebot.adapters.onebot.v11 import Bot
 from nonebot.adapters.onebot.v11.event import GroupMessageEvent
 from nonebot.adapters.onebot.v11.message import Message
 from nonebug import App
-import pytest
 from pytest_mock import MockerFixture
-from respx import MockRouter
 
-from tests.builtin_plugins.plugin_store.utils import init_mocked_api
 from tests.config import BotId, GroupId, MessageId, UserId
 from tests.utils import _v11_group_message_event
 
+test_path = Path(__file__).parent.parent.parent
+
 
-@pytest.mark.parametrize("package_api", ["jsd", "gh"])
-@pytest.mark.parametrize("is_commit", [True, False])
 async def test_add_plugin_basic(
-    package_api: str,
-    is_commit: bool,
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -31,24 +25,12 @@ async def test_add_plugin_basic(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
 
-    if package_api != "jsd":
-        mocked_api["zhenxun_bot_plugins_metadata"].respond(404)
-    if package_api != "gh":
-        mocked_api["zhenxun_bot_plugins_tree"].respond(404)
-
-    if not is_commit:
-        mocked_api["zhenxun_bot_plugins_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_commit_proxy"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit_proxy"].respond(404)
-
-    plugin_id = 1
+    plugin_id = "search_image"
 
     async with app.test_matcher(_matcher) as ctx:
         bot = create_bot(ctx)
@@ -65,7 +47,7 @@ async def test_add_plugin_basic(
         ctx.receive_event(bot=bot, event=event)
         ctx.should_call_send(
             event=event,
-            message=Message(message=f"正在添加插件 Id: {plugin_id}"),
+            message=Message(message=f"正在添加插件 Module: {plugin_id}"),
             result=None,
             bot=bot,
         )
@@ -75,25 +57,12 @@ async def test_add_plugin_basic(
             result=None,
             bot=bot,
         )
-    if is_commit:
-        assert mocked_api["search_image_plugin_file_init_commit"].called
-        assert mocked_api["basic_plugins"].called
-        assert mocked_api["extra_plugins"].called
-    else:
-        assert mocked_api["search_image_plugin_file_init"].called
-        assert mocked_api["basic_plugins_no_commit"].called
-        assert mocked_api["extra_plugins_no_commit"].called
     assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file()
 
 
-@pytest.mark.parametrize("package_api", ["jsd", "gh"])
-@pytest.mark.parametrize("is_commit", [True, False])
 async def test_add_plugin_basic_commit_version(
-    package_api: str,
-    is_commit: bool,
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -102,23 +71,12 @@ async def test_add_plugin_basic_commit_version(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
 
-    if package_api != "jsd":
-        mocked_api["zhenxun_bot_plugins_metadata_commit"].respond(404)
-    if package_api != "gh":
-        mocked_api["zhenxun_bot_plugins_tree_commit"].respond(404)
-
-    if not is_commit:
-        mocked_api["zhenxun_bot_plugins_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_commit_proxy"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit_proxy"].respond(404)
-    plugin_id = 3
+    plugin_id = "bilibili_sub"
 
     async with app.test_matcher(_matcher) as ctx:
         bot = create_bot(ctx)
@@ -135,7 +93,7 @@ async def test_add_plugin_basic_commit_version(
         ctx.receive_event(bot=bot, event=event)
         ctx.should_call_send(
             event=event,
-            message=Message(message=f"正在添加插件 Id: {plugin_id}"),
+            message=Message(message=f"正在添加插件 Module: {plugin_id}"),
             result=None,
             bot=bot,
         )
@@ -145,28 +103,12 @@ async def test_add_plugin_basic_commit_version(
             result=None,
             bot=bot,
         )
-    if package_api == "jsd":
-        assert mocked_api["zhenxun_bot_plugins_metadata_commit"].called
-    if package_api == "gh":
-        assert mocked_api["zhenxun_bot_plugins_tree_commit"].called
-    if is_commit:
-        assert mocked_api["basic_plugins"].called
-        assert mocked_api["extra_plugins"].called
-    else:
-        assert mocked_api["basic_plugins_no_commit"].called
-        assert mocked_api["extra_plugins_no_commit"].called
-    assert mocked_api["bilibili_sub_plugin_file_init"].called
     assert (mock_base_path / "plugins" / "bilibili_sub" / "__init__.py").is_file()
 
 
-@pytest.mark.parametrize("package_api", ["jsd", "gh"])
-@pytest.mark.parametrize("is_commit", [True, False])
 async def test_add_plugin_basic_is_not_dir(
-    package_api: str,
-    is_commit: bool,
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -175,24 +117,12 @@ async def test_add_plugin_basic_is_not_dir(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
 
-    if package_api != "jsd":
-        mocked_api["zhenxun_bot_plugins_metadata"].respond(404)
-    if package_api != "gh":
-        mocked_api["zhenxun_bot_plugins_tree"].respond(404)
-
-    if not is_commit:
-        mocked_api["zhenxun_bot_plugins_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_commit_proxy"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit_proxy"].respond(404)
-
-    plugin_id = 0
+    plugin_id = "jitang"
 
     async with app.test_matcher(_matcher) as ctx:
         bot = create_bot(ctx)
@@ -209,7 +139,7 @@ async def test_add_plugin_basic_is_not_dir(
         ctx.receive_event(bot=bot, event=event)
         ctx.should_call_send(
             event=event,
-            message=Message(message=f"正在添加插件 Id: {plugin_id}"),
+            message=Message(message=f"正在添加插件 Module: {plugin_id}"),
             result=None,
             bot=bot,
         )
@@ -219,25 +149,12 @@ async def test_add_plugin_basic_is_not_dir(
             result=None,
             bot=bot,
         )
-    if is_commit:
-        assert mocked_api["jitang_plugin_file_commit"].called
-        assert mocked_api["basic_plugins"].called
-        assert mocked_api["extra_plugins"].called
-    else:
-        assert mocked_api["jitang_plugin_file"].called
-        assert mocked_api["basic_plugins_no_commit"].called
-        assert mocked_api["extra_plugins_no_commit"].called
-    assert (mock_base_path / "plugins" / "alapi" / "jitang.py").is_file()
+    assert (mock_base_path / "plugins" / "jitang.py").is_file()
 
 
-@pytest.mark.parametrize("package_api", ["jsd", "gh"])
-@pytest.mark.parametrize("is_commit", [True, False])
 async def test_add_plugin_extra(
-    package_api: str,
-    is_commit: bool,
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -246,26 +163,12 @@ async def test_add_plugin_extra(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
 
-    if package_api != "jsd":
-        mocked_api["zhenxun_github_sub_metadata"].respond(404)
-    if package_api != "gh":
-        mocked_api["zhenxun_github_sub_tree"].respond(404)
-
-    if not is_commit:
-        mocked_api["zhenxun_github_sub_commit"].respond(404)
-        mocked_api["zhenxun_github_sub_commit_proxy"].respond(404)
-        mocked_api["zhenxun_bot_plugins_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_commit_proxy"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit"].respond(404)
-        mocked_api["zhenxun_bot_plugins_index_commit_proxy"].respond(404)
-
-    plugin_id = 4
+    plugin_id = "github_sub"
 
     async with app.test_matcher(_matcher) as ctx:
         bot = create_bot(ctx)
@@ -282,7 +185,7 @@ async def test_add_plugin_extra(
         ctx.receive_event(bot=bot, event=event)
         ctx.should_call_send(
             event=event,
-            message=Message(message=f"正在添加插件 Id: {plugin_id}"),
+            message=Message(message=f"正在添加插件 Module: {plugin_id}"),
             result=None,
             bot=bot,
         )
@@ -292,30 +195,18 @@ async def test_add_plugin_extra(
             result=None,
             bot=bot,
         )
-    if is_commit:
-        assert mocked_api["github_sub_plugin_file_init_commit"].called
-        assert mocked_api["basic_plugins"].called
-        assert mocked_api["extra_plugins"].called
-    else:
-        assert mocked_api["github_sub_plugin_file_init"].called
-        assert mocked_api["basic_plugins_no_commit"].called
-        assert mocked_api["extra_plugins_no_commit"].called
     assert (mock_base_path / "plugins" / "github_sub" / "__init__.py").is_file()
 
 
 async def test_plugin_not_exist_add(
     app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件不存在，添加插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     plugin_id = -1
 
     async with app.test_matcher(_matcher) as ctx:
@@ -339,7 +230,7 @@ async def test_plugin_not_exist_add(
         )
         ctx.should_call_send(
             event=event,
-            message=Message(message="插件ID不存在..."),
+            message=Message(message="添加插件 Id: -1 失败 e: 插件ID不存在..."),
             result=None,
             bot=bot,
         )
@@ -348,16 +239,13 @@ async def test_plugin_not_exist_add(
 async def test_add_plugin_exist(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件已经存在，添加插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.StoreManager.get_loaded_plugins",
         return_value=[("search_image", "0.1")],
@@ -385,7 +273,9 @@ async def test_add_plugin_exist(
         )
         ctx.should_call_send(
             event=event,
-            message=Message(message="插件 识图 已安装，无需重复安装"),
+            message=Message(
+                message="添加插件 Id: 1 失败 e: 插件 识图 已安装，无需重复安装"
+            ),
             result=None,
             bot=bot,
         )

tests/builtin_plugins/plugin_store/test_plugin_store.py

@@ -1,140 +0,0 @@
-from collections.abc import Callable
-from pathlib import Path
-from typing import cast
-
-from nonebot.adapters.onebot.v11 import Bot, Message
-from nonebot.adapters.onebot.v11.event import GroupMessageEvent
-from nonebug import App
-from pytest_mock import MockerFixture
-from respx import MockRouter
-
-from tests.builtin_plugins.plugin_store.utils import init_mocked_api
-from tests.config import BotId, GroupId, MessageId, UserId
-from tests.utils import _v11_group_message_event
-
-
-async def test_plugin_store(
-    app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
-    create_bot: Callable,
-    tmp_path: Path,
-) -> None:
-    """
-    测试插件商店
-    """
-    from zhenxun.builtin_plugins.plugin_store import _matcher
-    from zhenxun.builtin_plugins.plugin_store.data_source import row_style
-
-    init_mocked_api(mocked_api=mocked_api)
-
-    mock_table_page = mocker.patch(
-        "zhenxun.builtin_plugins.plugin_store.data_source.ImageTemplate.table_page"
-    )
-    mock_table_page_return = mocker.AsyncMock()
-    mock_table_page.return_value = mock_table_page_return
-
-    mock_build_message = mocker.patch(
-        "zhenxun.builtin_plugins.plugin_store.MessageUtils.build_message"
-    )
-    mock_build_message_return = mocker.AsyncMock()
-    mock_build_message.return_value = mock_build_message_return
-
-    async with app.test_matcher(_matcher) as ctx:
-        bot = create_bot(ctx)
-        bot: Bot = cast(Bot, bot)
-        raw_message = "插件商店"
-        event: GroupMessageEvent = _v11_group_message_event(
-            message=raw_message,
-            self_id=BotId.QQ_BOT,
-            user_id=UserId.SUPERUSER,
-            group_id=GroupId.GROUP_ID_LEVEL_5,
-            message_id=MessageId.MESSAGE_ID_3,
-            to_me=True,
-        )
-        ctx.receive_event(bot=bot, event=event)
-    mock_table_page.assert_awaited_once_with(
-        "插件列表",
-        "通过添加/移除插件 ID 来管理插件",
-        ["-", "ID", "名称", "简介", "作者", "版本", "类型"],
-        [
-            ["", 0, "鸡汤", "喏，亲手为你煮的鸡汤", "HibiKier", "0.1", "普通插件"],
-            ["", 1, "识图", "以图搜图，看破本源", "HibiKier", "0.1", "普通插件"],
-            ["", 2, "网易云热评", "生了个人，我很抱歉", "HibiKier", "0.1", "普通插件"],
-            [
-                "",
-                3,
-                "B站订阅",
-                "非常便利的B站订阅通知",
-                "HibiKier",
-                "0.3-b101fbc",
-                "普通插件",
-            ],
-            [
-                "",
-                4,
-                "github订阅",
-                "订阅github用户或仓库",
-                "xuanerwa",
-                "0.7",
-                "普通插件",
-            ],
-            [
-                "",
-                5,
-                "Minecraft查服",
-                "Minecraft服务器状态查询，支持IPv6",
-                "molanp",
-                "1.13",
-                "普通插件",
-            ],
-        ],
-        text_style=row_style,
-    )
-    mock_build_message.assert_called_once_with(mock_table_page_return)
-    mock_build_message_return.send.assert_awaited_once()
-
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
-
-
-async def test_plugin_store_fail(
-    app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
-    create_bot: Callable,
-    tmp_path: Path,
-) -> None:
-    """
-    测试插件商店
-    """
-    from zhenxun.builtin_plugins.plugin_store import _matcher
-
-    init_mocked_api(mocked_api=mocked_api)
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/b101fbc/plugins.json",
-        name="basic_plugins",
-    ).respond(404)
-
-    async with app.test_matcher(_matcher) as ctx:
-        bot = create_bot(ctx)
-        bot: Bot = cast(Bot, bot)
-        raw_message = "插件商店"
-        event: GroupMessageEvent = _v11_group_message_event(
-            message=raw_message,
-            self_id=BotId.QQ_BOT,
-            user_id=UserId.SUPERUSER,
-            group_id=GroupId.GROUP_ID_LEVEL_5,
-            message_id=MessageId.MESSAGE_ID_3,
-            to_me=True,
-        )
-        ctx.receive_event(bot=bot, event=event)
-        ctx.should_call_send(
-            event=event,
-            message=Message("获取插件列表失败..."),
-            result=None,
-            exception=None,
-            bot=bot,
-        )
-
-    assert mocked_api["basic_plugins"].called

tests/builtin_plugins/plugin_store/test_remove_plugin.py

@@ -9,9 +9,7 @@ from nonebot.adapters.onebot.v11.event import GroupMessageEvent
 from nonebot.adapters.onebot.v11.message import Message
 from nonebug import App
 from pytest_mock import MockerFixture
-from respx import MockRouter
 
-from tests.builtin_plugins.plugin_store.utils import get_content_bytes, init_mocked_api
 from tests.config import BotId, GroupId, MessageId, UserId
 from tests.utils import _v11_group_message_event
 
@@ -19,7 +17,6 @@ from tests.utils import _v11_group_message_event
 async def test_remove_plugin(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -28,7 +25,6 @@ async def test_remove_plugin(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
@@ -38,7 +34,7 @@ async def test_remove_plugin(
     plugin_path.mkdir(parents=True, exist_ok=True)
 
     with open(plugin_path / "__init__.py", "wb") as f:
-        f.write(get_content_bytes("search_image.py"))
+        f.write(b"A_nmi")
 
     plugin_id = 1
 
@@ -61,24 +57,18 @@ async def test_remove_plugin(
             result=None,
             bot=bot,
         )
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
     assert not (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file()
 
 
 async def test_plugin_not_exist_remove(
     app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件不存在，移除插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     plugin_id = -1
 
     async with app.test_matcher(_matcher) as ctx:
@@ -96,7 +86,7 @@ async def test_plugin_not_exist_remove(
         ctx.receive_event(bot=bot, event=event)
         ctx.should_call_send(
             event=event,
-            message=Message(message="插件ID不存在..."),
+            message=Message(message="移除插件 Id: -1 失败 e: 插件ID不存在..."),
             result=None,
             bot=bot,
         )
@@ -105,7 +95,6 @@ async def test_plugin_not_exist_remove(
 async def test_remove_plugin_not_install(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -114,7 +103,6 @@ async def test_remove_plugin_not_install(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     _ = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",

tests/builtin_plugins/plugin_store/test_search_plugin.py

@@ -1,5 +1,4 @@
 from collections.abc import Callable
-from pathlib import Path
 from typing import cast
 
 from nonebot.adapters.onebot.v11 import Bot
@@ -7,9 +6,7 @@ from nonebot.adapters.onebot.v11.event import GroupMessageEvent
 from nonebot.adapters.onebot.v11.message import Message
 from nonebug import App
 from pytest_mock import MockerFixture
-from respx import MockRouter
 
-from tests.builtin_plugins.plugin_store.utils import init_mocked_api
 from tests.config import BotId, GroupId, MessageId, UserId
 from tests.utils import _v11_group_message_event
 
@@ -17,17 +14,12 @@ from tests.utils import _v11_group_message_event
 async def test_search_plugin_name(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试搜索插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
-    from zhenxun.builtin_plugins.plugin_store.data_source import row_style
-
-    init_mocked_api(mocked_api=mocked_api)
 
     mock_table_page = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.ImageTemplate.table_page"
@@ -56,44 +48,19 @@ async def test_search_plugin_name(
             to_me=True,
         )
         ctx.receive_event(bot=bot, event=event)
-    mock_table_page.assert_awaited_once_with(
-        "商店插件列表",
-        "通过添加/移除插件 ID 来管理插件",
-        ["-", "ID", "名称", "简介", "作者", "版本", "类型"],
-        [
-            [
-                "",
-                4,
-                "github订阅",
-                "订阅github用户或仓库",
-                "xuanerwa",
-                "0.7",
-                "普通插件",
-            ]
-        ],
-        text_style=row_style,
-    )
     mock_build_message.assert_called_once_with(mock_table_page_return)
     mock_build_message_return.send.assert_awaited_once()
 
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
-
 
 async def test_search_plugin_author(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试搜索插件，作者
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
-    from zhenxun.builtin_plugins.plugin_store.data_source import row_style
-
-    init_mocked_api(mocked_api=mocked_api)
 
     mock_table_page = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.ImageTemplate.table_page"
@@ -122,43 +89,19 @@ async def test_search_plugin_author(
             to_me=True,
         )
         ctx.receive_event(bot=bot, event=event)
-    mock_table_page.assert_awaited_once_with(
-        "商店插件列表",
-        "通过添加/移除插件 ID 来管理插件",
-        ["-", "ID", "名称", "简介", "作者", "版本", "类型"],
-        [
-            [
-                "",
-                4,
-                "github订阅",
-                "订阅github用户或仓库",
-                "xuanerwa",
-                "0.7",
-                "普通插件",
-            ]
-        ],
-        text_style=row_style,
-    )
     mock_build_message.assert_called_once_with(mock_table_page_return)
     mock_build_message_return.send.assert_awaited_once()
 
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
-
 
 async def test_plugin_not_exist_search(
     app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件不存在，搜索插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     plugin_name = "not_exist_plugin_name"
 
     async with app.test_matcher(_matcher) as ctx:

tests/builtin_plugins/plugin_store/test_update_all_plugin.py

@@ -7,9 +7,7 @@ from nonebot.adapters.onebot.v11.event import GroupMessageEvent
 from nonebot.adapters.onebot.v11.message import Message
 from nonebug import App
 from pytest_mock import MockerFixture
-from respx import MockRouter
 
-from tests.builtin_plugins.plugin_store.utils import init_mocked_api
 from tests.config import BotId, GroupId, MessageId, UserId
 from tests.utils import _v11_group_message_event
 
@@ -17,7 +15,6 @@ from tests.utils import _v11_group_message_event
 async def test_update_all_plugin_basic_need_update(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -26,7 +23,6 @@ async def test_update_all_plugin_basic_need_update(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
@@ -63,16 +59,12 @@ async def test_update_all_plugin_basic_need_update(
             result=None,
             bot=bot,
         )
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
-    assert mocked_api["search_image_plugin_file_init_commit"].called
     assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file()
 
 
 async def test_update_all_plugin_basic_is_new(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -81,14 +73,13 @@ async def test_update_all_plugin_basic_is_new(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
     mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.StoreManager.get_loaded_plugins",
-        return_value=[("search_image", "0.1")],
+        return_value=[("search_image", "0.2")],
     )
 
     async with app.test_matcher(_matcher) as ctx:
@@ -116,5 +107,3 @@ async def test_update_all_plugin_basic_is_new(
             result=None,
             bot=bot,
         )
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called

tests/builtin_plugins/plugin_store/test_update_plugin.py

@@ -9,7 +9,6 @@ from nonebug import App
 from pytest_mock import MockerFixture
 from respx import MockRouter
 
-from tests.builtin_plugins.plugin_store.utils import init_mocked_api
 from tests.config import BotId, GroupId, MessageId, UserId
 from tests.utils import _v11_group_message_event
 
@@ -17,7 +16,6 @@ from tests.utils import _v11_group_message_event
 async def test_update_plugin_basic_need_update(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -26,7 +24,6 @@ async def test_update_plugin_basic_need_update(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mock_base_path = mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
@@ -63,16 +60,12 @@ async def test_update_plugin_basic_need_update(
             result=None,
             bot=bot,
         )
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
-    assert mocked_api["search_image_plugin_file_init_commit"].called
     assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file()
 
 
 async def test_update_plugin_basic_is_new(
     app: App,
     mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
     tmp_path: Path,
 ) -> None:
@@ -81,14 +74,13 @@ async def test_update_plugin_basic_is_new(
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.BASE_PATH",
         new=tmp_path / "zhenxun",
     )
     mocker.patch(
         "zhenxun.builtin_plugins.plugin_store.data_source.StoreManager.get_loaded_plugins",
-        return_value=[("search_image", "0.1")],
+        return_value=[("search_image", "0.2")],
     )
 
     plugin_id = 1
@@ -118,23 +110,17 @@ async def test_update_plugin_basic_is_new(
             result=None,
             bot=bot,
         )
-    assert mocked_api["basic_plugins"].called
-    assert mocked_api["extra_plugins"].called
 
 
 async def test_plugin_not_exist_update(
     app: App,
-    mocker: MockerFixture,
-    mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件不存在，更新插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     plugin_id = -1
 
     async with app.test_matcher(_matcher) as ctx:
@@ -158,7 +144,7 @@ async def test_plugin_not_exist_update(
         )
         ctx.should_call_send(
             event=event,
-            message=Message(message="插件ID不存在..."),
+            message=Message(message="更新插件 Id: -1 失败 e: 插件ID不存在..."),
             result=None,
             bot=bot,
         )
@@ -166,17 +152,14 @@ async def test_plugin_not_exist_update(
 
 async def test_update_plugin_not_install(
     app: App,
-    mocker: MockerFixture,
     mocked_api: MockRouter,
     create_bot: Callable,
-    tmp_path: Path,
 ) -> None:
     """
     测试插件不存在，更新插件
     """
     from zhenxun.builtin_plugins.plugin_store import _matcher
 
-    init_mocked_api(mocked_api=mocked_api)
     plugin_id = 1
 
     async with app.test_matcher(_matcher) as ctx:
@@ -200,7 +183,9 @@ async def test_update_plugin_not_install(
         )
         ctx.should_call_send(
             event=event,
-            message=Message(message="插件 识图 未安装，无法更新"),
+            message=Message(
+                message="更新插件 Id: 1 失败 e: 插件 识图 未安装，无法更新"
+            ),
             result=None,
             bot=bot,
         )

tests/builtin_plugins/plugin_store/utils.py

@@ -1,147 +0,0 @@
-# ruff: noqa: ASYNC230
-
-from pathlib import Path
-
-from respx import MockRouter
-
-from tests.utils import get_content_bytes as _get_content_bytes
-from tests.utils import get_response_json as _get_response_json
-
-
-def get_response_json(file: str) -> dict:
-    return _get_response_json(Path() / "plugin_store", file=file)
-
-
-def get_content_bytes(file: str) -> bytes:
-    return _get_content_bytes(Path() / "plugin_store", file)
-
-
-def init_mocked_api(mocked_api: MockRouter) -> None:
-    # metadata
-    mocked_api.get(
-        "https://data.jsdelivr.com/v1/packages/gh/zhenxun-org/zhenxun_bot_plugins@main",
-        name="zhenxun_bot_plugins_metadata",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_metadata.json"))
-    mocked_api.get(
-        "https://data.jsdelivr.com/v1/packages/gh/xuanerwa/zhenxun_github_sub@main",
-        name="zhenxun_github_sub_metadata",
-    ).respond(json=get_response_json("zhenxun_github_sub_metadata.json"))
-    mocked_api.get(
-        "https://data.jsdelivr.com/v1/packages/gh/zhenxun-org/zhenxun_bot_plugins@b101fbc",
-        name="zhenxun_bot_plugins_metadata_commit",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_metadata.json"))
-    mocked_api.get(
-        "https://data.jsdelivr.com/v1/packages/gh/xuanerwa/zhenxun_github_sub@f524632f78d27f9893beebdf709e0e7885cd08f1",
-        name="zhenxun_github_sub_metadata_commit",
-    ).respond(json=get_response_json("zhenxun_github_sub_metadata.json"))
-
-    # tree
-    mocked_api.get(
-        "https://api.github.com/repos/zhenxun-org/zhenxun_bot_plugins/git/trees/main?recursive=1",
-        name="zhenxun_bot_plugins_tree",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_tree.json"))
-    mocked_api.get(
-        "https://api.github.com/repos/xuanerwa/zhenxun_github_sub/git/trees/main?recursive=1",
-        name="zhenxun_github_sub_tree",
-    ).respond(json=get_response_json("zhenxun_github_sub_tree.json"))
-    mocked_api.get(
-        "https://api.github.com/repos/zhenxun-org/zhenxun_bot_plugins/git/trees/b101fbc?recursive=1",
-        name="zhenxun_bot_plugins_tree_commit",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_tree.json"))
-    mocked_api.get(
-        "https://api.github.com/repos/xuanerwa/zhenxun_github_sub/git/trees/f524632f78d27f9893beebdf709e0e7885cd08f1?recursive=1",
-        name="zhenxun_github_sub_tree_commit",
-    ).respond(json=get_response_json("zhenxun_github_sub_tree.json"))
-
-    mocked_api.head(
-        "https://raw.githubusercontent.com/",
-        name="head_raw",
-    ).respond(200, text="")
-
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/b101fbc/plugins.json",
-        name="basic_plugins",
-    ).respond(json=get_response_json("basic_plugins.json"))
-    mocked_api.get(
-        "https://cdn.jsdelivr.net/gh/zhenxun-org/zhenxun_bot_plugins@b101fbc/plugins.json",
-        name="basic_plugins_jsdelivr",
-    ).respond(200, json=get_response_json("basic_plugins.json"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/main/plugins.json",
-        name="basic_plugins_no_commit",
-    ).respond(json=get_response_json("basic_plugins.json"))
-    mocked_api.get(
-        "https://cdn.jsdelivr.net/gh/zhenxun-org/zhenxun_bot_plugins@main/plugins.json",
-        name="basic_plugins_jsdelivr_no_commit",
-    ).respond(200, json=get_response_json("basic_plugins.json"))
-
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins_index/2ed61284873c526802752b12a3fd3b5e1a59d948/plugins.json",
-        name="extra_plugins",
-    ).respond(200, json=get_response_json("extra_plugins.json"))
-    mocked_api.get(
-        "https://cdn.jsdelivr.net/gh/zhenxun-org/zhenxun_bot_plugins_index@2ed61284873c526802752b12a3fd3b5e1a59d948/plugins.json",
-        name="extra_plugins_jsdelivr",
-    ).respond(200, json=get_response_json("extra_plugins.json"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins_index/index/plugins.json",
-        name="extra_plugins_no_commit",
-    ).respond(200, json=get_response_json("extra_plugins.json"))
-    mocked_api.get(
-        "https://cdn.jsdelivr.net/gh/zhenxun-org/zhenxun_bot_plugins_index@index/plugins.json",
-        name="extra_plugins_jsdelivr_no_commit",
-    ).respond(200, json=get_response_json("extra_plugins.json"))
-
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/main/plugins/search_image/__init__.py",
-        name="search_image_plugin_file_init",
-    ).respond(content=get_content_bytes("search_image.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/b101fbc/plugins/search_image/__init__.py",
-        name="search_image_plugin_file_init_commit",
-    ).respond(content=get_content_bytes("search_image.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/main/plugins/alapi/jitang.py",
-        name="jitang_plugin_file",
-    ).respond(content=get_content_bytes("jitang.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/b101fbc/plugins/alapi/jitang.py",
-        name="jitang_plugin_file_commit",
-    ).respond(content=get_content_bytes("jitang.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/xuanerwa/zhenxun_github_sub/main/github_sub/__init__.py",
-        name="github_sub_plugin_file_init",
-    ).respond(content=get_content_bytes("github_sub.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/xuanerwa/zhenxun_github_sub/f524632f78d27f9893beebdf709e0e7885cd08f1/github_sub/__init__.py",
-        name="github_sub_plugin_file_init_commit",
-    ).respond(content=get_content_bytes("github_sub.py"))
-    mocked_api.get(
-        "https://raw.githubusercontent.com/zhenxun-org/zhenxun_bot_plugins/b101fbc/plugins/bilibili_sub/__init__.py",
-        name="bilibili_sub_plugin_file_init",
-    ).respond(content=get_content_bytes("bilibili_sub.py"))
-
-    mocked_api.get(
-        "https://api.github.com/repos/zhenxun-org/zhenxun_bot_plugins/commits/main",
-        name="zhenxun_bot_plugins_commit",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_commit.json"))
-    mocked_api.get(
-        "https://git-api.zhenxun.org/repos/zhenxun-org/zhenxun_bot_plugins/commits/main",
-        name="zhenxun_bot_plugins_commit_proxy",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_commit.json"))
-    mocked_api.get(
-        "https://api.github.com/repos/zhenxun-org/zhenxun_bot_plugins_index/commits/index",
-        name="zhenxun_bot_plugins_index_commit",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_index_commit.json"))
-    mocked_api.get(
-        "https://git-api.zhenxun.org/repos/zhenxun-org/zhenxun_bot_plugins_index/commits/index",
-        name="zhenxun_bot_plugins_index_commit_proxy",
-    ).respond(json=get_response_json("zhenxun_bot_plugins_index_commit.json"))
-    mocked_api.get(
-        "https://api.github.com/repos/xuanerwa/zhenxun_github_sub/commits/main",
-        name="zhenxun_github_sub_commit",
-    ).respond(json=get_response_json("zhenxun_github_sub_commit.json"))
-    mocked_api.get(
-        "https://git-api.zhenxun.org/repos/xuanerwa/zhenxun_github_sub/commits/main",
-        name="zhenxun_github_sub_commit_proxy",
-    ).respond(json=get_response_json("zhenxun_github_sub_commit.json"))