Changeset 9224 for django/trunk/django

Timestamp:

10/10/08 16:41:12 (3 months ago)

Author:

jacob

Message:

More be-nice-to-the-buildbot: be better about cleaning up files created by the cache/session tests.

Files:

• django/trunk/django/contrib/sessions/tests.py (modified) (3 diffs)

django/trunk/django/contrib/sessions/tests.py

@@ -55 +55 @@
->>> DatabaseSession('1').get('cat')
-
+# Do file session tests in an isolated directory, and kill it after we're done.
+>>> original_session_file_path = settings.SESSION_FILE_PATH
+>>> import tempfile
+>>> temp_session_store = settings.SESSION_FILE_PATH = tempfile.mkdtemp()
+
->>> file_session = FileSession()
->>> file_session.modified
@@ -106 +111 @@
-ImproperlyConfigured: The session storage path '/if/this/directory/exists/you/have/a/weird/computer' doesn't exist. Please set your SESSION_FILE_PATH setting to an existing directory in which Django can store session data.
-
+# Clean up after the file tests
+>>> settings.SESSION_FILE_PATH = original_session_file_path
+>>> import shutil
+>>> shutil.rmtree(temp_session_store)
+
+#
+# Cache-based tests
+# NB: be careful to delete any sessions created; stale sessions fill up the
+# /tmp and eventually overwhelm it after lots of runs (think buildbots)
+#
+
->>> cache_session = CacheSession()
->>> cache_session.modified
@@ -145 +161 @@
->>> cache_session = CacheSession(cache_session.session_key)
->>> cache_session.save()
-CacheSession('1').get('cat'
+cache_session.delete(cache_session.session_key
-
->>> s = SessionBase()