Rate python files via pylint

It higlights programming errors which are only printed for the time
being. It adds pylint in test-requirements.txt as it should have
been for coverage, mock, etc.

At least all pylint errors should be fixed.

Kilo and Newton are out of scope as deprecated.

Issue-ID: MULTICLOUD-416

Change-Id: I35f2cfec0327ba163e796b05bdff5edf0246de09
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

diff --git a/ocata/test-requirements.txt b/ocata/test-requirements.txt
new file mode 100644 (file)
index 0000000..97044b5
--- /dev/null
+++ b/ocata/test-requirements.txt
@@ -0,0 +1 @@
+pylint # GPLv2

diff --git a/ocata/tox.ini b/ocata/tox.ini
index b882540..79e0a15 100644 (file)
--- a/ocata/tox.ini
+++ b/ocata/tox.ini
@@ -13,7 +13,7 @@
 # limitations under the License.
 
 [tox]
-envlist = py27,pep8,cov
+envlist = py27,pep8,cov,pylint
 skipsdist = true
 
 [tox:jenkins]
@@ -27,8 +27,10 @@ max-complexity = 27
 [testenv]
 setenv =
     PYTHONPATH = {toxinidir}/../share
-deps = -r{toxinidir}/requirements.txt
-commands = 
+deps =
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
+commands =
     coverage run --branch manage.py test ocata
     coverage report --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*" --fail-under=30
 
@@ -39,3 +41,8 @@ commands=flake8
 [testenv:cov]
 commands = coverage xml --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*, *site-packages*"
 
+[testenv:pylint]
+whitelist_externals = bash
+commands =
+  bash -c "\
+    pylint -f parseable --reports=y ocata | tee pylint.out"

diff --git a/pike/test-requirements.txt b/pike/test-requirements.txt
new file mode 100644 (file)
index 0000000..97044b5
--- /dev/null
+++ b/pike/test-requirements.txt
@@ -0,0 +1 @@
+pylint # GPLv2

diff --git a/pike/tox.ini b/pike/tox.ini
index cd5327a..c5e9e5c 100644 (file)
--- a/pike/tox.ini
+++ b/pike/tox.ini
@@ -13,7 +13,7 @@
 # limitations under the License.
 
 [tox]
-envlist = py27,pep8,cov
+envlist = py27,pep8,cov,pylint
 skipsdist = true
 
 [tox:jenkins]
@@ -27,7 +27,9 @@ max-complexity = 27
 [testenv]
 setenv =
     PYTHONPATH = {toxinidir}/../share
-deps = -r{toxinidir}/requirements.txt
+deps =
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
 commands =
     coverage run --branch manage.py test pike
     coverage report --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*" --fail-under=30
@@ -39,3 +41,8 @@ commands=flake8
 [testenv:cov]
 commands = coverage xml --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*, *site-packages*"
 
+[testenv:pylint]
+whitelist_externals = bash
+commands =
+  bash -c "\
+    pylint -f parseable --reports=y pike | tee pylint.out"

diff --git a/windriver/test-requirements.txt b/windriver/test-requirements.txt
new file mode 100644 (file)
index 0000000..97044b5
--- /dev/null
+++ b/windriver/test-requirements.txt
@@ -0,0 +1 @@
+pylint # GPLv2

diff --git a/windriver/tox.ini b/windriver/tox.ini
index dd34e47..e5a65d3 100644 (file)
--- a/windriver/tox.ini
+++ b/windriver/tox.ini
@@ -1,5 +1,5 @@
 [tox]
-envlist = py27,cov
+envlist = py27,cov,pylint
 skipsdist = true
 
 [tox:jenkins]
@@ -13,7 +13,9 @@ max-complexity = 27
 [testenv]
 setenv =
     PYTHONPATH = {toxinidir}/../share
-deps = -r{toxinidir}/requirements.txt
+deps =
+    -r{toxinidir}/requirements.txt
+    -r{toxinidir}/test-requirements.txt
 commands =
     coverage run --branch manage.py test titanium_cloud
     coverage report --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*,*starlingx_base*" --fail-under=30
@@ -25,3 +27,8 @@ commands=flake8
 [testenv:cov]
 commands = coverage xml --omit="./venv-tox/*,./.tox/*,*tests*,*__init__.py,*newton_base*,*common*, *site-packages*"
 
+[testenv:pylint]
+whitelist_externals = bash
+commands =
+  bash -c "\
+    pylint -f parseable --reports=y titanium_cloud | tee pylint.out"