#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]