comparison tests/testlib/crash_transaction_late.py @ 48685:21ac6aedd5e5

transaction: do not rely on a global variable to post_finalize file

We can just add a new argument to the `addfilegenerator` function. This is more
explicit and therefor clearer and less error prone.

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

48684:568f63b5a30f

from __future__ import absolute_import

from mercurial import (
    error,
    transaction,
)


def abort(fp):
    raise error.Abort(b"This is a late abort")


def reposetup(ui, repo):

    transaction.postfinalizegenerators.add(b'late-abort')

    class LateAbortRepo(repo.__class__):
        def transaction(self, *args, **kwargs):
            tr = super(LateAbortRepo, self).transaction(*args, **kwargs)
            tr.addfilegenerator(
                b'late-abort', [b'late-abort'], abort, order=9999999
            )
            return tr

    repo.__class__ = LateAbortRepo

48685:21ac6aedd5e5

from __future__ import absolute_import

from mercurial import (
    error,
)


def abort(fp):
    raise error.Abort(b"This is a late abort")


def reposetup(ui, repo):
    class LateAbortRepo(repo.__class__):
        def transaction(self, *args, **kwargs):
            tr = super(LateAbortRepo, self).transaction(*args, **kwargs)
            tr.addfilegenerator(
                b'late-abort',
                [b'late-abort'],
                abort,
                order=9999999,
                post_finalize=True,
            )
            return tr

    repo.__class__ = LateAbortRepo