Ensure mapped objects are expired following a transaction.commit() when

  no database commit was required. Closes #8.

CHANGES.txt

@@ -4,6 +4,10 @@ Changes
 0.7.5 (unreleased)
 ------------------
 
+* Ensure mapped objects are expired following a ``transaction.commit()`` when
+  no database commit was required.
+
+  See: https://github.com/zopefoundation/zope.sqlalchemy/issues/8
 
 0.7.4 (2014-01-06)
 ------------------

src/zope/sqlalchemy/datamanager.py

@@ -92,6 +92,9 @@ def tpc_begin(self, trans):
     def commit(self, trans):
         status = _SESSION_STATE[id(self.session)]
         if status is not STATUS_INVALIDATED:
+            session = self.session
+            if session.expire_on_commit:
+                session.expire_all()
             self._finish('no work')
 
     def tpc_vote(self, trans):

src/zope/sqlalchemy/tests.py

@@ -666,6 +666,17 @@ def testKeepSession(self):
             # KeepSession does not rollback on transaction abort
             session.rollback()
 
+    def testExpireAll(self):
+        session = Session()
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        transaction.commit()
+
+        session = Session()
+        instance = session.query(User).get(1)
+        transaction.commit()  # No work, session.close()
+
+        self.assertEqual(sa.inspect(instance).expired, True)
+
 
 class RetryTests(unittest.TestCase):