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

wireprotov2: implement commands as a generator of objects...

wireprotov2: implement commands as a generator of objects Previously, wire protocol version 2 inherited version 1's model of having separate types to represent the results of different wire protocol commands. As I implemented more powerful commands in future commits, I found I was using a common pattern of returning a special type to hold a generator. This meant the command function required a closure to do most of the work. That made logic flow more difficult to follow. I also noticed that many commands were effectively a sequence of objects to be CBOR encoded. I think it makes sense to define version 2 commands as generators. This way, commands can simply emit the data structures they wish to send to the client. This eliminates the need for a closure in command functions and removes encoding from the bodies of commands. As part of this commit, the handling of response objects has been moved into the serverreactor class. This puts the reactor in the driver's seat with regards to CBOR encoding and error handling. Having error handling in the function that emits frames is particularly important because exceptions in that function can lead to things getting in a bad state: I'm fairly certain that uncaught exceptions in the frame generator were causing deadlocks. I also introduced a dedicated error type for explicit error reporting in command handlers. This will be used in subsequent commits. There's still a bit of work to be done here, especially around formalizing the error handling "protocol." I've added yet another TODO to track this so we don't forget. Test output changed because we're using generators and no longer know we are at the end of the data until we hit the end of the generator. This means we can't emit the end-of-stream flag until we've exhausted the generator. Hence the introduction of 0-sized end-of-stream frames. Differential Revision: https://phab.mercurial-scm.org/D4472

Meirambek Omyrzak - - Load All Authors

File last commit:

r39525:f1186c29 default


                r39595:07b58266

default

Download file

             test-basic.t
        
                    103 lines
            
             | 1.9 KiB
            
                | text/troff
            
             |
                Tads3Lexer
            
             / tests / test-basic.t
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        Matt Mackall
    
tests: unify test-basic

              r11742
            
      Create a repository:

        Gregory Szorc
    
tests: conditionalize tests based on presence of custom extensions...

              r37360
            
      #if no-extraextensions

        Matt Mackall
    
config: make repo optional again

              r20608
            
        $ hg config

        Pierre-Yves David
    
devel: rename 'all' to 'all-warnings' (BC)...

              r25290
            
        devel.all-warnings=true

        marmoute
    
devel: activate default-date in tests...

              r32410
            
        devel.default-date=0 0

        FUJIWARA Katsunori
    
tests: add extra output for fsmonitor at showconfig

              r33426
            
        extensions.fsmonitor= (fsmonitor !)

        Matt Harbison
    
tests: remove (glob) annotations that were only for '\' matches...

              r35394
            
        largefiles.usercache=$TESTTMP/.cache/largefiles

        lfs.usercache=$TESTTMP/.cache/lfs

        Matt Mackall
    
config: make repo optional again

              r20608
            
        ui.slash=True

        ui.interactive=False

        FUJIWARA Katsunori
    
filemerge: use 'basic' as the default of '[ui] mergemarkers' for safety...

              r21918
            
        ui.mergemarkers=detailed

        Yuya Nishihara
    
ui: separate option to show prompt echo, enabled only in tests (issue4417)...

              r23053
            
        ui.promptecho=True

        Jun Wu
    
runtests: set web.address to localhost...

              r31009
            
        web.address=localhost

        Jun Wu
    
runtests: always set web.ipv6...

              r31010
            
        web\.ipv6=(?:True|False) (re)

        Gregory Szorc
    
hgweb: allow defining Server response header for HTTP server...

              r37027
            
        web.server-header=testing stub value

        Gregory Szorc
    
tests: conditionalize tests based on presence of custom extensions...

              r37360
            
      #endif

        Martin Geisler
    
tests: remove redundant mkdir...

              r13956
            
        $ hg init t

        Matt Mackall
    
tests: unify test-basic

              r11742
            
        $ cd t

        Bryan O'Sullivan
    
stdio: add Linux-specific tests for error checking

              r31964
            
      Prepare a changeset:

        Matt Mackall
    
tests: unify test-basic

              r11742
            
        $ echo a > a

        $ hg add a

        Bryan O'Sullivan
    
stdio: add Linux-specific tests for error checking

              r31964
            
        $ hg status

        A a

      Writes to stdio succeed and fail appropriately

      #if devfull

        $ hg status 2>/dev/full

        A a

        $ hg status >/dev/full

        abort: No space left on device

        [255]

        Saurabh Singh
    
test-basic: make test compatible with chg...

              r34449
            
      #endif

        Bryan O'Sullivan
    
stdio: add Linux-specific tests for error checking

              r31964
            
        Gregory Szorc
    
dispatch: handle IOError when writing to stderr...

              r35671
            
      #if devfull

        Saurabh Singh
    
test-basic: make test compatible with chg...

              r34449
            
        $ hg status >/dev/full 2>&1

        [255]

        $ hg status ENOENT 2>/dev/full

        [255]

      #endif

        Martin Geisler
    
tests: remove unneeded -d flags...

              r12156
            
        $ hg commit -m test

        Matt Mackall
    
tests: unify test-basic

              r11742
            
      This command is ancient:

        $ hg history

        Martin Geisler
    
tests: remove unneeded -d flags...

              r12156
            
        changeset:   0:acb14030fe0a

        Matt Mackall
    
tests: unify test-basic

              r11742
            
        tag:         tip

        user:        test

        Martin Geisler
    
tests: remove unneeded -d flags...

              r12156
            
        date:        Thu Jan 01 00:00:00 1970 +0000

        Matt Mackall
    
tests: unify test-basic

              r11742
            
        summary:     test

        Mark Drago
    
commands.update() now works properly with a revision of 0...

              r13568
            
      Verify that updating to revision 0 via commands.update() works properly

        $ cat <<EOF > update_to_rev0.py

        > from mercurial import ui, hg, commands

        Yuya Nishihara
    
ui: factor out ui.load() to create a ui without loading configs (API)...

              r30559
            
        > myui = ui.ui.load()

        Yuya Nishihara
    
py3: make test-basic.t pass on Python 3

              r36656
            
        > repo = hg.repository(myui, path=b'.')

        Martin von Zweigbergk
    
tests: pass "rev" argument to commands.update() as string...

              r38499
            
        > commands.update(myui, repo, rev=b"0")

        Mark Drago
    
commands.update() now works properly with a revision of 0...

              r13568
            
        > EOF

        $ hg up null

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

        Augie Fackler
    
cleanup: use $PYTHON to run python in many more tests...

              r32940
            
        $ $PYTHON ./update_to_rev0.py

        Mark Drago
    
commands.update() now works properly with a revision of 0...

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

        $ hg identify -n

        0

        Mads Kiilerich
    
check-code: fix check for trailing whitespace on empty lines...

              r17346
            
        Mark Drago
    
commands.update() now works properly with a revision of 0...

              r13568
            
        Matt Mackall
    
tests: unify test-basic

              r11742
            
      Poke around at hashes:

        $ hg manifest --debug

        b789fdd96dc2f3bd229c1dd8eedf0fc60e2b68e3 644   a

        $ hg cat a

        a

      Verify should succeed:

        $ hg verify

        checking changesets

        checking manifests

        crosschecking files in changesets and manifests

        checking files

        Meirambek Omyrzak
    
verify: make output less confusing (issue5924)...

              r39525
            
        checked 1 changesets with 1 changes to 1 files

        Matt Mackall
    
tests: unify test-basic

              r11742
            
        Yuya Nishihara
    
templatekw: add {reporoot} keyword...

              r36261
            
      Repository root:

        $ hg root

        $TESTTMP/t

        $ hg log -l1 -T '{reporoot}\n'

        $TESTTMP/t

        Matt Mackall
    
tests: unify test-basic

              r11742
            
      At the end...

        Mads Kiilerich
    
tests: add missing trailing 'cd ..'...

              r16913
            
        $ 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

Matt Mackall tests: unify test-basic	r11742	Create a repository:

Gregory Szorc tests: conditionalize tests based on presence of custom extensions...	r37360	#if no-extraextensions
Matt Mackall config: make repo optional again	r20608	$ hg config
Pierre-Yves David devel: rename 'all' to 'all-warnings' (BC)...	r25290	devel.all-warnings=true
marmoute devel: activate default-date in tests...	r32410	devel.default-date=0 0
FUJIWARA Katsunori tests: add extra output for fsmonitor at showconfig	r33426	extensions.fsmonitor= (fsmonitor !)
Matt Harbison tests: remove (glob) annotations that were only for '\' matches...	r35394	largefiles.usercache=$TESTTMP/.cache/largefiles
		lfs.usercache=$TESTTMP/.cache/lfs
Matt Mackall config: make repo optional again	r20608	ui.slash=True
		ui.interactive=False
FUJIWARA Katsunori filemerge: use 'basic' as the default of '[ui] mergemarkers' for safety...	r21918	ui.mergemarkers=detailed
Yuya Nishihara ui: separate option to show prompt echo, enabled only in tests (issue4417)...	r23053	ui.promptecho=True
Jun Wu runtests: set web.address to localhost...	r31009	web.address=localhost
Jun Wu runtests: always set web.ipv6...	r31010	web\.ipv6=(?:True\|False) (re)
Gregory Szorc hgweb: allow defining Server response header for HTTP server...	r37027	web.server-header=testing stub value
Gregory Szorc tests: conditionalize tests based on presence of custom extensions...	r37360	#endif

Martin Geisler tests: remove redundant mkdir...	r13956	$ hg init t
Matt Mackall tests: unify test-basic	r11742	$ cd t

Bryan O'Sullivan stdio: add Linux-specific tests for error checking	r31964	Prepare a changeset:
Matt Mackall tests: unify test-basic	r11742
		$ echo a > a
		$ hg add a
Bryan O'Sullivan stdio: add Linux-specific tests for error checking	r31964
		$ hg status
		A a

		Writes to stdio succeed and fail appropriately

		#if devfull
		$ hg status 2>/dev/full
		A a

		$ hg status >/dev/full
		abort: No space left on device
		[255]
Saurabh Singh test-basic: make test compatible with chg...	r34449	#endif
Bryan O'Sullivan stdio: add Linux-specific tests for error checking	r31964
Gregory Szorc dispatch: handle IOError when writing to stderr...	r35671	#if devfull
Saurabh Singh test-basic: make test compatible with chg...	r34449	$ hg status >/dev/full 2>&1
		[255]

		$ hg status ENOENT 2>/dev/full
		[255]
		#endif

Martin Geisler tests: remove unneeded -d flags...	r12156	$ hg commit -m test
Matt Mackall tests: unify test-basic	r11742
		This command is ancient:

		$ hg history
Martin Geisler tests: remove unneeded -d flags...	r12156	changeset: 0:acb14030fe0a
Matt Mackall tests: unify test-basic	r11742	tag: tip
		user: test
Martin Geisler tests: remove unneeded -d flags...	r12156	date: Thu Jan 01 00:00:00 1970 +0000
Matt Mackall tests: unify test-basic	r11742	summary: test


Mark Drago commands.update() now works properly with a revision of 0...	r13568	Verify that updating to revision 0 via commands.update() works properly

		$ cat <<EOF > update_to_rev0.py
		> from mercurial import ui, hg, commands
Yuya Nishihara ui: factor out ui.load() to create a ui without loading configs (API)...	r30559	> myui = ui.ui.load()
Yuya Nishihara py3: make test-basic.t pass on Python 3	r36656	> repo = hg.repository(myui, path=b'.')
Martin von Zweigbergk tests: pass "rev" argument to commands.update() as string...	r38499	> commands.update(myui, repo, rev=b"0")
Mark Drago commands.update() now works properly with a revision of 0...	r13568	> EOF
		$ hg up null
		0 files updated, 0 files merged, 1 files removed, 0 files unresolved
Augie Fackler cleanup: use $PYTHON to run python in many more tests...	r32940	$ $PYTHON ./update_to_rev0.py
Mark Drago commands.update() now works properly with a revision of 0...	r13568	1 files updated, 0 files merged, 0 files removed, 0 files unresolved
		$ hg identify -n
		0
Mads Kiilerich check-code: fix check for trailing whitespace on empty lines...	r17346
Mark Drago commands.update() now works properly with a revision of 0...	r13568
Matt Mackall tests: unify test-basic	r11742	Poke around at hashes:

		$ hg manifest --debug
		b789fdd96dc2f3bd229c1dd8eedf0fc60e2b68e3 644 a

		$ hg cat a
		a

		Verify should succeed:

		$ hg verify
		checking changesets
		checking manifests
		crosschecking files in changesets and manifests
		checking files
Meirambek Omyrzak verify: make output less confusing (issue5924)...	r39525	checked 1 changesets with 1 changes to 1 files
Matt Mackall tests: unify test-basic	r11742
Yuya Nishihara templatekw: add {reporoot} keyword...	r36261	Repository root:

		$ hg root
		$TESTTMP/t
		$ hg log -l1 -T '{reporoot}\n'
		$TESTTMP/t

Matt Mackall tests: unify test-basic	r11742	At the end...
Mads Kiilerich tests: add missing trailing 'cd ..'...	r16913
		$ cd ..