transaction: use ProgrammingError for when an committed transaction is used

It seems to me that ProgrammingError is the right type of error here.

Differential Revision: https://phab.mercurial-scm.org/D9215

mercurial/transaction.py

@@ -38,11 +38,9 b" GEN_GROUP_POST_FINALIZE = b'postfinalize"
 def active(func):
     def _active(self, *args, **kwds):
         if self._count == 0:
-            raise error.Abort(
-                _(
+            raise error.ProgrammingError(
                     b'cannot use transaction when it is already committed/aborted'
                 )
-            )
         return func(self, *args, **kwds)
 
     return _active