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

login
register
mail settings
Submitter phabricator
Date April 5, 2018, 1:25 a.m.
Message ID <differential-rev-PHID-DREV-xchomklmcw5z2a7j6phq-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/30325/
State Superseded
Headers show

Comments

phabricator - April 5, 2018, 1:25 a.m.
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION 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.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D3101

AFFECTED FILES
  tests/test-audit-path.t

CHANGE DETAILS




To: indygreg, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/test-audit-path.t b/tests/test-audit-path.t
--- a/tests/test-audit-path.t
+++ b/tests/test-audit-path.t
@@ -1,3 +1,7 @@ 
+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
 
 audit of .hg