Vinay Sajip <vinay_sa...@yahoo.co.uk> added the comment:

The AssertionError in Brian's initial post indicates that the test is too 
restrictive. Though the discussion here has talked about removing the popups 
altogether, the test passes on Windows with a small change: instead of 
assertEqual(), I used

    self.assertTrue(err.rstrip().startswith(
                    b'Fatal Python error:'
                    b' PyThreadState_Get: no current thread'))

Patch attached. As this seems to be a sufficiently specific test, but not too 
specific, I'd like to apply this patch soon, unless someone thinks the test 
needs to stay exactly like it is.

----------
nosy: +vinay.sajip
Added file: http://bugs.python.org/file25457/test_capi.diff

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue9116>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to