unittest.mock.patch.object holds references for too long

Bug report

Bug description:

import sys
from unittest import mock
import json

def main():
    obj = 'asdfa' * 2000

    print("BEGIN", sys.getrefcount(obj))
    for i in range(10):
        with mock.patch.object(json, 'dumps', return_value=3):
            from json import dumps
            dumps(obj)
    del dumps
    print("END", sys.getrefcount(obj))
    import gc; gc.collect()
    print("AFTERGC", sys.getrefcount(obj))


if __name__ == '__main__':
    main()

The code prints:

BEGIN 2
END 12
AFTERGC 2

Because it is the user's data that is leaked, the leaked object could be huge and could cause non-trivial problems.

CPython versions tested on:

3.13

Operating systems tested on:

Linux

Linked PRs

• gh-139937

unittest.mock.patch.object と提供された再現コードから始め、その後、現在の調査についてはリンク先の PR gh-139937 を読んでください。patch コンテキストを繰り返し使用した場合とガベージコレクションの前後で参照カウントを確認してください。コンテキストの終了後にユーザーのオブジェクトへの参照が残っていなければ完了です。