This is an archive of the discontinued Mercurial Phabricator instance.

Differential D3101

tests: disable test-audit-path.t with simple store
ClosedPublic

Authored by indygreg on Apr 4 2018, 9:24 PM.

Download Raw Diff

Details

Reviewers

durin42

Group Reviewers

hg-reviewers

Commits

rHG9984488550ea: tests: disable test-audit-path.t with simple store

Summary

The simple store is using a primitive filename encoding scheme.
Many tests in this file fail. So let's disable it for now.

At some point, we should support a mode where we can run tests
without fnstore or dotencode and have all tests assuming use of
these filename encoding mechanisms be conditional on their usage.

Diff Detail

Repository

rHG Mercurial

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Event Timeline

indygreg created this revision.Apr 4 2018, 9:24 PM

Herald added a reviewer: hg-reviewers. · View Herald TranscriptApr 4 2018, 9:24 PM

Herald added a subscriber: mercurial-devel. · View Herald Transcript

indygreg added a child revision: D3102: tests: skip test-hgweb-bundle.t if we don't support bundlerepos.Apr 4 2018, 9:24 PM

durin42 accepted this revision.Apr 6 2018, 8:34 PM

This revision is now accepted and ready to land.Apr 6 2018, 8:34 PM

Closed by commit rHG9984488550ea: tests: disable test-audit-path.t with simple store (authored by indygreg). · Explain WhyApr 6 2018, 9:20 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

			Path	Packages
M			tests/test-audit-path.t (4 lines)

Status	Author	Revision
Closed	indygreg	D3154 filelog: wrap revlog instead of inheriting it (API)
Closed	indygreg	D3153 tests: call rawsize() directly
Closed	indygreg	D3152 upgrade: sniff for filelog type
Closed	indygreg	D3151 revlog: move censor logic into main revlog class
Closed	indygreg	D3150 revlog: move parsemeta() and packmeta() from filelog (API)
Closed	indygreg	D3149 filelog: declare that filelog implements a storage interface
Closed	indygreg	D3148 repository: define existing interface for file storage
Closed	indygreg	D3147 tests: run some largefiles and lfs tests with simple store
Closed	indygreg	D3119 commands: don't violate storage abstractions in `manifest --all`
Closed	indygreg	D3118 commands: document the layering violation in `manifest --all`
Abandoned	indygreg	D3117 tests: make test-convert-git.t more deterministic
Closed	indygreg	D3116 simplestore: correctly implement flag processors
Closed	indygreg	D3115 tests: `hg init` after resetting HGRCPATH
Closed	indygreg	D3114 tests: work around potential repo incompatibility
Closed	indygreg	D3113 tests: disable test-keyword.t with simple store
Closed	indygreg	D3112 tests: conditionalize test-treemanifest.t
Closed	indygreg	D3111 tests: use unbundle in test-symlink-os-yes-fs-no.py
Closed	indygreg	D3110 tests: disable `hg clone --stream` test with simple store
Closed	indygreg	D3109 tests: use `hg unbundle` in test-setdiscovery.t
Closed	indygreg	D3108 tests: require revlog store for test-verify.t
Closed	indygreg	D3107 tests: conditionalize test-commandserver.t based on extra extensions
Closed	indygreg	D3106 tests: conditionalize test-bundle.t
Closed	indygreg	D3105 tests: require revlog store with test-repair-strip.t
Closed	indygreg	D3104 simplestore: back up index when adding a revision
Closed	indygreg	D3103 tests: disable shallow narrow tests with simple store
Closed	indygreg	D3102 tests: skip test-hgweb-bundle.t if we don't support bundlerepos
Closed	indygreg	D3101 tests: disable test-audit-path.t with simple store
Closed	indygreg	D3100 tests: port test-convert-filemap.t to simple store
Closed	indygreg	D3099 tests: disable test-static-http.t with simple store
Closed	indygreg	D3098 tests: don't drop global hgrc
Closed	indygreg	D3097 verify: allow suppressing warnings about extra files
Closed	indygreg	D3096 tests: conditionalize tests for various repo features
Closed	indygreg	D3095 simplestore: use a custom store for the simple store repo
Closed	indygreg	D3094 verify: drop "revlog" from warning message
Closed	indygreg	D3093 store: make file filtering during walk configurable
Closed	indygreg	D3092 simplestore: shore up lookup errors
Closed	indygreg	D3091 tests: extract dumprevlog tests to own file
Closed	indygreg	D3090 tests: skip test-censor.t when using simple store

Diff 7827

tests/test-audit-path.t

				The simple store doesn't escape paths robustly and can't store paths
				with periods, etc. So much of this test fails with it.
				#require no-reposimplestore

	$ hg init			$ hg init

	audit of .hg			audit of .hg

	$ hg add .hg/00changelog.i			$ hg add .hg/00changelog.i
	abort: path contains illegal component: .hg/00changelog.i			abort: path contains illegal component: .hg/00changelog.i
	[255]			[255]

Diff	ID	Description	Created	Lint	Unit
Base		Base
Diff 1	7683		Apr 4 2018, 9:24 PM	★	★
Diff 2	7827	rHG9984488550ea95b56dea94da749b07595bcc4154	Apr 4 2018, 6:31 PM	★	★