588b5651dc
A number of places tried to use undefined names. This included one powervm test which turned out to not check anything at all (used fake implementation of tested method) and needed to be moved. Make sure that this class of errors causes run_pep8 failure in the future. Change-Id: I82ccb63bbc6f6d2b20ecb7f06b2fc22f8f034a33
27 lines
1.0 KiB
Bash
Executable File
27 lines
1.0 KiB
Bash
Executable File
#!/bin/bash
|
|
|
|
set -e
|
|
# This is used by run_tests.sh and tox.ini
|
|
python tools/hacking.py --doctest
|
|
|
|
# Until all these issues get fixed, ignore.
|
|
PEP8='python tools/hacking.py --ignore=E12,E711,E721,E712,N303,N403,N404'
|
|
|
|
EXCLUDE='--exclude=.venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*'
|
|
EXCLUDE+=',*egg,build,./plugins/xenserver/networking/etc/xensource/scripts'
|
|
EXCLUDE+=',./plugins/xenserver/xenapi/etc/xapi.d/plugins'
|
|
${PEP8} ${EXCLUDE} .
|
|
|
|
SCRIPT_ROOT=$(echo $(cd "$(dirname $0)"; pwd) | sed s/\\/tools//)
|
|
|
|
SCRIPTS_PATH=${SCRIPT_ROOT}/plugins/xenserver/networking/etc/xensource/scripts
|
|
PYTHONPATH=${SCRIPTS_PATH} ${PEP8} ./plugins/xenserver/networking
|
|
|
|
# NOTE(sirp): Also check Dom0 plugins w/o .py extension
|
|
PLUGINS_PATH=${SCRIPT_ROOT}/plugins/xenserver/xenapi/etc/xapi.d/plugins
|
|
PYTHONPATH=${PLUGINS_PATH} ${PEP8} ./plugins/xenserver/xenapi \
|
|
`find plugins/xenserver/xenapi/etc/xapi.d/plugins -type f -perm +111`
|
|
|
|
! pyflakes nova/ | grep "imported but unused\|redefinition of function\|undefined name '" |
|
|
grep -v "undefined name '_'"
|