summaryrefslogtreecommitdiffstats
path: root/test/test_vapi.py
diff options
context:
space:
mode:
authorPaul Vinciguerra <pvinci@vinciconsulting.com>2018-12-06 07:46:13 -0800
committerOle Trøan <otroan@employees.org>2018-12-20 12:43:38 +0000
commitdefde0f87067eb473660794cbd4a2da69fdd191d (patch)
tree41a8e78df46ab9483aaf772622e1ba1e5e2e56c1 /test/test_vapi.py
parent57f170bdf9967e3f8ea6e937a70c7f86187f95a2 (diff)
Tests: Cleanup @skip decorator.
The runnning environment is static as of module load time, so only evalute the conditions once at module load time. Track-by: VPP-1518 Change-Id: I73b0d17ae1ff90789e70307f168d43921829aec8 Signed-off-by: Paul Vinciguerra <pvinci@vinciconsulting.com>
Diffstat (limited to 'test/test_vapi.py')
-rw-r--r--test/test_vapi.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/test/test_vapi.py b/test/test_vapi.py
index 35fb8e795b3..ebd189cd7c8 100644
--- a/test/test_vapi.py
+++ b/test/test_vapi.py
@@ -8,7 +8,7 @@ from framework import VppTestCase, running_extended_tests, \
running_on_centos, VppTestRunner, Worker
-@unittest.skipUnless(running_extended_tests(), "part of extended tests")
+@unittest.skipUnless(running_extended_tests, "part of extended tests")
class VAPITestCase(VppTestCase):
""" VAPI test """
@@ -41,7 +41,7 @@ class VAPITestCase(VppTestCase):
"Timeout! Worker did not finish in %ss" % timeout)
self.assert_equal(worker.result, 0, "Binary test return code")
- @unittest.skipIf(running_on_centos(), "Centos's gcc can't compile our C++")
+ @unittest.skipIf(running_on_centos, "Centos's gcc can't compile our C++")
def test_vapi_cpp(self):
""" run C++ VAPI tests """
var = "TEST_DIR"