upstream/mercurial-mirror Files · tests/test-pull.t

largefiles: access to specific fields only if largefiles enabled (issue4547)...

largefiles: access to specific fields only if largefiles enabled (issue4547) Even if largefiles extension is enabled in a repository, "repo" object, which isn't "largefiles.reposetup()"-ed, is passed to overridden functions in the cases below unexpectedly, because extensions are enabled for each repositories strictly. (1) clone without -U: (2) pull with -U: (3) pull with --rebase: combination of "enabled@src", "disabled@dst" and "not-required@src" cause this situation. largefiles requirement @src @dst @src result -------- -------- --------------- -------------------- enabled disabled not-required aborted unexpectedly required requirement error (intentional) -------- -------- --------------- -------------------- enabled enabled * success -------- -------- --------------- -------------------- disabled enabled * success (only for "pull") -------- -------- --------------- -------------------- disabled disabled not-required success required requirement error (intentional) -------- -------- --------------- -------------------- (4) update/revert with a subrepo disabling largefiles In these cases, overridden functions cause accessing to largefiles specific fields of not "largefiles.reposetup()"-ed "repo" object, and execution is aborted. - (1), (2), (4) cause accessing to "_lfstatuswriters" in "getstatuswriter()" invoked via "updatelfiles()" - (3) causes accessing to "_lfcommithooks" in "overriderebase()" For safe accessing to these fields, this patch examines whether passed "repo" object is "largefiles.reposetup()"-ed or not before accessing to them. This patch chooses examining existence of newly introduced "_largefilesenabled" instead of "_lfcommithooks" and "_lfstatuswriters" directly, because the former is better name for the generic "largefiles is enabled in this repo" mark than the latter. In the future, all other overridden functions should avoid largefiles specific processing for efficiency, and "_largefilesenabled" is better also for such purpose. BTW, "lfstatus" can't be used for such purpose, because some code paths set it forcibly regardless of existence of it in specified "repo" object.

Augie Fackler - - Load All Authors

File last commit:

r22947:c63a09b6 default


                r24158:d414c28d

stable

Download file

             test-pull.t
        
                    92 lines
            
             | 2.3 KiB
            
                | text/troff
            
             |
                Tads3Lexer
            
             / tests / test-pull.t
          
                    History
                
                 |
                  Annotation
                 | Raw
                 |Copy content
                 |Copy permalink

      #require serve

        $ hg init test

        $ cd test

        $ echo foo>foo

        $ hg addremove

        adding foo

        $ hg commit -m 1

        $ hg verify

        checking changesets

        checking manifests

        crosschecking files in changesets and manifests

        checking files

        1 files, 1 changesets, 1 total revisions

        $ hg serve -p $HGPORT -d --pid-file=hg.pid

        $ cat hg.pid >> $DAEMON_PIDS

        $ cd ..

        $ hg clone --pull http://foo:bar@localhost:$HGPORT/ copy

        requesting all changes

        adding changesets

        adding manifests

        adding file changes

        added 1 changesets with 1 changes to 1 files

        updating to branch default

        1 files updated, 0 files merged, 0 files removed, 0 files unresolved

        $ cd copy

        $ hg verify

        checking changesets

        checking manifests

        crosschecking files in changesets and manifests

        checking files

        1 files, 1 changesets, 1 total revisions

        $ hg co

        0 files updated, 0 files merged, 0 files removed, 0 files unresolved

        $ cat foo

        foo

        $ hg manifest --debug

        2ed2a3912a0b24502043eae84ee4b279c18b90dd 644   foo

        $ hg pull

        pulling from http://foo@localhost:$HGPORT/

        searching for changes

        no changes found

        $ hg rollback --dry-run --verbose

        repository tip rolled back to revision -1 (undo pull: http://foo:***@localhost:$HGPORT/)

      Issue622: hg init && hg pull -u URL doesn't checkout default branch

        $ cd ..

        $ hg init empty

        $ cd empty

        $ hg pull -u ../test

        pulling from ../test

        requesting all changes

        adding changesets

        adding manifests

        adding file changes

        added 1 changesets with 1 changes to 1 files

        1 files updated, 0 files merged, 0 files removed, 0 files unresolved

      Test 'file:' uri handling:

        $ hg pull -q file://../test-does-not-exist

        abort: file:// URLs can only refer to localhost

        [255]

        $ hg pull -q file://../test

        abort: file:// URLs can only refer to localhost

        [255]

        $ hg pull -q file:../test

      It's tricky to make file:// URLs working on every platform with

      regular shell commands.

        $ URL=`$PYTHON -c "import os; print 'file://foobar' + ('/' + os.getcwd().replace(os.sep, '/')).replace('//', '/') + '/../test'"`

        $ hg pull -q "$URL"

        abort: file:// URLs can only refer to localhost

        [255]

        $ URL=`$PYTHON -c "import os; print 'file://localhost' + ('/' + os.getcwd().replace(os.sep, '/')).replace('//', '/') + '/../test'"`

        $ hg pull -q "$URL"

        $ cd ..

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

				#require serve

				$ hg init test
				$ cd test

				$ echo foo>foo
				$ hg addremove
				adding foo
				$ hg commit -m 1

				$ hg verify
				checking changesets
				checking manifests
				crosschecking files in changesets and manifests
				checking files
				1 files, 1 changesets, 1 total revisions

				$ hg serve -p $HGPORT -d --pid-file=hg.pid
				$ cat hg.pid >> $DAEMON_PIDS
				$ cd ..

				$ hg clone --pull http://foo:bar@localhost:$HGPORT/ copy
				requesting all changes
				adding changesets
				adding manifests
				adding file changes
				added 1 changesets with 1 changes to 1 files
				updating to branch default
				1 files updated, 0 files merged, 0 files removed, 0 files unresolved

				$ cd copy
				$ hg verify
				checking changesets
				checking manifests
				crosschecking files in changesets and manifests
				checking files
				1 files, 1 changesets, 1 total revisions

				$ hg co
				0 files updated, 0 files merged, 0 files removed, 0 files unresolved
				$ cat foo
				foo

				$ hg manifest --debug
				2ed2a3912a0b24502043eae84ee4b279c18b90dd 644 foo

				$ hg pull
				pulling from http://foo@localhost:$HGPORT/
				searching for changes
				no changes found

				$ hg rollback --dry-run --verbose
				repository tip rolled back to revision -1 (undo pull: http://foo:***@localhost:$HGPORT/)

				Issue622: hg init && hg pull -u URL doesn't checkout default branch

				$ cd ..
				$ hg init empty
				$ cd empty
				$ hg pull -u ../test
				pulling from ../test
				requesting all changes
				adding changesets
				adding manifests
				adding file changes
				added 1 changesets with 1 changes to 1 files
				1 files updated, 0 files merged, 0 files removed, 0 files unresolved

				Test 'file:' uri handling:

				$ hg pull -q file://../test-does-not-exist
				abort: file:// URLs can only refer to localhost
				[255]

				$ hg pull -q file://../test
				abort: file:// URLs can only refer to localhost
				[255]

				$ hg pull -q file:../test

				It's tricky to make file:// URLs working on every platform with
				regular shell commands.

				$ URL=`$PYTHON -c "import os; print 'file://foobar' + ('/' + os.getcwd().replace(os.sep, '/')).replace('//', '/') + '/../test'"`
				$ hg pull -q "$URL"
				abort: file:// URLs can only refer to localhost
				[255]

				$ URL=`$PYTHON -c "import os; print 'file://localhost' + ('/' + os.getcwd().replace(os.sep, '/')).replace('//', '/') + '/../test'"`
				$ hg pull -q "$URL"

				$ cd ..