Mercurial > hg
view tests/test-imports-checker.t @ 51402:2e10ddbb9faa
phases: check secret presence the right way during discovery
There is an official function for this, lets use it.
This will prevent the code to break in the future while we refactor the phase
code.
author | Pierre-Yves David <pierre-yves.david@octobus.net> |
---|---|
date | Tue, 20 Feb 2024 17:17:54 +0100 |
parents | 42d2b31cee0b |
children |
line wrap: on
line source
#require test-repo $ . "$TESTDIR/helpers-testrepo.sh" $ testrepohgenv $ import_checker="$TESTDIR"/../contrib/import-checker.py Run the doctests from the import checker, and make sure it's working correctly. $ TERM=dumb $ export TERM $ "$PYTHON" -m doctest $import_checker Run additional tests for the import checker $ mkdir testpackage $ touch testpackage/__init__.py $ cat > testpackage/multiple.py << EOF > import os, sys > EOF $ cat > testpackage/unsorted.py << EOF > import sys > import os > EOF $ cat > testpackage/stdafterlocal.py << EOF > from . import unsorted > import os > EOF $ cat > testpackage/requirerelative.py << EOF > import testpackage.unsorted > EOF $ cat > testpackage/importalias.py << EOF > import ui > EOF $ cat > testpackage/relativestdlib.py << EOF > from .. import os > EOF $ cat > testpackage/stdlibfrom.py << EOF > from collections import abc > EOF $ cat > testpackage/symbolimport.py << EOF > from .unsorted import foo > EOF $ cat > testpackage/latesymbolimport.py << EOF > from . import unsorted > from mercurial.node import hex > EOF $ cat > testpackage/multiplegroups.py << EOF > from . import unsorted > from . import more > EOF $ mkdir testpackage/subpackage $ cat > testpackage/subpackage/levelpriority.py << EOF > from . import foo > from .. import parent > EOF $ touch testpackage/subpackage/foo.py $ cat > testpackage/subpackage/__init__.py << EOF > from . import levelpriority # should not cause cycle > EOF $ cat > testpackage/subpackage/localimport.py << EOF > from . import foo > def bar(): > # should not cause "higher-level import should come first" > from .. import unsorted > # but other errors should be detected > from .. import more > import testpackage.subpackage.levelpriority > EOF $ cat > testpackage/importmodulefromsub.py << EOF > from .subpackage import foo # not a "direct symbol import" > EOF $ cat > testpackage/importsymbolfromsub.py << EOF > from .subpackage import foo, nonmodule > EOF $ cat > testpackage/sortedentries.py << EOF > from . import ( > foo, > bar, > ) > EOF $ cat > testpackage/importfromalias.py << EOF > from . import ui > EOF $ cat > testpackage/importfromrelative.py << EOF > from testpackage.unsorted import foo > EOF $ mkdir testpackage2 $ touch testpackage2/__init__.py $ cat > testpackage2/latesymbolimport.py << EOF > from testpackage import unsorted > from mercurial.node import hex > EOF # Shadowing a stdlib module to test "relative import of stdlib module" is # allowed if the module is also being checked $ mkdir email $ touch email/__init__.py $ touch email/errors.py $ cat > email/utils.py << EOF > from . import errors > EOF $ "$PYTHON" "$import_checker" testpackage*/*.py testpackage/subpackage/*.py \ > email/*.py testpackage/importalias.py:1: ui module must be "as" aliased to uimod testpackage/importfromalias.py:1: ui from testpackage must be "as" aliased to uimod testpackage/importfromrelative.py:1: import should be relative: testpackage.unsorted testpackage/importfromrelative.py:1: direct symbol import foo from testpackage.unsorted testpackage/importsymbolfromsub.py:1: direct symbol import nonmodule from testpackage.subpackage testpackage/latesymbolimport.py:2: symbol import follows non-symbol import: mercurial.node testpackage/multiple.py:1: multiple imported names: os, sys testpackage/multiplegroups.py:2: multiple "from . import" statements testpackage/relativestdlib.py:1: relative import of stdlib module testpackage/requirerelative.py:1: import should be relative: testpackage.unsorted testpackage/sortedentries.py:1: imports from testpackage not lexically sorted: bar < foo testpackage/stdafterlocal.py:2: stdlib import "os" follows local import: testpackage testpackage/stdlibfrom.py:1: direct symbol import abc from collections testpackage/subpackage/levelpriority.py:2: higher-level import should come first: testpackage testpackage/subpackage/localimport.py:6: multiple "from .. import" statements testpackage/subpackage/localimport.py:7: import should be relative: testpackage.subpackage.levelpriority testpackage/symbolimport.py:1: direct symbol import foo from testpackage.unsorted testpackage/unsorted.py:2: imports not lexically sorted: os < sys testpackage2/latesymbolimport.py:2: symbol import follows non-symbol import: mercurial.node [1]