You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
./python.exe -Werror -m test test_warnings fails, which feels sort-of ironic:
~/dev/cpython (main)⚡ % ./python.exe -We -m test test_warnings
Using random seed: 673638140
Raised RLIMIT_NOFILE: 256 -> 1024
0:00:00 load avg: 2.83 Run 1 test sequentially in a single process
0:00:00 load avg: 2.83 [1/1] test_warnings
test test_warnings failed -- Traceback (most recent call last):
File "/Users/alexw/dev/cpython/Lib/test/test_warnings/__init__.py", line 904, in test_issue31285
wmod.warn_explicit(
~~~~~~~~~~~~~~~~~~^'foo', UserWarning, 'bar', 1,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
module_globals={'__loader__': get_bad_loader(42),
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^'__name__': 'foobar'})
^^^^^^^^^^^^^^^^^^^^^^
File "<frozen importlib._bootstrap_external>", line 934, in _bless_my_loaderDeprecationWarning: Module globals is missing a __spec__.loader
test_warnings failed (1 error)
== Tests result: FAILURE ==
1 test failed:
test_warnings
Total duration: 482 ms
Total tests: run=146
Total test files: run=1/1 failed=1
Result: FAILURE
Well, I spent some time tracking the error back to '/**/cpython/cpython/Lib/importlib/_bootstrap_external.py' at "_bless_my_loader," where it checks if the module globals contain loader or spec. At line 935, it raises a warning:
_warnings.warn(
'Module globals is missing a __spec__.loader',
DeprecationWarning
)
Since test_warning tests warnings, it doesn't make sense to run it with --Werror, which treats warnings as errors.
so a possible solution is to check if there is -Werror in args when runing warning tests
Activity
x0rw commentedon Jul 23, 2024
Well, I spent some time tracking the error back to '/**/cpython/cpython/Lib/importlib/_bootstrap_external.py' at "_bless_my_loader," where it checks if the module globals contain loader or spec. At line 935, it raises a warning:
Since test_warning tests warnings, it doesn't make sense to run it with --Werror, which treats warnings as errors.
so a possible solution is to check if there is -Werror in args when runing warning tests
pythongh-122191: Fix test_warnings failure if run with -Werror
serhiy-storchaka commentedon Jul 24, 2024
In future, warnings will be turned into errors. Unless we are planning to remove these tests, the warnings should not be ignored.
gh-122191: Fix test_warnings failure if run with -Werror (GH-122222)
pythongh-122191: Fix test_warnings failure if run with -Werror (pytho…
[3.12] pythongh-122191: Fix test_warnings failure if run with -Werror (…
[3.12] gh-122191: Fix test_warnings failure if run with -Werror (GH-1…
[3.13] gh-122191: Fix test_warnings failure if run with -Werror (GH-1…