diff options
| author |   Christian Trautman <ctrautma@redhat.com> | 2016-04-20 15:30:42 -0400 |
|---|---|---|
| committer | Christian Trautman <ctrautma@redhat.com> | 2016-05-04 09:10:46 -0400 |
| commit | 47875a6f01fa9dc1738d4c4a29ad75e12e251077 (patch) | |
| tree | cc53a18eb151cafb200493057b0bfc89c870c44d /testcases | |
| parent | 25969600ac9508ecc54a25d7b0f628e0713a82a2 (diff) | |
bidirectional value: Fix inconsistency for bidir param
Fixes bi-directional param and enforces string type in
testcase configuration. Modifies all instance where bi
directional value is used to string.
* Changes documentation to specify value type as needed
* Changes traffic defaults to use string for bidir value
* Fixes testcases configuration files to use bidir as str
* Change testcase conf documentation to note enforcement
of string type
* Sets bidir to title format to keep consitency in traffic
profile when passed to traffic generators
* Fix pvp/pvvp check for bidir value
* Raise exception if testcase configuration is not set
as str for bidir value
* Change Xena bi-dir check to string compare instead of
typecasting to boolean to compare.
JIRA: VSPERF-282
Change-Id: I7c91fcd7333a1d1e1371bfd31700c01d7da268b5
Signed-off-by: Christian Trautman <ctrautma@redhat.com>
Diffstat (limited to 'testcases')
| -rw-r--r-- | testcases/testcase.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/testcases/testcase.py b/testcases/testcase.py index f7908af9..5b9ead69 100644 --- a/testcases/testcase.py +++ b/testcases/testcase.py @@ -82,6 +82,10 @@ class TestCase(object): bidirectional = cfg.get('biDirectional', TRAFFIC_DEFAULTS['bidir']) bidirectional = get_test_param('bidirectional', bidirectional) + if not isinstance(bidirectional, str): + raise TypeError( + 'Bi-dir value must be of type string in testcase configuration') + bidirectional = bidirectional.title() # Keep things consistent traffic_type = cfg.get('Traffic Type', TRAFFIC_DEFAULTS['traffic_type']) traffic_type = get_test_param('traffic_type', traffic_type) |