Describe the changes explicitely in Changes.rst (and not only via a…

… reference to #58).

Change comment formatting in the test suite.

CHANGES.rst

@@ -5,8 +5,14 @@
 2.4.1 (unreleased)
 ==================
 
-- Fix transaction manager run problems
-  (`#58 <https://github.com/zopefoundation/transaction/issues/58>`_)
+- ``TransactionManager.run`` now commits/aborts the transaction
+  "active" after the execution of *func* (and no longer the initial
+  transaction which might already have been committed/aborted by *func*)
+  (`#58 <https://github.com/zopefoundation/transaction/issues/58>`_).
+
+  It aborts the transaction now for all exceptions raised by *func* - even
+  if it is only an instance of `BaseException` but not of `Exception`,
+  such as e.g. a ``SystemExit`` or ``KeyboardInterupt`` exception.
 
 
 2.4.0 (2018-10-23)

transaction/tests/test__manager.py

@@ -704,15 +704,17 @@ def run(self):
         runner.join(1)
 
     # The lack of the method below caused a test failure in one run
-    #   -- maybe caused indirectly by the failure of another test
-    # Defining the methods below reduced the "test coverage". We
-    #   therefore remove them again
-    # the preceeding test (maybe others as well) registers `self` as
-    #   synchronizer; satisfy the `ISynchronizer` requirements
-##    def newTransaction(self, transaction):
-##        pass
-
-##    beforeCompletion = afterCompletion = newTransaction
+    #   -- caused indirectly by the failure of another test (this
+    #   indicates that the tests in this suite are not fully isolated).
+    #   However, defining the methods below reduced the "test coverage"
+    #   once the initial test failure has been fixed.
+    #   We therefore comment them out.
+    ##    # the preceeding test (maybe others as well) registers `self` as
+    ##    #   synchronizer; satisfy the `ISynchronizer` requirements
+    ##    def newTransaction(self, transaction):
+    ##        pass
+    ##
+    ##    beforeCompletion = afterCompletion = newTransaction
 
 
 class TestThreadTransactionManager(unittest.TestCase):