Author: russellm
Date: 2011-09-10 11:44:33 -0700 (Sat, 10 Sep 2011)
New Revision: 16784
Modified:
django/trunk/tests/modeltests/select_for_update/tests.py
Log:
Fixed #16490 -- Skipped a test failure that only occurs under Python 2.6.1
(it's the old iteration-eats-exceptions problem).
Modified: django/trunk/tests/modeltests/select_for_update/tests.py
===================================================================
--- django/trunk/tests/modeltests/select_for_update/tests.py 2011-09-10
18:04:27 UTC (rev 16783)
+++ django/trunk/tests/modeltests/select_for_update/tests.py 2011-09-10
18:44:33 UTC (rev 16784)
@@ -101,10 +101,11 @@
# are swallowed (Python issue 1242657), so these cases return an empty
# list, rather than raising an exception. Not a lot we can do about that,
# unfortunately, due to the way Python handles list() calls internally.
- # Thus, we skip this test for Python 2.6.
+ # Python 2.6.1 is the "in the wild" version affected by this, so we skip
+ # the test for that version.
@requires_threading
@skipUnlessDBFeature('has_select_for_update_nowait')
- @unittest.skipIf(sys.version_info[:2] == (2, 6), "Python version is 2.6")
+ @unittest.skipIf(sys.version_info[:3] == (2, 6, 1), "Python version is
2.6.1")
def test_nowait_raises_error_on_block(self):
"""
If nowait is specified, we expect an error to be raised rather
@@ -124,8 +125,15 @@
self.end_blocking_transaction()
self.check_exc(status[-1])
+ # In Python 2.6 beta and some final releases, exceptions raised in __len__
+ # are swallowed (Python issue 1242657), so these cases return an empty
+ # list, rather than raising an exception. Not a lot we can do about that,
+ # unfortunately, due to the way Python handles list() calls internally.
+ # Python 2.6.1 is the "in the wild" version affected by this, so we skip
+ # the test for that version.
@skipIfDBFeature('has_select_for_update_nowait')
@skipUnlessDBFeature('has_select_for_update')
+ @unittest.skipIf(sys.version_info[:3] == (2, 6, 1), "Python version is
2.6.1")
def test_unsupported_nowait_raises_error(self):
"""
If a SELECT...FOR UPDATE NOWAIT is run on a database backend
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.