diff options
| author |   Klement Sekera <ksekera@cisco.com> | 2017-01-17 10:37:05 +0100 |
|---|---|---|
| committer |   Damjan Marion <dmarion.lists@gmail.com> | 2017-01-17 18:35:21 +0000 |
| commit | 72715ee4e2bba6fd4ce5c077bca68bb08ac729a0 (patch) | |
| tree | 0d12a2ab578ec74b4f76bd6e407f948b46a4628c | |
| parent | 46f133d522b0f247f3b395ad39a12f6898cb67eb (diff) | |
make test: add checkstyle target
Change-Id: I59d3c3bc77474c96e1d6fa51811c1b13fb9a6c5b
Signed-off-by: Klement Sekera <ksekera@cisco.com>
| -rw-r--r-- | Makefile | 4 | ||||
| -rw-r--r-- | test/Makefile | 17 | ||||
| -rw-r--r-- | test/test_bfd.py | 2 | ||||
| -rw-r--r-- | test/util.py | 3 |
4 files changed, 23 insertions, 3 deletions
@@ -96,6 +96,7 @@ help: @echo " test-wipe-doc - wipe documentation for test framework" @echo " test-cov - generate code coverage report for test framework" @echo " test-wipe-cov - wipe code coverage report for test framework" + @echo " test-checkstyle - check PEP8 compliance for test framework" @echo "" @echo "Make Arguments:" @echo " V=[0|1] - set build verbosity level" @@ -239,6 +240,9 @@ test-cov: bootstrap test-wipe-cov: @make -C test wipe-cov +test-checkstyle: + @make -C test checkstyle + retest: $(call test,vpp_lite,vpp_lite,retest) diff --git a/test/Makefile b/test/Makefile index 543fe913c55..db64ad93e07 100644 --- a/test/Makefile +++ b/test/Makefile @@ -77,6 +77,20 @@ cov: wipe-cov reset verify-python-path $(PAPI_INSTALL_DONE) wipe-cov: wipe @rm -rf $(BUILD_COV_DIR) +.PHONY: checkstyle +checkstyle: verify-python-path + @virtualenv $(PYTHON_VENV_PATH) -p python2.7 + @bash -c "source $(PYTHON_VENV_PATH)/bin/activate && pip install $(PYTHON_DEPENDS) pep8" + @bash -c "source $(PYTHON_VENV_PATH)/bin/activate &&\ + pep8 --show-source -v $(WS_ROOT)/test/*.py ||\ + (echo \"*******************************************************************\" &&\ + echo \"* Test framework PEP8 compliance check FAILED \" &&\ + echo \"*******************************************************************\" &&\ + false)" + @echo "*******************************************************************" + @echo "* Test framework PEP8 compliance check passed" + @echo "*******************************************************************" + help: @echo "Running tests:" @echo "" @@ -106,3 +120,6 @@ help: @echo " test-cov - generate code coverage report for test framework" @echo " test-wipe-cov - wipe code coverage report for test framework" @echo "" + @echo "Verifying code-style" + @echo " test-checkstyle - check PEP8 compliance" + @echo "" diff --git a/test/test_bfd.py b/test/test_bfd.py index b6222524a4e..b783224732f 100644 --- a/test/test_bfd.py +++ b/test/test_bfd.py @@ -27,8 +27,6 @@ class BFDAPITestCase(VppTestCase): super(BFDAPITestCase, cls).tearDownClass() raise - - def test_add_bfd(self): """ create a BFD session """ session = VppBFDUDPSession(self, self.pg0, self.pg0.remote_ip4) diff --git a/test/util.py b/test/util.py index ae887a54543..7989360216a 100644 --- a/test/util.py +++ b/test/util.py @@ -41,7 +41,8 @@ def ip4_range(ip4, s, e): def ip4n_range(ip4n, s, e): ip4 = socket.inet_ntop(socket.AF_INET, ip4n) - return (socket.inet_pton(socket.AF_INET, ip) for ip in ip4_range(ip4, s, e)) + return (socket.inet_pton(socket.AF_INET, ip) + for ip in ip4_range(ip4, s, e)) class NumericConstant(object): |