diff options
author | Tim Graham <timograham@gmail.com> | 2016-06-28 11:21:26 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-06-28 11:21:26 -0400 |
commit | c9ae09addffb839403312131d4251e9d8b454508 (patch) | |
tree | 74913fbe2e90d88e094f8594947ebf313ec5f12f /tests/sessions_tests | |
parent | c1b6f554e405fe733e8d80f7e3d77c277810e707 (diff) | |
download | django-c9ae09addffb839403312131d4251e9d8b454508.tar.gz |
Replaced use of TestCase.fail() with assertRaises().
Also removed try/except/fail antipattern that hides exceptions.
Diffstat (limited to 'tests/sessions_tests')
-rw-r--r-- | tests/sessions_tests/tests.py | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/tests/sessions_tests/tests.py b/tests/sessions_tests/tests.py index dbba1235b5..480eb72c8f 100644 --- a/tests/sessions_tests/tests.py +++ b/tests/sessions_tests/tests.py @@ -198,13 +198,7 @@ class SessionTestsMixin(object): # removed the key) results in a new key being generated. try: session = self.backend('1') - try: - session.save() - except AttributeError: - self.fail( - "The session object did not save properly. " - "Middleware may be saving cache items without namespaces." - ) + session.save() self.assertNotEqual(session.session_key, '1') self.assertIsNone(session.get('cat')) session.delete() |