annotate tests/testlib/ext-phase-report.py @ 51274:bec6e9c108fd

matchers: use correct method for finding index in vector

The path matcher has an optimization for when all paths are `rootfilesin:`. This
optimization exists in both Python and Rust. However, the Rust implementation
currently has a bug that makes it fail in most cases. The bug is that it
`rfind()` where it was clearly intended to use `rposition()`. This patch fixes
that and adds a test.

# tiny extension to report phase changes during transaction


def reposetup(ui, repo):
    def reportphasemove(tr):
        for revs, move in sorted(tr.changes[b"phases"], key=lambda r: r[0][0]):
            for rev in revs:
                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