[PATCH 1 of 2 STABLE V2] tests: add test demonstrating buggy path handling

[Gregory Szorc]

On Sat, Apr 8, 2017 at 1:39 AM, Yuya Nishihara <yuya at tcha.org> wrote:

> On Fri, 07 Apr 2017 11:54:54 -0700, Gregory Szorc wrote:
> > # HG changeset patch
> > # User Gregory Szorc <gregory.szorc at gmail.com>
> > # Date 1491590783 25200
> > #      Fri Apr 07 11:46:23 2017 -0700
> > # Branch stable
> > # Node ID f8ba1fb4458b60b1d129f97d1f47a542aa0daf98
> > # Parent  68f263f52d2e3e2798b4f1e55cb665c6b043f93b
> > tests: add test demonstrating buggy path handling
> >
> > `hg debugupgraderepo` is currently buggy with regards to path
> > handling when copying files in .hg/store/. Specifically, it applies
> > the store filename encoding to paths instead of operating on raw
> > files.
> >
> > This commit adds a test demonstrating the buggy behavior.
> >
> > diff --git a/tests/test-upgrade-repo.t b/tests/test-upgrade-repo.t
> > --- a/tests/test-upgrade-repo.t
> > +++ b/tests/test-upgrade-repo.t
> > @@ -310,3 +310,38 @@ old store should be backed up
> >    undo.phaseroots
> >
> >    $ cd ..
> > +
> > +store files with special filenames aren't encoded during copy
> > +
> > +  $ hg init store-filenames
> > +  $ cd store-filenames
> > +  $ touch foo
> > +  $ hg -q commit -A -m initial
> > +  $ touch .hg/store/.XX_special_filename
> > +
> > +  $ hg debugupgraderepo --run
> > +  upgrade will perform the following actions:
> > +
> > +  requirements
> > +     preserved: dotencode, fncache, generaldelta, revlogv1, store
> > +
> > +  beginning upgrade...
> > +  repository locked and read-only
> > +  creating temporary repository to stage migrated data:
> $TESTTMP/store-filenames/.hg/upgrade.* (glob)
> > +  (it is safe to interrupt this process any time before data migration
> completes)
> > +  migrating 3 total revisions (1 in filelogs, 1 in manifests, 1 in
> changelog)
> > +  migrating 109 bytes in store; 107 bytes tracked data
> > +  migrating 1 filelogs containing 1 revisions (0 bytes in store; 0
> bytes tracked data)
> > +  finished migrating 1 filelog revisions across 1 filelogs; change in
> size: 0 bytes
> > +  migrating 1 manifests containing 1 revisions (46 bytes in store; 45
> bytes tracked data)
> > +  finished migrating 1 manifest revisions across 1 manifests; change in
> size: 0 bytes
> > +  migrating changelog containing 1 revisions (63 bytes in store; 62
> bytes tracked data)
> > +  finished migrating 1 changelog revisions; change in size: 0 bytes
> > +  finished migrating 3 total revisions; total change in store size: 0
> bytes
> > +  copying phaseroots
> > +  copying .XX_special_filename
>
> The order is unstable. I could add "2>&1 | egrep 'abort|XX'", but do you
> have
> any better idea?
>

Doh. V3 sent with a new patch to add a sorted(). I also added a missing
(glob) for the "0.0s" bit.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20170408/af6c8158/attachment.html>