diff options
| author |   Romanos Skiadas <rski@intracom-telecom.com> | 2016-12-13 10:55:35 +0200 |
|---|---|---|
| committer | Romanos Skiadas <rski@intracom-telecom.com> | 2016-12-13 15:16:57 +0200 |
| commit | 2417d16aa6767a4c1bb6f28104076bb4b11d5e10 (patch) | |
| tree | 551a546a11e6ecf83b48f3de96184aad1110b0fa /test/functest/testcase_1.py | |
| parent | 2445ba0a0325611e0b927ecc7ed4d361426e5fc4 (diff) | |
Encapsulate some functionality in Results class
- Move adding actions taken during tests to summary in a method in Results
- Provide add_success/failure methods that hide the column size manipulation
Change-Id: If7416f3f9399125fe21632328a49af285a4b6cc6
Signed-off-by: Romanos Skiadas <rski@intracom-telecom.com>
Diffstat (limited to 'test/functest/testcase_1.py')
| -rw-r--r-- | test/functest/testcase_1.py | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/test/functest/testcase_1.py b/test/functest/testcase_1.py index eba39bb..a84478f 100644 --- a/test/functest/testcase_1.py +++ b/test/functest/testcase_1.py @@ -135,8 +135,7 @@ def main(): vm_1_ip = vm_1.networks.itervalues().next()[0] msg = ("Create VPN with eRT<>iRT") - logger.info(msg) - results.add_to_summary(1, msg) + results.record_action(msg) vpn_name = "sdnvpn-" + str(randint(100000, 999999)) kwargs = { "import_targets": TESTCASE_CONFIG.targets1, @@ -149,8 +148,7 @@ def main(): logger.debug("VPN created details: %s" % bgpvpn) msg = ("Associate network '%s' to the VPN." % TESTCASE_CONFIG.net_1_name) - logger.info(msg) - results.add_to_summary(1, msg) + results.record_action(msg) results.add_to_summary(0, "-") os_utils.create_network_association( @@ -176,9 +174,8 @@ def main(): expected="FAIL", timeout=30) msg = ("Associate network '%s' to the VPN." % TESTCASE_CONFIG.net_2_name) - logger.info(msg) results.add_to_summary(0, "-") - results.add_to_summary(1, msg) + results.record_action(msg) results.add_to_summary(0, "-") os_utils.create_network_association( neutron_client, bgpvpn_id, network_2_id) @@ -203,9 +200,8 @@ def main(): expected="FAIL", timeout=30) msg = ("Update VPN with eRT=iRT ...") - logger.info(msg) results.add_to_summary(0, "-") - results.add_to_summary(1, msg) + results.record_action(msg) results.add_to_summary(0, "-") kwargs = {"import_targets": TESTCASE_CONFIG.targets1, "export_targets": TESTCASE_CONFIG.targets1, |