e5bd64ff26
E711 comparison to False should be 'if cond is False:' or 'if not cond:'
comparison to None should be 'if cond is None:' or 'if not cond:'
E712 comparison to True should be 'if cond is True:' or 'if cond:'
Most violations were in DB queries. Replace as follows:
False -> sqlalchemy.sql.false()
None -> sqlalchemy.sql.null()
True -> sqlalchemy.sql.true()
Change-Id: Iff54747b70f504d5466cfdc6e2ec4d7a0f9ddb7c
Closes-bug: #1305377
50 lines
1.4 KiB
INI
50 lines
1.4 KiB
INI
[tox]
|
|
envlist = py26,py27,py33,pep8
|
|
minversion = 1.6
|
|
skipsdist = True
|
|
|
|
[testenv]
|
|
setenv = VIRTUAL_ENV={envdir}
|
|
usedevelop = True
|
|
install_command = pip install -U {opts} {packages}
|
|
deps = -r{toxinidir}/requirements.txt
|
|
-r{toxinidir}/test-requirements.txt
|
|
setuptools_git>=0.4
|
|
commands =
|
|
python -m neutron.openstack.common.lockutils python setup.py testr --slowest --testr-args='{posargs}'
|
|
|
|
[testenv:functional]
|
|
setenv = OS_TEST_PATH=./neutron/tests/functional
|
|
commands =
|
|
python setup.py testr --slowest --testr-args='{posargs}'
|
|
|
|
[tox:jenkins]
|
|
sitepackages = True
|
|
downloadcache = ~/cache/pip
|
|
|
|
[testenv:pep8]
|
|
commands =
|
|
flake8
|
|
neutron-db-manage check_migration
|
|
bash -c "find neutron -type f -regex '.*\.pot?' -print0|xargs -0 -n 1 msgfmt --check-format -o /dev/null"
|
|
whitelist_externals = bash
|
|
|
|
[testenv:i18n]
|
|
commands = python ./tools/check_i18n.py ./neutron ./tools/i18n_cfg.py
|
|
|
|
[testenv:cover]
|
|
commands =
|
|
python -m neutron.openstack.common.lockutils python setup.py testr --coverage --testr-args='{posargs}'
|
|
|
|
[testenv:venv]
|
|
commands = {posargs}
|
|
|
|
[flake8]
|
|
# E125 continuation line does not distinguish itself from next logical line
|
|
# H302 import only modules
|
|
# TODO(marun) H404 multi line docstring should start with a summary
|
|
ignore = E125,H302,H404
|
|
show-source = true
|
|
builtins = _
|
|
exclude = .venv,.git,.tox,dist,doc,*openstack/common*,*lib/python*,*egg,build,tools
|