Unittest.mock.patch - record result of wrapped function call
まだ誰も着手していません。
- 主要言語
- Python
- スター
- 77.2k
- フォーク
- 36k
- PR マージ指標
- PR 指標を取得中
説明
Feature or enhancement
Proposal:
I would like to access the return value of a function call wrapped with unittest.mock.patch, that I don't call myself directly.
Motivation
Let's say I have a framework that supports some "before" hooks to allow users to customize some data flowing through the framework.
One could write hook functions for such a framework and publish them.
When doing so, it would make integration tests easier if one could access the return value of a function wrapped with a MagicMock.
# framework.py
def init(before_hook):
... # register the update hook that is later used before doing some action to allow customization of the action
# my_plugin.py
def my_before_hook(some_data: dict) -> dict:
... #do something with some_data eg adding a key
some_data["foo"] = "bar"
return some_data # return the altered data
# test_my_plugin.py
import my_plugin
import framework
def test_my_before_hook():
with unittest.mock.patch("my_plugin.my_before_hook", wraps=my_plugin.my_before_hook) as m:
framework.init(before_hook=m)
... #do something that runs the framework
assert m.called
assert ... # assert m's return value has the key "foo" with the value "bar"
Unfortunately, currently the second assert statement is not really possible - at least I couldn't find anything in the docs, debugger, and LLMs just hallucinated on this one 😅.
Alternatives
There are two alternatives I have considered:
- Digging into the framework and finding a mock target - while this would work, it can be quite tedious, especially when the framework is big
- Wrapping the function in question manually - Definitely possible, but seems to introduce an extra step every time this is necessary and moves the assertion to a an odd place:
# test_my_plugin.py
import my_plugin
import framework
def foo(*args, **kwargs)
result = my_plug.my_before_hook(*args, **kwargs)
assert result["foo"] == "bar" # this only works if the framework doesn't handle AssertionErrors
return result
def test_my_before_hook():
with unittest.mock.patch("my_plugin.my_before_hook", wraps=foo) as m:
framework.init(before_hook=m)
... #do something that runs the framework
assert m.called
Proposed Solution
Record the return value of a "wraps" call and make it available on the call tuple.
I know this could be somewhat problematic due to backwards compatibility, but I'm not entirely sure.
I dove a little bit into the source code and don't think it will be too big of a change both in terms of code and functionality - please correct me if I'm wrong.
I'd be happy to contribute if this is enhancement is desired 🙂
Has this already been discussed elsewhere?
This is a minor feature, which does not need previous discussion elsewhere
Links to previous discussion of this feature:
No response
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、issueで説明されている unittest.mock.patch、MagicMock、wraps、および call tuple の既存の動作を読み、その後、関連する unittest.mock の実装とテストを調べます。現在の call API を壊さずに、ラップされた関数の戻り値を公開する方法を判断します。ラップされた呼び出しと互換性について動作が仕様化、実装され、テストでカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- python
- 領域
- testing-qa
- issue の種類
- 機能追加
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100