diff options
| author | Grzegorz Nosek <root@localdomain.pl> | 2014-02-15 17:45:16 +0100 |
|---|---|---|
| committer | Grzegorz Nosek <root@localdomain.pl> | 2014-02-15 17:51:18 +0100 |
| commit | 0b6f9f7c6fb3589157c3231f28409c97c5c451ab (patch) | |
| tree | 8fdc1ce1b44cb906bffa7a4b3e1d29bdaa1264d0 /tests/urlpatterns_reverse | |
| parent | d0133504e57589dc8983a20bf488e069bddd772c (diff) | |
Improved fix for #18373 -- backward compatibility
Commit 79558c78 cleaned up the (undocumented) interface of Resolver404
exception, which breaks compatibility with code messing with .args[0]
directly. Revert the cleanup part and simply leave the fix itself.
Diffstat (limited to 'tests/urlpatterns_reverse')
| -rw-r--r-- | tests/urlpatterns_reverse/tests.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/urlpatterns_reverse/tests.py b/tests/urlpatterns_reverse/tests.py index 6b0d5f37c6..a7fb342da7 100644 --- a/tests/urlpatterns_reverse/tests.py +++ b/tests/urlpatterns_reverse/tests.py @@ -272,10 +272,10 @@ class ResolverTests(unittest.TestCase): self.fail('resolve did not raise a 404') except Resolver404 as e: # make sure we at least matched the root ('/') url resolver: - self.assertTrue(hasattr(e, 'tried')) - tried = e.tried - self.assertEqual(len(tried), len(url_types_names), 'Wrong number of tried URLs returned. Expected %s, got %s.' % (len(url_types_names), len(tried))) - for tried, expected in zip(tried, url_types_names): + self.assertTrue('tried' in e.args[0]) + tried = e.args[0]['tried'] + self.assertEqual(len(e.args[0]['tried']), len(url_types_names), 'Wrong number of tried URLs returned. Expected %s, got %s.' % (len(url_types_names), len(e.args[0]['tried']))) + for tried, expected in zip(e.args[0]['tried'], url_types_names): for t, e in zip(tried, expected): self.assertIsInstance(t, e['type']), str('%s is not an instance of %s') % (t, e['type']) if 'name' in e: |
