call_extra() drops arguments for partial callbacks
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
Start with the reproducer in the issue, which uses HookCaller.call_extra() with a functools.partial callback. The problem is in varnames(), which reads the partial's own __call__ instead of its remaining signature, so the bound arguments are never forwarded. Run the snippet and confirm it returns [("demo", 7)], then check the existing varnames() tests for plain functions and partials.
索引モデルが issue の本文から書いたものです。
説明
Passing a functools.partial callback to HookCaller.call_extra() drops the remaining hook arguments. A callback with a required argument raises TypeError; one with a default silently uses that default instead of the supplied value.
from functools import partial
import pluggy
hookspec = pluggy.HookspecMarker("example")
class Hooks:
@hookspec
def event(self, value):
pass
def on_event(context, value):
return context, value
pm = pluggy.PluginManager("example")
pm.add_hookspecs(Hooks)
callback = partial(on_event, "demo")
assert callback(7) == ("demo", 7)
print(pm.hook.event.call_extra([callback], {"value": 7}))
Expected: [("demo", 7)].
Actual: TypeError: on_event() missing 1 required positional argument: 'value'.
varnames() inspects the partial's built-in __call__ rather than its remaining signature, so no parameters are forwarded.
Reproduced on Python 3.12.16/Linux with main 3fc9e561ea7e72a95549673d79ebdaf4a9fae829.
- 主要言語
- Python
- スター
- 1.7k
- フォーク
- 200
- 平均マージ
- 10時間 4分
- マージ済み PR(30日)
- 13
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pytest-dev/pluggy のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 71/100
pytest-dev/pluggy#830 ·
メンテナーはふだん 1 日以内に返信
-
Plugin registry compares plugins by equality, not identity対応中かも @00200200 が 30 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
pytest-dev/pluggy#760 ·
メンテナーはふだん 1 日以内に返信
-
add_hookcall_monitoring undo breaks when called out of LIFO order対応中かも @AlvaroBalbin が 7 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
pytest-dev/pluggy#815 ·
メンテナーはふだん 1 日以内に返信
-
call_historic() memorizes the caller's kwargs mapping, so later mutation changes what late plugins receive対応中かも @RonnyPfannschmidt が 21 日前に担当しました。 オープン
pytest-dev/pluggy#808 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
deprecate passing None as a plugin instance再び着手できるかも このイシューのプルリクエストはマージされずにクローズされました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 30/100
pytest-dev/pluggy#790 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
pytest-dev/pluggy の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
FAIRmat-NFDI/pynxtools#861 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
NVIDIA/nsight-python#88 ·
メンテナーはふだん 1 日以内に返信
-
Criteria can pass when the deliverable is missing対応中かも @clementtisseau が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
harveyai/harvey-labs#217 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
priority/medium roadmap/v1.1.1 stakeholder/rapr type/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
databrickslabs/ontos#1420 ·
メンテナーはふだん 3 日以内に返信