mirror of
https://github.com/facebook/sapling.git
synced 2024-10-07 07:17:55 +03:00
tests: use the system hg for examining the local repository
Most test scripts use "hg" to interact with a temporary test repository. However a few tests also want to run hg commands to interact with the local repository containing the mercurial source code. Notably, many of the test-check-* tests want to check local files and commit messages. These tests were previously using the version of hg being tested to query the source repository. However, this will fail if the source repository requires extensions or other settings not supported by the version of mercurial being tested. The source repository was typically initially cloned using the system hg installation, so we should use the system hg installation to query it. There was already a helpers-testrepo.sh script designed to help cope with different requirements for the source repository versus the test repositories. However, it only handled the evolve extension. This new behavior works with any extensions that are different between the system installation and the test installation.
This commit is contained in:
parent
b5772c6bc4
commit
55a457de69
@ -1,19 +1,27 @@
|
|||||||
# The test-repo is a live hg repository which may have evolution
|
# Invoke the system hg installation (rather than the local hg version being
|
||||||
# markers created, e.g. when a ~/.hgrc enabled evolution.
|
# tested).
|
||||||
#
|
#
|
||||||
# Tests are run using a custom HGRCPATH, which do not
|
# We want to use the hg version being tested when interacting with the test
|
||||||
# enable evolution markers by default.
|
# repository, and the system hg when interacting with the mercurial source code
|
||||||
|
# repository.
|
||||||
#
|
#
|
||||||
# If test-repo includes evolution markers, and we do not
|
# The mercurial source repository was typically orignally cloned with the
|
||||||
# enable evolution markers, hg will occasionally complain
|
# system mercurial installation, and may require extensions or settings from
|
||||||
# when it notices them, which disrupts tests resulting in
|
# the system installation.
|
||||||
# sporadic failures.
|
syshg () {
|
||||||
#
|
(
|
||||||
# Since we aren't performing any write operations on the
|
syshgenv
|
||||||
# test-repo, there's no harm in telling hg that we support
|
exec hg "$@"
|
||||||
# evolution markers, which is what the following lines
|
)
|
||||||
# for the hgrc file do:
|
}
|
||||||
cat >> $HGRCPATH << EOF
|
|
||||||
[experimental]
|
# Revert the environment so that running "hg" runs the system hg
|
||||||
evolution=createmarkers
|
# rather than the test hg installation.
|
||||||
EOF
|
syshgenv () {
|
||||||
|
PATH="$ORIG_PATH"
|
||||||
|
PYTHONPATH="$ORIG_PYTHONPATH"
|
||||||
|
JYTHONPATH="$ORIG_JYTHONPATH"
|
||||||
|
unset HGRCPATH
|
||||||
|
HGPLAIN=1
|
||||||
|
export HGPLAIN
|
||||||
|
}
|
||||||
|
@ -7,7 +7,7 @@
|
|||||||
New errors are not allowed. Warnings are strongly discouraged.
|
New errors are not allowed. Warnings are strongly discouraged.
|
||||||
(The writing "no-che?k-code" is for not skipping this file when checking.)
|
(The writing "no-che?k-code" is for not skipping this file when checking.)
|
||||||
|
|
||||||
$ hg locate -X contrib/python-zstandard -X hgext/fsmonitor/pywatchman |
|
$ syshg locate -X contrib/python-zstandard -X hgext/fsmonitor/pywatchman |
|
||||||
> sed 's-\\-/-g' | "$check_code" --warnings --per-file=0 - || false
|
> sed 's-\\-/-g' | "$check_code" --warnings --per-file=0 - || false
|
||||||
Skipping i18n/polib.py it has no-che?k-code (glob)
|
Skipping i18n/polib.py it has no-che?k-code (glob)
|
||||||
Skipping mercurial/httpclient/__init__.py it has no-che?k-code (glob)
|
Skipping mercurial/httpclient/__init__.py it has no-che?k-code (glob)
|
||||||
@ -33,7 +33,7 @@ New errors are not allowed. Warnings are strongly discouraged.
|
|||||||
|
|
||||||
Prevent adding new files in the root directory accidentally.
|
Prevent adding new files in the root directory accidentally.
|
||||||
|
|
||||||
$ hg files 'glob:*'
|
$ syshg files 'glob:*'
|
||||||
.editorconfig
|
.editorconfig
|
||||||
.hgignore
|
.hgignore
|
||||||
.hgsigs
|
.hgsigs
|
||||||
|
@ -8,8 +8,8 @@ Go back in the hg repo
|
|||||||
|
|
||||||
$ cd $TESTDIR/..
|
$ cd $TESTDIR/..
|
||||||
|
|
||||||
$ for node in `hg log --rev 'not public() and ::. and not desc("# no-check-commit")' --template '{node|short}\n'`; do
|
$ for node in `syshg log --rev 'not public() and ::. and not desc("# no-check-commit")' --template '{node|short}\n'`; do
|
||||||
> hg export --git $node | contrib/check-commit > ${TESTTMP}/check-commit.out
|
> syshg export --git $node | contrib/check-commit > ${TESTTMP}/check-commit.out
|
||||||
> if [ $? -ne 0 ]; then
|
> if [ $? -ne 0 ]; then
|
||||||
> echo "Revision $node does not comply with rules"
|
> echo "Revision $node does not comply with rules"
|
||||||
> echo '------------------------------------------------------'
|
> echo '------------------------------------------------------'
|
||||||
|
@ -31,7 +31,7 @@ Sanity check check-config.py
|
|||||||
|
|
||||||
New errors are not allowed. Warnings are strongly discouraged.
|
New errors are not allowed. Warnings are strongly discouraged.
|
||||||
|
|
||||||
$ hg files "set:(**.py or **.txt) - tests/**" | sed 's|\\|/|g' |
|
$ syshg files "set:(**.py or **.txt) - tests/**" | sed 's|\\|/|g' |
|
||||||
> $PYTHON contrib/check-config.py
|
> $PYTHON contrib/check-config.py
|
||||||
limit = ui.configwith(fraction, 'profiling', 'showmin', 0.05)
|
limit = ui.configwith(fraction, 'profiling', 'showmin', 0.05)
|
||||||
|
|
||||||
|
@ -5,20 +5,20 @@
|
|||||||
|
|
||||||
look for python scripts without the execute bit
|
look for python scripts without the execute bit
|
||||||
|
|
||||||
$ hg files 'set:**.py and not exec() and grep(r"^#!.*?python")'
|
$ syshg files 'set:**.py and not exec() and grep(r"^#!.*?python")'
|
||||||
[1]
|
[1]
|
||||||
|
|
||||||
look for python scripts with execute bit but not shebang
|
look for python scripts with execute bit but not shebang
|
||||||
|
|
||||||
$ hg files 'set:**.py and exec() and not grep(r"^#!.*?python")'
|
$ syshg files 'set:**.py and exec() and not grep(r"^#!.*?python")'
|
||||||
[1]
|
[1]
|
||||||
|
|
||||||
look for shell scripts with execute bit but not shebang
|
look for shell scripts with execute bit but not shebang
|
||||||
|
|
||||||
$ hg files 'set:**.sh and exec() and not grep(r"^#!.*(ba)?sh")'
|
$ syshg files 'set:**.sh and exec() and not grep(r"^#!.*(ba)?sh")'
|
||||||
[1]
|
[1]
|
||||||
|
|
||||||
look for non scripts with no shebang
|
look for non scripts with no shebang
|
||||||
|
|
||||||
$ hg files 'set:exec() and not **.sh and not **.py and not grep(r"^#!")'
|
$ syshg files 'set:exec() and not **.sh and not **.py and not grep(r"^#!")'
|
||||||
[1]
|
[1]
|
||||||
|
@ -23,6 +23,7 @@
|
|||||||
Check if ":hg:`help TOPIC`" is valid:
|
Check if ":hg:`help TOPIC`" is valid:
|
||||||
(use "xargs -n1 -t" to see which help commands are executed)
|
(use "xargs -n1 -t" to see which help commands are executed)
|
||||||
|
|
||||||
$ hg files 'glob:{hgdemandimport,hgext,mercurial}/**/*.py' | sed 's|\\|/|g' \
|
$ syshg files 'glob:{hgdemandimport,hgext,mercurial}/**/*.py' \
|
||||||
|
> | sed 's|\\|/|g' \
|
||||||
> | xargs $PYTHON "$TESTTMP/scanhelptopics.py" \
|
> | xargs $PYTHON "$TESTTMP/scanhelptopics.py" \
|
||||||
> | xargs -n1 hg help > /dev/null
|
> | xargs -n1 hg help > /dev/null
|
||||||
|
@ -14,7 +14,7 @@ these may expose other cycles.
|
|||||||
Known-bad files are excluded by -X as some of them would produce unstable
|
Known-bad files are excluded by -X as some of them would produce unstable
|
||||||
outputs, which should be fixed later.
|
outputs, which should be fixed later.
|
||||||
|
|
||||||
$ hg locate 'set:**.py or grep(r"^#!.*?python")' \
|
$ syshg locate 'set:**.py or grep(r"^#!.*?python")' \
|
||||||
> 'tests/**.t' \
|
> 'tests/**.t' \
|
||||||
> -X contrib/debugshell.py \
|
> -X contrib/debugshell.py \
|
||||||
> -X contrib/python-zstandard/ \
|
> -X contrib/python-zstandard/ \
|
||||||
|
@ -3,7 +3,7 @@
|
|||||||
$ . "$TESTDIR/helpers-testrepo.sh"
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
$ cd "$TESTDIR"/..
|
$ cd "$TESTDIR"/..
|
||||||
|
|
||||||
$ hg files 'set:(**.py)' | sed 's|\\|/|g' | xargs $PYTHON contrib/check-py3-compat.py
|
$ syshg files 'set:(**.py)' | sed 's|\\|/|g' | xargs $PYTHON contrib/check-py3-compat.py
|
||||||
contrib/python-zstandard/setup.py not using absolute_import
|
contrib/python-zstandard/setup.py not using absolute_import
|
||||||
contrib/python-zstandard/setup_zstd.py not using absolute_import
|
contrib/python-zstandard/setup_zstd.py not using absolute_import
|
||||||
contrib/python-zstandard/tests/common.py not using absolute_import
|
contrib/python-zstandard/tests/common.py not using absolute_import
|
||||||
@ -22,7 +22,7 @@
|
|||||||
tests/test-demandimport.py not using absolute_import
|
tests/test-demandimport.py not using absolute_import
|
||||||
|
|
||||||
#if py3exe
|
#if py3exe
|
||||||
$ hg files 'set:(**.py) - grep(pygments)' -X hgext/fsmonitor/pywatchman \
|
$ syshg files 'set:(**.py) - grep(pygments)' -X hgext/fsmonitor/pywatchman \
|
||||||
> | sed 's|\\|/|g' | xargs $PYTHON3 contrib/check-py3-compat.py \
|
> | sed 's|\\|/|g' | xargs $PYTHON3 contrib/check-py3-compat.py \
|
||||||
> | sed 's/[0-9][0-9]*)$/*)/'
|
> | sed 's/[0-9][0-9]*)$/*)/'
|
||||||
hgext/convert/transport.py: error importing: <*Error> No module named 'svn.client' (error at transport.py:*) (glob)
|
hgext/convert/transport.py: error importing: <*Error> No module named 'svn.client' (error at transport.py:*) (glob)
|
||||||
@ -38,7 +38,7 @@
|
|||||||
#endif
|
#endif
|
||||||
|
|
||||||
#if py3exe py3pygments
|
#if py3exe py3pygments
|
||||||
$ hg files 'set:(**.py) and grep(pygments)' | sed 's|\\|/|g' \
|
$ syshg files 'set:(**.py) and grep(pygments)' | sed 's|\\|/|g' \
|
||||||
> | xargs $PYTHON3 contrib/check-py3-compat.py \
|
> | xargs $PYTHON3 contrib/check-py3-compat.py \
|
||||||
> | sed 's/[0-9][0-9]*)$/*)/'
|
> | sed 's/[0-9][0-9]*)$/*)/'
|
||||||
#endif
|
#endif
|
||||||
|
@ -6,7 +6,8 @@
|
|||||||
run pyflakes on all tracked files ending in .py or without a file ending
|
run pyflakes on all tracked files ending in .py or without a file ending
|
||||||
(skipping binary file random-seed)
|
(skipping binary file random-seed)
|
||||||
|
|
||||||
$ hg locate 'set:**.py or grep("^#!.*python")' -X hgext/fsmonitor/pywatchman \
|
$ syshg locate 'set:**.py or grep("^#!.*python")' \
|
||||||
|
> -X hgext/fsmonitor/pywatchman \
|
||||||
> -X mercurial/pycompat.py -X contrib/python-zstandard \
|
> -X mercurial/pycompat.py -X contrib/python-zstandard \
|
||||||
> 2>/dev/null \
|
> 2>/dev/null \
|
||||||
> | xargs pyflakes 2>/dev/null | "$TESTDIR/filterpyflakes.py"
|
> | xargs pyflakes 2>/dev/null | "$TESTDIR/filterpyflakes.py"
|
||||||
|
@ -5,11 +5,11 @@
|
|||||||
|
|
||||||
look for python scripts that do not use /usr/bin/env
|
look for python scripts that do not use /usr/bin/env
|
||||||
|
|
||||||
$ hg files 'set:grep(r"^#!.*?python") and not grep(r"^#!/usr/bi{1}n/env python") - **/*.t'
|
$ syshg files 'set:grep(r"^#!.*?python") and not grep(r"^#!/usr/bi{1}n/env python") - **/*.t'
|
||||||
[1]
|
[1]
|
||||||
|
|
||||||
In tests, enforce $PYTHON and *not* /usr/bin/env python or similar:
|
In tests, enforce $PYTHON and *not* /usr/bin/env python or similar:
|
||||||
$ hg files 'set:grep(r"#!.*?python") and **/*.t' \
|
$ syshg files 'set:grep(r"#!.*?python") and **/*.t' \
|
||||||
> -X tests/test-check-execute.t \
|
> -X tests/test-check-execute.t \
|
||||||
> -X tests/test-check-module-imports.t \
|
> -X tests/test-check-module-imports.t \
|
||||||
> -X tests/test-check-pyflakes.t \
|
> -X tests/test-check-pyflakes.t \
|
||||||
@ -21,5 +21,5 @@ contain Python but don't end in .py - please avoid adding more.
|
|||||||
|
|
||||||
look for shell scripts that do not use /bin/sh
|
look for shell scripts that do not use /bin/sh
|
||||||
|
|
||||||
$ hg files 'set:grep(r"^#!.*/bi{1}n/sh") and not grep(r"^#!/bi{1}n/sh")'
|
$ syshg files 'set:grep(r"^#!.*/bi{1}n/sh") and not grep(r"^#!/bi{1}n/sh")'
|
||||||
[1]
|
[1]
|
||||||
|
@ -170,6 +170,6 @@ Check perf.py for historical portability
|
|||||||
|
|
||||||
$ cd "$TESTDIR/.."
|
$ cd "$TESTDIR/.."
|
||||||
|
|
||||||
$ (hg files -r 1.2 glob:mercurial/*.c glob:mercurial/*.py;
|
$ (syshg files -r 1.2 glob:mercurial/*.c glob:mercurial/*.py;
|
||||||
> hg files -r tip glob:mercurial/*.c glob:mercurial/*.py) |
|
> syshg files -r tip glob:mercurial/*.c glob:mercurial/*.py) |
|
||||||
> "$TESTDIR"/check-perf-code.py contrib/perf.py
|
> "$TESTDIR"/check-perf-code.py contrib/perf.py
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
#require test-repo slow debhelper
|
#require test-repo slow debhelper
|
||||||
|
|
||||||
$ . "$TESTDIR/helpers-testrepo.sh"
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
|
$ syshgenv
|
||||||
|
|
||||||
Ensure debuild doesn't run the testsuite, as that could get silly.
|
Ensure debuild doesn't run the testsuite, as that could get silly.
|
||||||
$ DEB_BUILD_OPTIONS=nocheck
|
$ DEB_BUILD_OPTIONS=nocheck
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
#require test-repo slow docker
|
#require test-repo slow docker
|
||||||
|
|
||||||
$ . "$TESTDIR/helpers-testrepo.sh"
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
|
$ syshgenv
|
||||||
|
|
||||||
Ensure debuild doesn't run the testsuite, as that could get silly.
|
Ensure debuild doesn't run the testsuite, as that could get silly.
|
||||||
$ DEB_BUILD_OPTIONS=nocheck
|
$ DEB_BUILD_OPTIONS=nocheck
|
||||||
|
@ -1,3 +1,5 @@
|
|||||||
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
|
|
||||||
Testing that hghave does not crash when checking features
|
Testing that hghave does not crash when checking features
|
||||||
|
|
||||||
$ hghave --test-features 2>/dev/null
|
$ hghave --test-features 2>/dev/null
|
||||||
@ -18,7 +20,10 @@ Testing hghave extensibility for third party tools
|
|||||||
> $ echo foo
|
> $ echo foo
|
||||||
> foo
|
> foo
|
||||||
> EOF
|
> EOF
|
||||||
$ run-tests.py $HGTEST_RUN_TESTS_PURE test-hghaveaddon.t
|
$ ( \
|
||||||
|
> syshgenv; \
|
||||||
|
> $TESTDIR/run-tests.py $HGTEST_RUN_TESTS_PURE test-hghaveaddon.t \
|
||||||
|
> )
|
||||||
.
|
.
|
||||||
# Ran 1 tests, 0 skipped, 0 failed.
|
# Ran 1 tests, 0 skipped, 0 failed.
|
||||||
|
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
#require test-repo
|
#require test-repo
|
||||||
|
|
||||||
$ . "$TESTDIR/helpers-testrepo.sh"
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
|
$ syshgenv
|
||||||
$ import_checker="$TESTDIR"/../contrib/import-checker.py
|
$ import_checker="$TESTDIR"/../contrib/import-checker.py
|
||||||
|
|
||||||
Run the doctests from the import checker, and make sure
|
Run the doctests from the import checker, and make sure
|
||||||
|
@ -159,7 +159,7 @@ path variables are expanded (~ is the same as $TESTTMP)
|
|||||||
> print(' %s' % f)
|
> print(' %s' % f)
|
||||||
> EOF
|
> EOF
|
||||||
|
|
||||||
$ $PYTHON wixxml.py help
|
$ (syshgenv; $PYTHON wixxml.py help )
|
||||||
Not installed:
|
Not installed:
|
||||||
help/common.txt
|
help/common.txt
|
||||||
help/hg-ssh.8.txt
|
help/hg-ssh.8.txt
|
||||||
@ -168,7 +168,7 @@ path variables are expanded (~ is the same as $TESTTMP)
|
|||||||
help/hgrc.5.txt
|
help/hgrc.5.txt
|
||||||
Not tracked:
|
Not tracked:
|
||||||
|
|
||||||
$ $PYTHON wixxml.py templates
|
$ ( syshgenv; $PYTHON wixxml.py templates )
|
||||||
Not installed:
|
Not installed:
|
||||||
Not tracked:
|
Not tracked:
|
||||||
|
|
||||||
|
@ -1,6 +1,7 @@
|
|||||||
#require test-repo slow osx osxpackaging
|
#require test-repo slow osx osxpackaging
|
||||||
|
|
||||||
$ . "$TESTDIR/helpers-testrepo.sh"
|
$ . "$TESTDIR/helpers-testrepo.sh"
|
||||||
|
$ syshgenv
|
||||||
|
|
||||||
$ OUTPUTDIR="`pwd`"
|
$ OUTPUTDIR="`pwd`"
|
||||||
$ export OUTPUTDIR
|
$ export OUTPUTDIR
|
||||||
|
Loading…
Reference in New Issue
Block a user