PEP8 pass.

gtaylor · gtaylor · commit 03080e861eb2 · 2015-03-03T05:00:58.000-05:00
diff --git a/README.rst b/README.rst
@@ -97,6 +97,12 @@ The following settings may be used in your ``settings.py``:
 Changes
 -------
 
+0.6
+^^^
+
+* Removing some no longer used imports.
+* PEP8 cleanup.
+
 0.5
 ^^^
 
@@ -127,4 +133,4 @@ License
 
 django-dynamodb-sessions is licensed under the `BSD License`_.
 
-.. _BSD License: https://github.com/gtaylor/django-dynamodb-sessions/blob/master/LICENSE
+.. _BSD License: https://github.com/gtaylor/django-dynamodb-sessions/blob/master/LICENSE
diff --git a/dynamodb_sessions/backends/cached_dynamodb.py b/dynamodb_sessions/backends/cached_dynamodb.py
@@ -3,11 +3,13 @@
 """
 
 from django.conf import settings
-from dynamodb_sessions.backends.dynamodb import SessionStore as DynamoDBStore
 from django.core.cache import cache
 
+from dynamodb_sessions.backends.dynamodb import SessionStore as DynamoDBStore
+
 KEY_PREFIX = "dynamodb_sessions.backends.cached_dynamodb"
 
+
 class SessionStore(DynamoDBStore):
     """
     Implements cached, database backed sessions.
@@ -49,6 +51,7 @@ def flush(self):
         Removes the current session data from the database and regenerates the
         key.
         """
+
         self.clear()
         self.delete(self.session_key)
         self.create()
diff --git a/dynamodb_sessions/backends/dynamodb.py b/dynamodb_sessions/backends/dynamodb.py
@@ -45,6 +45,7 @@ def dynamodb_connection_factory():
     boto.dynamodb.layer2.Layer2 objects are state-less (aside from security
     tokens), we're not too concerned about thread safety issues.
     """
+
     global _DYNAMODB_CONN
     if not _DYNAMODB_CONN:
         logger.debug("Creating a DynamoDB connection.")
@@ -60,11 +61,11 @@ class SessionStore(SessionBase):
     """
     Implements DynamoDB session store.
     """
+
     def __init__(self, session_key=None):
         super(SessionStore, self).__init__(session_key)
         self.table = dynamodb_connection_factory().get_table(TABLE_NAME)
 
-
     def load(self):
         """
         Loads session data from DynamoDB, runs it through the session
@@ -75,9 +76,9 @@ def load(self):
         """
 
         try:
-            item = self.table.get_item(self.session_key,
-                                       consistent_read=ALWAYS_CONSISTENT)
-        except (DynamoDBKeyNotFoundError,SuspiciousOperation):
+            item = self.table.get_item(
+                self.session_key, consistent_read=ALWAYS_CONSISTENT)
+        except (DynamoDBKeyNotFoundError, SuspiciousOperation):
             self.create()
             return {}
 
@@ -92,6 +93,7 @@ def exists(self, session_key):
         :returns: ``True`` if a session with the given key exists in the DB,
             ``False`` if not.
         """
+
         key_already_exists = self.table.has_item(
             session_key,
             consistent_read=ALWAYS_CONSISTENT,
@@ -106,6 +108,7 @@ def create(self):
         Creates a new entry in DynamoDB. This may or may not actually
         have anything in it.
         """
+
         while True:
             try:
                 # Save immediately to ensure we have a unique entry in the
@@ -127,6 +130,7 @@ def save(self, must_create=False):
         :raises: ``CreateError`` if ``must_create`` is ``True`` and a session
             with the current session key already exists.
         """
+
         # If the save method is called with must_create equal to True, I'm
         # setting self._session_key equal to None and when
         # self.get_or_create_session_key is called the new
@@ -138,11 +142,11 @@ def save(self, must_create=False):
         item = self.table.new_item(self.session_key)
         # Queue up a PUT operation for UpdateItem, which preserves the
         # existing 'created' attribute.
-        item.put_attribute('data',self.encode(self._get_session(no_load=must_create)))
+        item.put_attribute('data', self.encode(self._get_session(no_load=must_create)))
 
         if must_create:
 
-            item.put_attribute('created',int(time.time()))
+            item.put_attribute('created', int(time.time()))
             # We expect the data value to be False because we are creating a
             # new session
             item.put(expected_value={'data': False})
@@ -158,6 +162,7 @@ def delete(self, session_key=None):
         :keyword str session_key: Optionally, override the session key
             to delete.
         """
+
         if session_key is None:
             if self.session_key is None:
                 return
diff --git a/dynamodb_sessions/models.py b/dynamodb_sessions/models.py
@@ -1,5 +0,0 @@
-'''
-Created on Feb 13, 2013
-
-@author: brian
-'''
diff --git a/dynamodb_sessions/tests.py b/dynamodb_sessions/tests.py
@@ -1,19 +1,15 @@
-'''
-Created on Feb 13, 2013
-
-@author: brian
-'''
-from django.contrib.sessions.tests import SessionTestsMixin,DatabaseSessionTests
+from django.contrib.sessions.tests import SessionTestsMixin
 from django.test import TestCase
 
 from .backends.dynamodb import SessionStore as DynamoDBSession
 from .backends.cached_dynamodb import SessionStore as CachedDynamoDBSession
 
-class DynamoDBTestCase(SessionTestsMixin,TestCase):
+
+class DynamoDBTestCase(SessionTestsMixin, TestCase):
     
     backend = DynamoDBSession
     
-class CachedDynamoDBTestCase(SessionTestsMixin,TestCase):
+
+class CachedDynamoDBTestCase(SessionTestsMixin, TestCase):
     
     backend = CachedDynamoDBSession
-    

-Original file line number
+Diff line change
@@ @@ -1,5 +0,0 @@ @@
 -'''
 -Created on Feb 13, 2013
+-
 -@author: brian
 -'''