Fix: bot.call_api 在被 called api hook mock 后应该忽略 exception

[Ailitonia]

fix #2373

[codecov]

Codecov Report

Merging #2374 (7e6035f) into master (ede1a20) will increase coverage by 0.84%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2374      +/-   ##
==========================================
+ Coverage   90.81%   91.65%   +0.84%     
==========================================
  Files          46       46              
  Lines        3428     3429       +1     
==========================================
+ Hits         3113     3143      +30     
+ Misses        315      286      -29     

Flag	Coverage Δ
unittests	91.65% <100.00%> (+0.84%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed	Coverage Δ
nonebot/internal/adapter/bot.py	89.39% <100.00%> (+43.24%)	⬆️

... and 1 file with indirect coverage changes

[github-actions]

🚀 Deployed on https://deploy-preview-2374--nonebot2.netlify.app

[yanyongyu]

请补充一下测试用例

[Ailitonia]

请补充一下测试用例

nonebug 似乎没有检验 call_api 返回后的 result, 还是这样写有问题?

@pytest.mark.asyncio
async def test_on_called_api_mocked(app: App, monkeypatch: pytest.MonkeyPatch):
    runned = False

    async def test_on_called_api_hook(
        bot: Bot,
        exception: Optional[Exception],
        api: str,
        data: Dict[str, Any],
        result: Any,
    ):
        nonlocal runned
        runned = True
        raise MockApiException(result={"mocked": True})

    async def test_call_api_handler(bot: Bot):
        await bot.call_api("test", test=True)

    with app.provider.context({}):
        matcher = on_message()
        matcher.handle()(test_call_api_handler)

        async with app.test_matcher(matcher) as ctx:
            bot = ctx.create_bot()
            bot.on_called_api(test_on_called_api_hook)
            event = make_fake_event()()
            ctx.receive_event(bot, event)
            ctx.should_call_api("test", {"test": True}, {"mocked": True})

    assert runned, "on_called_api_hook should runned"

nonebug 这里对 call_api 返回的 result 没有做任何检验
https://github.com/nonebot/nonebug/blob/6345a32957ab570f60ff3c242e48cfc8e06a7d2f/nonebug/mixin/call_api/__init__.py#L163-L186