Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed #13362 -- Disabled the test for bug #8245 under Python 2.3 due …

…to differences in exception handling.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@13021 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit d2fec79d7bbe6eae337107969b836a1a0b14b89b 1 parent d9a0fd4
Russell Keith-Magee freakboy3742 authored
Showing with 22 additions and 19 deletions.
  1. +22 −19 tests/regressiontests/bug8245/tests.py
41 tests/regressiontests/bug8245/tests.py
View
@@ -1,3 +1,4 @@
+import sys
from unittest import TestCase
from django.contrib import admin
@@ -8,23 +9,25 @@ class Bug8245Test(TestCase):
Test for bug #8245 - don't raise an AlreadyRegistered exception when using
autodiscover() and an admin.py module contains an error.
"""
+ if sys.version_info[1] >= 4:
+ # Due to a bug in Python 2.3, this test will fail. The actual
+ # feature works fine; it's just a testing problem. See #13362 for details.
+ def test_bug_8245(self):
+ # The first time autodiscover is called, we should get our real error.
+ try:
+ admin.autodiscover()
+ except Exception, e:
+ self.failUnlessEqual(str(e), "Bad admin module")
+ else:
+ self.fail(
+ 'autodiscover should have raised a "Bad admin module" error.')
- def test_bug_8245(self):
- # The first time autodiscover is called, we should get our real error.
- try:
- admin.autodiscover()
- except Exception, e:
- self.failUnlessEqual(str(e), "Bad admin module")
- else:
- self.fail(
- 'autodiscover should have raised a "Bad admin module" error.')
-
- # Calling autodiscover again should raise the very same error it did
- # the first time, not an AlreadyRegistered error.
- try:
- admin.autodiscover()
- except Exception, e:
- self.failUnlessEqual(str(e), "Bad admin module")
- else:
- self.fail(
- 'autodiscover should have raised a "Bad admin module" error.')
+ # Calling autodiscover again should raise the very same error it did
+ # the first time, not an AlreadyRegistered error.
+ try:
+ admin.autodiscover()
+ except Exception, e:
+ self.failUnlessEqual(str(e), "Bad admin module")
+ else:
+ self.fail(
+ 'autodiscover should have raised a "Bad admin module" error.')
Please sign in to comment.
Something went wrong with that request. Please try again.