Skip to content

Commit 332f12b

Browse files
committed
fixup! Improve accuaracy of unit tests: Count num of copies/moves
1 parent 5a15689 commit 332f12b

File tree

1 file changed

+13
-13
lines changed

1 file changed

+13
-13
lines changed

tests/test_class_sh_basic.py

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -17,15 +17,15 @@ def test_atyp_constructors():
1717

1818

1919
def check_regex(expected, actual):
20-
result = re.fullmatch(expected, actual)
20+
result = re.match(expected + "$", actual)
2121
if result is None:
2222
pytest.fail("expected: '{}' != actual: '{}'".format(expected, actual))
2323

2424

2525
@pytest.mark.parametrize(
2626
"rtrn_f, expected",
2727
[
28-
(m.rtrn_valu, "rtrn_valu(_MvCtor){1}"),
28+
(m.rtrn_valu, "rtrn_valu(_MvCtor){1,3}"),
2929
(m.rtrn_rref, "rtrn_rref(_MvCtor){1}"),
3030
(m.rtrn_cref, "rtrn_cref_CpCtor"),
3131
(m.rtrn_mref, "rtrn_mref"),
@@ -46,15 +46,15 @@ def test_cast(rtrn_f, expected):
4646
@pytest.mark.parametrize(
4747
"pass_f, mtxt, expected",
4848
[
49-
(m.pass_valu, "Valu", "pass_valu:Valu(_MvCtor){1}_CpCtor"),
50-
(m.pass_cref, "Cref", "pass_cref:Cref(_MvCtor){1}"),
51-
(m.pass_mref, "Mref", "pass_mref:Mref(_MvCtor){1}"),
52-
(m.pass_cptr, "Cptr", "pass_cptr:Cptr(_MvCtor){1}"),
53-
(m.pass_mptr, "Mptr", "pass_mptr:Mptr(_MvCtor){1}"),
54-
(m.pass_shmp, "Shmp", "pass_shmp:Shmp(_MvCtor){1}"),
55-
(m.pass_shcp, "Shcp", "pass_shcp:Shcp(_MvCtor){1}"),
56-
(m.pass_uqmp, "Uqmp", "pass_uqmp:Uqmp(_MvCtor){1}"),
57-
(m.pass_uqcp, "Uqcp", "pass_uqcp:Uqcp(_MvCtor){1}"),
49+
(m.pass_valu, "Valu", "pass_valu:Valu(_MvCtor){1,2}_CpCtor"),
50+
(m.pass_cref, "Cref", "pass_cref:Cref(_MvCtor){1,2}"),
51+
(m.pass_mref, "Mref", "pass_mref:Mref(_MvCtor){1,2}"),
52+
(m.pass_cptr, "Cptr", "pass_cptr:Cptr(_MvCtor){1,2}"),
53+
(m.pass_mptr, "Mptr", "pass_mptr:Mptr(_MvCtor){1,2}"),
54+
(m.pass_shmp, "Shmp", "pass_shmp:Shmp(_MvCtor){1,2}"),
55+
(m.pass_shcp, "Shcp", "pass_shcp:Shcp(_MvCtor){1,2}"),
56+
(m.pass_uqmp, "Uqmp", "pass_uqmp:Uqmp(_MvCtor){1,2}"),
57+
(m.pass_uqcp, "Uqcp", "pass_uqcp:Uqcp(_MvCtor){1,2}"),
5858
],
5959
)
6060
def test_load_with_mtxt(pass_f, mtxt, expected):
@@ -117,7 +117,7 @@ def test_unique_ptr_roundtrip(num_round_trips=1000):
117117
for _ in range(num_round_trips):
118118
id_orig = id(recycled)
119119
recycled = m.unique_ptr_roundtrip(recycled)
120-
check_regex("passenger(_MvCtor){1}", m.get_mtxt(recycled))
120+
check_regex("passenger(_MvCtor){1,2}", m.get_mtxt(recycled))
121121
id_rtrn = id(recycled)
122122
# Ensure the returned object is a different Python instance.
123123
assert id_rtrn != id_orig
@@ -138,7 +138,7 @@ def test_unique_ptr_consumer_roundtrip(pass_f, rtrn_f, moved_out, moved_in):
138138
recycled = m.atyp("passenger")
139139
mtxt_orig = m.get_mtxt(recycled)
140140
ptr_orig = m.get_ptr(recycled)
141-
check_regex("passenger_(MvCtor){1}", mtxt_orig)
141+
check_regex("passenger_(MvCtor){1,2}", mtxt_orig)
142142

143143
pass_f(c, recycled) # pass object to C++ consumer c
144144
if moved_out: # if moved (always), ensure it is flagged as disowned

0 commit comments

Comments
 (0)