try to increase test coverage

transaction/_transaction.py

@@ -120,7 +120,6 @@ def __init__(self, synchronizers=None, manager=None):
         # List of (hook, args, kws) tuples added by addAfterCommitHook().
         self._after_commit = []
 
-
         # List of (hook, args, kws) tuples added by addBeforeAbortHook().
         self._before_abort = []
 

transaction/tests/test__transaction.py

@@ -1082,6 +1082,29 @@ def _hook2(*args, **kw):
         self.assertTrue(logger._log[0][1].startswith(
                             "Error in abort() on manager"))
 
+    def test_callAfterAbortHook_w_error_w_abort_error(self):
+        from transaction.tests.common import DummyLogger
+        from transaction.tests.common import Monkey
+        from transaction import _transaction
+        _hooked2 = []
+        def _hook1(*args, **kw):
+            raise ValueError()
+        def _hook2(*args, **kw):
+            _hooked2.append((args, kw))
+        logger = DummyLogger()
+        with Monkey(_transaction, _LOGGER=logger):
+            txn = self._makeOne()
+        logger._clear()
+        r = Resource("r", "abort")
+        txn.join(r)
+        txn.addAfterAbortHook(_hook1, ('one',), dict(dos=1))
+        txn.addAfterAbortHook(_hook2, ('two',), dict(dos=2))
+        with self.assertRaises(ValueError):
+            txn._callAfterAbortHooks()
+        self.assertEqual(logger._log[0][0], 'error')
+        self.assertTrue(logger._log[0][1].startswith(
+                            "Error in abort() on manager"))
+
     def test_abort_w_abortHooks(self):
         comm = []
         txn = self._makeOne()