Skip to content

gh-146238: add missing tests for 'e', 'Zf' and 'Zd' array type codes in test_buffer.py #149345

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
skirpichev wants to merge 5 commits into
base: main
Choose a base branch
from
+7 −3
Open

gh-146238: add missing tests for 'e', 'Zf' and 'Zd' array type codes in test_buffer.py #149345

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d00c981
gh-146238: add missing tests for 'e' type code in test_buffer.py
skirpichev May 4, 2026
6d1251c
Merge branch 'master' into amend-e-format
skirpichev May 5, 2026
b4569cb
+ Zf/d
skirpichev May 5, 2026
2e1c01b
skip Zd in test_array_alignment()
skirpichev May 5, 2026
b97670b
fix test
skirpichev May 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions Lib/test/test_buffer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@

try:
import _testcapi
import _testlimitedcapi
except ImportError:
_testcapi = None

Expand Down Expand Up @@ -143,7 +144,7 @@ def native_type_range(fmt):
# Format codes supported by array.array
ARRAY = NATIVE.copy()
for k in NATIVE:
if not k in "bBhHiIlLfd":
if not k in list("bBhHiIlLefd") + ['Zf', 'Zd']:
del ARRAY[k]

BYTEFMT = NATIVE.copy()
Expand Down Expand Up @@ -4495,8 +4496,11 @@ def test_bytearray_alignment(self):
def test_array_alignment(self):
# gh-140557: pointer alignment of buffers including empty allocation
# should match the maximum array alignment.
align = max(struct.calcsize(fmt) for fmt in ARRAY)
cases = [array.array(fmt) for fmt in ARRAY]
MAX_ALIGN = _testlimitedcapi.ALIGNOF_MAX_ALIGN_T
align = max(struct.calcsize(fmt) for fmt in ARRAY
if struct.calcsize(fmt) <= MAX_ALIGN)
cases = [array.array(fmt) for fmt in ARRAY
if struct.calcsize(fmt) <= MAX_ALIGN]
# Empty arrays
self.assertEqual(
[_testcapi.buffer_pointer_as_int(case) % align for case in cases],
Expand Down
Loading