Skip to content

gh-143959 Make _datetime,_types optional for test_types #144042

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
youknowone wants to merge 2 commits into
base: main
Choose a base branch
from
+28 −9
Open

gh-143959 Make _datetime,_types optional for test_types #144042

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a3b9317
gh-143959 Make _datetime optional for test_types
youknowone Jan 19, 2026
e17be43
gh-143959: Make _types optional for test_types
youknowone Jan 19, 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
37 changes: 28 additions & 9 deletions Lib/test/test_types.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
import collections.abc
from collections import namedtuple, UserDict
import copy
import _datetime
import gc
import inspect
import pickle
Expand All @@ -22,6 +21,14 @@
import weakref
import typing
import re
try:
import _datetime
except ModuleNotFoundError:
_datetime = None
try:
import _types
except ModuleNotFoundError:
_types = None

c_types = import_fresh_module('types', fresh=['_types'])
py_types = import_fresh_module('types', blocked=['_types'])
Expand All @@ -40,15 +47,10 @@ def clear_typing_caches():

class TypesTests(unittest.TestCase):

def test_names(self):
Copy link
Member

@serhiy-storchaka serhiy-storchaka Jan 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why these changes were needed?

Copy link
Contributor Author

@youknowone youknowone Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Current test only runs meaningful when _types optional module is implemented.
Usually Python unittest includes conditional handling to run tests under proper environment, but this test was exposed without it.
Adding checking _types split the test and verify the behavior regardless the environment has _types or not.

Copy link
Member

@serhiy-storchaka serhiy-storchaka Jan 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How did you test this?

Note that import_fresh_module() returns None when one of the "fresh" modules can not be imported.

def check_types_names(self, module_types, *, c_extension=False):
c_only_names = {'CapsuleType'}
ignored = {'new_class', 'resolve_bases', 'prepare_class',
'get_original_bases', 'DynamicClassAttribute', 'coroutine'}

for name in c_types.__all__:
if name not in c_only_names | ignored:
self.assertIs(getattr(c_types, name), getattr(py_types, name))

all_names = ignored | {
'AsyncGeneratorType', 'BuiltinFunctionType', 'BuiltinMethodType',
'CapsuleType', 'CellType', 'ClassMethodDescriptorType', 'CodeType',
Expand All @@ -60,8 +62,24 @@ def test_names(self):
'ModuleType', 'NoneType', 'NotImplementedType', 'SimpleNamespace',
'TracebackType', 'UnionType', 'WrapperDescriptorType',
}
self.assertEqual(all_names, set(c_types.__all__))
self.assertEqual(all_names - c_only_names, set(py_types.__all__))
expected = all_names if c_extension else all_names - c_only_names
self.assertEqual(expected, set(module_types.__all__))

@unittest.skipUnless(_types, "requires C _types module")
def test_c_types_names(self):
self.check_types_names(c_types, c_extension=True)

def test_py_types_names(self):
self.check_types_names(py_types, c_extension=False)

@unittest.skipUnless(_types, "requires C _types module")
def test_types_names_consistency(self):
c_only_names = {'CapsuleType'}
ignored = {'new_class', 'resolve_bases', 'prepare_class',
'get_original_bases', 'DynamicClassAttribute', 'coroutine'}
for name in c_types.__all__:
if name not in c_only_names | ignored:
self.assertIs(getattr(c_types, name), getattr(py_types, name))

def test_truth_values(self):
if None: self.fail('None is true instead of false')
Expand Down Expand Up @@ -691,6 +709,7 @@ def test_traceback_and_frame_types(self):
self.assertIsInstance(exc.__traceback__, types.TracebackType)
self.assertIsInstance(exc.__traceback__.tb_frame, types.FrameType)

@unittest.skipUnless(_datetime, "requires _datetime module")
def test_capsule_type(self):
self.assertIsInstance(_datetime.datetime_CAPI, types.CapsuleType)

Expand Down
Loading