New submission from Zachary Ware:

Attached is a patch that aims to clean up the ctypes tests a bit, namely by 
removing the custom resource management (which conflicts with regrtest), the 
custom test discovery (which is better left to unittest), and the custom test 
running (which is better covered by unittest and regrtest).  The one thing I'm 
not entirely confident about removing is the custom refleak testing, but it 
does not seem to work correctly in 3.x anyway (though in 2.7, the custom 
refleak hunter reports "leaks" that the regrtest refleak hunter does not).

There were only a few uses of the custom resource management, all of which were 
replaced or removed.  test_SEH in test_win32 used "requires('SEH')", but that 
test should now be sufficiently guarded with unittest skip decorators (only 
trying the test on Windows, with Python built in Release configuration by 
MSVC).  test_PyLong_Long in test_python_api used "requires('refcount')", but 
that should be covered by the @support.refcount_test decorator (added long 
after the 'requires' call).  Two instances of "is_resource_enabled('printing')" 
were replaced by "if test.support.verbose".

The same number of tests run (all successfully) on Windows, I have not yet 
tested on any other platforms.

----------
components: Tests, ctypes
files: ctypes.test-cleanup.diff
keywords: patch
messages: 223891
nosy: amaury.forgeotdarc, belopolsky, meador.inge, serhiy.storchaka, zach.ware
priority: normal
severity: normal
stage: patch review
status: open
title: Clean up ctypes.test, use unittest test discovery
type: enhancement
versions: Python 3.5
Added file: http://bugs.python.org/file36073/ctypes.test-cleanup.diff

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

Reply via email to