This is an archive of the discontinued Mercurial Phabricator instance.

tests/test-http-protocol.t
3	The new code make sure it stay disabled. The end of the series enable it by default for rust build, we want to keep it off in some tests (where it does not make a different and introduce a LOT of noise)

tests/test-http-protocol.t
3	Okay seems like it's a different understanding of English which confused me. Re-reading few times I can understand what you meant.

This revision is now accepted and ready to land.Apr 11 2021, 5:30 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			tests/test-http-protocol.t (5 lines)

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	26740		Apr 8 2021, 11:34 PM	★	★
Diff 2	26761		Apr 9 2021, 5:27 AM	★	★
Diff 3	26789	rHG1a85c5d75d53b0ec8b348c4dd2e5025d6d210ede	Apr 8 2021, 9:44 PM	★	★

Commit	Parents	Author	Summary	Date
63ffa9bfe85c	3378a68f6824	Pierre-Yves David		Apr 8 2021, 9:44 PM

Status	Author	Revision
Closed	marmoute	D9765 persistent-nodemap: enable the feature by default when using Rust
Closed	marmoute	D10338 persistent-nodemap: disable it for test-wireproto-content-redirects.t
Closed	marmoute	D10337 persistent-nodemap: disable it for test-wireproto-command-capabilities.t
Closed	marmoute	D10336 persistent-nodemap: disable it unconditionally for test-wireproto-caching.t
Closed	marmoute	D10335 persistent-nodemap: disable it unconditionally for test-ssh-proto-unbundle.t
Closed	marmoute	D10334 persistent-nodemap: disable it unconditionally for test-ssh-proto.t
Closed	marmoute	D10333 persistent-nodemap: disable it unconditionally for test-share-safe.t
Closed	marmoute	D10332 persistent-nodemap: disable it unconditionally for test-http-protocol.t
Closed	marmoute	D10331 persistent-nodemap: disable it unconditionally for test-http-bad-server.t

	#require no-chg			#require no-chg

				persistent-nodemap is not enabled by default. It is not relevant for this test
				pulkitUnsubmitted Not Done This sentence confuses me. The test is disabling it right? pulkit: This sentence confuses me. The test is disabling it right?
				marmouteAuthorUnsubmitted Done The new code make sure it stay disabled. The end of the series enable it by default for rust build, we want to keep it off in some tests (where it does not make a different and introduce a LOT of noise) marmoute: The new code make sure it stay disabled. The end of the series enable it by default for rust…
				pulkitUnsubmitted Not Done Okay seems like it's a different understanding of English which confused me. Re-reading few times I can understand what you meant. pulkit: Okay seems like it's a different understanding of English which confused me. Re-reading few…
				so disable it.

	$ . $TESTDIR/wireprotohelpers.sh			$ . $TESTDIR/wireprotohelpers.sh

	$ cat >> $HGRCPATH << EOF			$ cat >> $HGRCPATH << EOF
				> [format]
				> use-persistent-nodemap = no
	> [web]			> [web]
	> push_ssl = false			> push_ssl = false
	> allow_push = *			> allow_push = *
	> EOF			> EOF

	$ hg init server			$ hg init server
	$ cd server			$ cd server
	$ touch a			$ touch a