From a72e5f0c8fda411083a67d0881d350200fca26ee Mon Sep 17 00:00:00 2001 From: "Ralf W. Grosse-Kunstleve" Date: Fri, 15 Jan 2021 16:47:47 -0800 Subject: [PATCH] Inserting additional assert to ensure a returned unique_ptr is always a new Python instance. --- tests/test_classh_wip.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/test_classh_wip.py b/tests/test_classh_wip.py index 4f467b2bd..172e1d6ff 100644 --- a/tests/test_classh_wip.py +++ b/tests/test_classh_wip.py @@ -70,7 +70,13 @@ def test_pass_unique_ptr_disowns(pass_mpty, argm, rtrn): def test_unique_ptr_roundtrip(num_round_trips=1000): + # Multiple roundtrips to stress-test instance registration/deregistration. recycled = m.mpty("passenger") for _ in range(num_round_trips): + id_orig = id(recycled) recycled = m.unique_ptr_roundtrip(recycled) assert m.get_mtxt(recycled) == "passenger" + id_rtrn = id(recycled) + # Ensure the returned object is a different Python instance. + assert id_rtrn != id_orig + id_orig = id_rtrn