diff options
author | Mark Beierl <mark.beierl@emc.com> | 2016-07-13 23:26:00 -0400 |
---|---|---|
committer | Mark Beierl <mark.beierl@emc.com> | 2016-07-14 20:09:55 +0000 |
commit | ce3f927c2bc98eaf124e96f99cbcc33fbebabeed (patch) | |
tree | a59151955913ec2131e2f55fcca3b1cf974096ac /ci/verify.sh | |
parent | 94550f3b427ec525771ac0d6a9d884e4e69f838b (diff) |
Enforce Style Guidelines
Update to fail verify if pep8 compliance not met.
Revert back to Coverage 4.1 to show code coverage
report in Jenkins again
Change-Id: Icb467451f101942255c7944ad796497609dcb07e
Signed-off-by: Mark Beierl <mark.beierl@emc.com>
Diffstat (limited to 'ci/verify.sh')
-rwxr-xr-x | ci/verify.sh | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/ci/verify.sh b/ci/verify.sh index d309319..89b1406 100755 --- a/ci/verify.sh +++ b/ci/verify.sh @@ -21,7 +21,7 @@ source $WORKSPACE/storperf_venv/bin/activate pip install setuptools pip install autoflake==0.6.6 pip install autopep8==1.2.2 -pip install coverage==3.4 +pip install coverage==4.1 pip install flask==0.10 pip install flask-restful==0.3.5 pip install flask-restful-swagger==0.19 @@ -29,6 +29,7 @@ pip install flask-swagger==0.2.12 pip install funcsigs==0.4 pip install flake8==2.5.4 pip install html2text==2016.1.8 +pip install matplotlib==1.3.1 pip install mock==1.3.0 pip install nose==1.3.7 pip install python-cinderclient==1.6.0 @@ -45,6 +46,8 @@ python ci/setup.py develop flake8 storperf +flake8rc=$? + nosetests --with-xunit \ --with-coverage \ --cover-package=storperf\ @@ -54,4 +57,10 @@ rc=$? deactivate -exit $rc +if [ $flake8rc -ne 0 ] +then + echo "Formatting did not meet guidelines" + exit $flake8rc +fi + +exit $rc
\ No newline at end of file |