This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Title: Tests fail instead of skip
Type: Stage:
Components: Build Versions: Python 2.4
Status: closed Resolution: fixed
Dependencies: Superseder:
Assigned To: Nosy List: brett.cannon, dvch, rhettinger
Priority: normal Keywords:

Created on 2004-12-11 23:06 by dvch, last changed 2022-04-11 14:56 by admin. This issue is now closed.

File name Uploaded Description Edit
bug-rep01 dvch, 2004-12-18 11:48 test log
deci.diff rhettinger, 2004-12-18 14:34 Removed decimal's threading dependency
Messages (8)
msg23653 - (view) Author: Detlef Vollmann (dvch) Date: 2004-12-11 23:06
On Linux, with threads disabled (--with-threads=no),
the tests test_imp, test_list and test_userlist
fail instead of skipping.
msg23654 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2004-12-18 04:33
Logged In: YES 

Can you run these tests individually and save the output to see how they 
are failing?
msg23655 - (view) Author: Detlef Vollmann (dvch) Date: 2004-12-18 11:48
Logged In: YES 

Sorry, from the short output of 'make test' the problem
seemed obvious:
test test_imp failed -- expected imp.lock_held() to be True
ImportError: No module named thread
ImportError: No module named thread

But after running the tests manually, I noticed that the latter
two tests are complete test packs.
The complete output is attached.

msg23656 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2004-12-18 14:34
Logged In: YES 

Please try out the attached patch and let me know if the
full test_suite passes.
msg23657 - (view) Author: Detlef Vollmann (dvch) Date: 2004-12-18 16:04
Logged In: YES 

test_list and test_userlist succeed now, great, thanks.
test_imp still fails as before :-(

msg23658 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2004-12-18 19:41
Logged In: YES 

Applied the decimal fixup.  It is no longer dependent on

See Lib/ 1.32 and

Am unassigning myself for the rest.  It's not an especially
high priority to teach about tests that should
be skipped when there are no threads.
msg23659 - (view) Author: Brett Cannon (brett.cannon) * (Python committer) Date: 2004-12-18 21:10
Logged In: YES 

test_imp is now skipped on platforms that don't have 
threading.  Covered by rev. 1.5 in HEAD, rev. for 2.4, and rev. for 2.3 .

This takes care of all the reported tests.
msg23660 - (view) Author: Detlef Vollmann (dvch) Date: 2004-12-19 00:22
Logged In: YES 

I just tested the patch and it works fine:
it correcly skips test_imp :-)

Date User Action Args
2022-04-11 14:56:08adminsetgithub: 41311
2004-12-11 23:06:41dvchcreate