zopefoundation / AccessControl / 27044389014

84%

master: 84%

Pull #165

github

Fix guarded_getattr ImplPython/ImplC assertion-key parity

The pure-Python ``guarded_getattr`` keyed the container assertion on
``type(value.__self__)`` while the C implementation keys on ``type(inst)``.
As a result, under ``PURE_PYTHON`` a bound method whose ``__self__`` is an
allow-listed simple type (``tuple``/``bytes``/``range``/...) took the
truthy-assertion short-circuit and was returned without calling
``validate()``, diverging from the C engine.

Key the assertion lookup on ``type(inst)`` to match ``cAccessControl``, and
add a regression test (red under ``PURE_PYTHON`` before the fix, green
after; the C engine was already correct).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

417 of 598 branches covered (69.73%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 2 files covered. (100.0%)

5075 of 5914 relevant lines covered (85.81%)

5.15 hits per line