D5041: py3: add a b'' prefix in tests/test-requires.t

pulkit (Pulkit Goyal) phabricator at mercurial-scm.org
Sat Oct 13 02:22:51 UTC 2018


pulkit created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  This makes the test pass on Python 3.
  
  1. skip-blame because just b'' prefix.

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  contrib/python3-whitelist
  tests/test-requires.t

CHANGE DETAILS

diff --git a/tests/test-requires.t b/tests/test-requires.t
--- a/tests/test-requires.t
+++ b/tests/test-requires.t
@@ -38,7 +38,7 @@
   >     for name, module in extensions.extensions(ui):
   >         if __name__ == module.__name__:
   >             # support specific feature locally
-  >             supported |= {'featuresetup-test'}
+  >             supported |= {b'featuresetup-test'}
   >             return
   > def uisetup(ui):
   >     localrepo.featuresetupfuncs.add(featuresetup)
diff --git a/contrib/python3-whitelist b/contrib/python3-whitelist
--- a/contrib/python3-whitelist
+++ b/contrib/python3-whitelist
@@ -492,6 +492,7 @@
 test-rename.t
 test-repair-strip.t
 test-repo-compengines.t
+test-requires.t
 test-resolve.t
 test-revert-flags.t
 test-revert-interactive.t



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


More information about the Mercurial-devel mailing list