tests/testlib/ext-phase-report.py

rebase: delete seemingly unnecessary needupdate()

This seemed to be about checking that the user hasn't updated away
when we asked them to resolve merge conflicts. These days we call
`cmdutil.checkunfinished()` and refuse to update, so the user
shouldn't be able to get into this state.

`test-rebase-interruptions.t` actually has some tests where it
disables the rebase extension in order to be allowed to do some of
these updates. That still passes, but I wouldn't personally haved
cared if that failed.

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

# tiny extension to report phase changes during transaction

from __future__ import absolute_import


def reposetup(ui, repo):
    def reportphasemove(tr):
        for rev, move in sorted(tr.changes[b'phases'].items()):
            if move[0] is None:
                ui.write(
                    (
                        b'test-debug-phase: new rev %d:  x -> %d\n'
                        % (rev, move[1])
                    )
                )
            else:
                ui.write(
                    (
                        b'test-debug-phase: move rev %d: %d -> %d\n'
                        % (rev, move[0], move[1])
                    )
                )

    class reportphaserepo(repo.__class__):
        def transaction(self, *args, **kwargs):
            tr = super(reportphaserepo, self).transaction(*args, **kwargs)
            tr.addpostclose(b'report-phase', reportphasemove)
            return tr

    repo.__class__ = reportphaserepo