Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Improved tests introduced in 04b00b6.

These tests are expected to fail for the file session backend because it
doesn't handle expiry properly. They didn't because of an error in the
test setup sequence.

Refs #19200, #18194.
  • Loading branch information...
commit 882c47cd405cfd29194f2e968678a5aa1d6ec75f 1 parent cd17a24
Aymeric Augustin aaugustin authored
Showing with 5 additions and 3 deletions.
  1. +5 −3 django/contrib/sessions/tests.py
8 django/contrib/sessions/tests.py
View
@@ -277,9 +277,11 @@ def test_actual_expiry(self):
try:
self.session['foo'] = 'bar'
self.session.set_expiry(-timedelta(seconds=10))
- self.session.create()
+ self.session.save()
+ old_session_key = self.session.session_key
# With an expiry date in the past, the session expires instantly.
new_session = self.backend(self.session.session_key)
+ new_session_key = new_session.session_key
self.assertNotIn('foo', new_session)
finally:
self.session.delete(old_session_key)
@@ -353,15 +355,15 @@ class FileSessionTests(SessionTestsMixin, unittest.TestCase):
backend = FileSession
def setUp(self):
- super(FileSessionTests, self).setUp()
# Do file session tests in an isolated directory, and kill it after we're done.
self.original_session_file_path = settings.SESSION_FILE_PATH
self.temp_session_store = settings.SESSION_FILE_PATH = tempfile.mkdtemp()
+ super(FileSessionTests, self).setUp()
def tearDown(self):
+ super(FileSessionTests, self).tearDown()
settings.SESSION_FILE_PATH = self.original_session_file_path
shutil.rmtree(self.temp_session_store)
- super(FileSessionTests, self).tearDown()
@override_settings(
SESSION_FILE_PATH="/if/this/directory/exists/you/have/a/weird/computer")
Please sign in to comment.
Something went wrong with that request. Please try again.