From 2760183d1a03326472ba3f1ceea5d9c0f3fccfb0 Mon Sep 17 00:00:00 2001 From: molanp <104612722+molanp@users.noreply.github.com> Date: Wed, 13 Aug 2025 14:38:05 +0800 Subject: [PATCH] =?UTF-8?q?test:=20=E4=BD=BF=E7=94=A8=20xfail=20=E6=9B=BF?= =?UTF-8?q?=E4=BB=A3=20skip=20=E6=A0=87=E8=AE=B0=E6=B5=8B=E8=AF=95?= =?UTF-8?q?=E7=94=A8=E4=BE=8B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 将多个测试用例中的 @pytest.mark.skip 标记替换为 @pytest.mark.xfail - 这一变更可以更准确地反映测试用例的预期行为 - 主要涉及 auto_update、plugin_store 相关的测试文件 --- .../builtin_plugins/auto_update/test_check_update.py | 4 ++-- .../builtin_plugins/plugin_store/test_add_plugin.py | 12 ++++++------ .../plugin_store/test_remove_plugin.py | 6 +++--- .../plugin_store/test_search_plugin.py | 6 +++--- .../plugin_store/test_update_all_plugin.py | 4 ++-- .../plugin_store/test_update_plugin.py | 8 ++++---- 6 files changed, 20 insertions(+), 20 deletions(-) diff --git a/tests/builtin_plugins/auto_update/test_check_update.py b/tests/builtin_plugins/auto_update/test_check_update.py index acb2b7c4..831a3c9b 100644 --- a/tests/builtin_plugins/auto_update/test_check_update.py +++ b/tests/builtin_plugins/auto_update/test_check_update.py @@ -225,7 +225,7 @@ def init_mocker_path(mocker: MockerFixture, tmp_path: Path): ) -@pytest.mark.skip("不会修") +@pytest.mark.xfail async def test_check_update_release( app: App, mocker: MockerFixture, @@ -322,7 +322,7 @@ async def test_check_update_release( assert (mock_backup_path / folder).exists() -@pytest.mark.skip("不会修") +@pytest.mark.xfail async def test_check_update_main( app: App, mocker: MockerFixture, diff --git a/tests/builtin_plugins/plugin_store/test_add_plugin.py b/tests/builtin_plugins/plugin_store/test_add_plugin.py index 0d7b8aae..6c35cbb2 100644 --- a/tests/builtin_plugins/plugin_store/test_add_plugin.py +++ b/tests/builtin_plugins/plugin_store/test_add_plugin.py @@ -15,7 +15,7 @@ from tests.utils import _v11_group_message_event test_path = Path(__file__).parent.parent.parent -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_add_plugin_basic( app: App, mocker: MockerFixture, @@ -62,7 +62,7 @@ async def test_add_plugin_basic( assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_add_plugin_basic_commit_version( app: App, mocker: MockerFixture, @@ -109,7 +109,7 @@ async def test_add_plugin_basic_commit_version( assert (mock_base_path / "plugins" / "bilibili_sub" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_add_plugin_basic_is_not_dir( app: App, mocker: MockerFixture, @@ -156,7 +156,7 @@ async def test_add_plugin_basic_is_not_dir( assert (mock_base_path / "plugins" / "jitang.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_add_plugin_extra( app: App, mocker: MockerFixture, @@ -203,7 +203,7 @@ async def test_add_plugin_extra( assert (mock_base_path / "plugins" / "github_sub" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_plugin_not_exist_add( app: App, create_bot: Callable, @@ -242,7 +242,7 @@ async def test_plugin_not_exist_add( ) -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_add_plugin_exist( app: App, mocker: MockerFixture, diff --git a/tests/builtin_plugins/plugin_store/test_remove_plugin.py b/tests/builtin_plugins/plugin_store/test_remove_plugin.py index 58c61891..e9bb6822 100644 --- a/tests/builtin_plugins/plugin_store/test_remove_plugin.py +++ b/tests/builtin_plugins/plugin_store/test_remove_plugin.py @@ -15,7 +15,7 @@ from tests.config import BotId, GroupId, MessageId, UserId from tests.utils import _v11_group_message_event -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_remove_plugin( app: App, mocker: MockerFixture, @@ -62,7 +62,7 @@ async def test_remove_plugin( assert not (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_plugin_not_exist_remove( app: App, create_bot: Callable, @@ -95,7 +95,7 @@ async def test_plugin_not_exist_remove( ) -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_remove_plugin_not_install( app: App, mocker: MockerFixture, diff --git a/tests/builtin_plugins/plugin_store/test_search_plugin.py b/tests/builtin_plugins/plugin_store/test_search_plugin.py index 43699f41..f8bde078 100644 --- a/tests/builtin_plugins/plugin_store/test_search_plugin.py +++ b/tests/builtin_plugins/plugin_store/test_search_plugin.py @@ -12,7 +12,7 @@ from tests.config import BotId, GroupId, MessageId, UserId from tests.utils import _v11_group_message_event -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_search_plugin_name( app: App, mocker: MockerFixture, @@ -54,7 +54,7 @@ async def test_search_plugin_name( mock_build_message_return.send.assert_awaited_once() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_search_plugin_author( app: App, mocker: MockerFixture, @@ -96,7 +96,7 @@ async def test_search_plugin_author( mock_build_message_return.send.assert_awaited_once() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_plugin_not_exist_search( app: App, create_bot: Callable, diff --git a/tests/builtin_plugins/plugin_store/test_update_all_plugin.py b/tests/builtin_plugins/plugin_store/test_update_all_plugin.py index 31ed7a42..6fb5ef9d 100644 --- a/tests/builtin_plugins/plugin_store/test_update_all_plugin.py +++ b/tests/builtin_plugins/plugin_store/test_update_all_plugin.py @@ -13,7 +13,7 @@ from tests.config import BotId, GroupId, MessageId, UserId from tests.utils import _v11_group_message_event -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_update_all_plugin_basic_need_update( app: App, mocker: MockerFixture, @@ -64,7 +64,7 @@ async def test_update_all_plugin_basic_need_update( assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_update_all_plugin_basic_is_new( app: App, mocker: MockerFixture, diff --git a/tests/builtin_plugins/plugin_store/test_update_plugin.py b/tests/builtin_plugins/plugin_store/test_update_plugin.py index 44f64546..dca6a6ae 100644 --- a/tests/builtin_plugins/plugin_store/test_update_plugin.py +++ b/tests/builtin_plugins/plugin_store/test_update_plugin.py @@ -13,7 +13,7 @@ from tests.config import BotId, GroupId, MessageId, UserId from tests.utils import _v11_group_message_event -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_update_plugin_basic_need_update( app: App, mocker: MockerFixture, @@ -64,7 +64,7 @@ async def test_update_plugin_basic_need_update( assert (mock_base_path / "plugins" / "search_image" / "__init__.py").is_file() -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_update_plugin_basic_is_new( app: App, mocker: MockerFixture, @@ -114,7 +114,7 @@ async def test_update_plugin_basic_is_new( ) -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_plugin_not_exist_update( app: App, create_bot: Callable, @@ -153,7 +153,7 @@ async def test_plugin_not_exist_update( ) -@pytest.mark.skip("修不好") +@pytest.mark.xfail async def test_update_plugin_not_install( app: App, create_bot: Callable,