summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorrandyl <r.levensalor@cablelabs.com>2017-03-06 17:38:46 -0700
committerrandyl <r.levensalor@cablelabs.com>2017-03-06 17:38:46 -0700
commitf4b8f02ebc859f2ee86dd565d60b302d879b27bc (patch)
treea93e92b22f07c2ff66688e32621c57c6d60f204a
parentebf6b1653e731c757f56a26a67096fa8bed38557 (diff)
Add check that SSH validate returns true
The SSH Validate call needs to be checked before passing tests. test_ssh_client_fip_before_active and test_ssh_client_fip_after_active run the SSH Verify function, but do not check the return value. You will see an error in the test log file, but the test will appear to have passed. JIRA: SNAPS-45 Added an Assert True test to the SSH Validate return. Change-Id: Ib86ec38c7b5a64061a1ace4293336e3021c59013 Signed-off-by: randyl <r.levensalor@cablelabs.com>
-rw-r--r--snaps/openstack/tests/create_instance_tests.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/snaps/openstack/tests/create_instance_tests.py b/snaps/openstack/tests/create_instance_tests.py
index 9a02765..827f305 100644
--- a/snaps/openstack/tests/create_instance_tests.py
+++ b/snaps/openstack/tests/create_instance_tests.py
@@ -624,7 +624,7 @@ class CreateInstanceSingleNetworkTests(OSIntegrationTestCase):
self.assertTrue(inst_creator.vm_active(block=True))
self.assertEquals(vm_inst, inst_creator.get_vm_inst())
- validate_ssh_client(inst_creator)
+ self.assertTrue(validate_ssh_client(inst_creator))
def test_ssh_client_fip_after_active(self):
"""
@@ -651,7 +651,7 @@ class CreateInstanceSingleNetworkTests(OSIntegrationTestCase):
self.assertTrue(inst_creator.vm_active(block=True))
self.assertEquals(vm_inst, inst_creator.get_vm_inst())
- validate_ssh_client(inst_creator)
+ self.assertTrue(validate_ssh_client(inst_creator))
# TODO - Determine how allowed_address_pairs is supposed to operate before continuing this test
# see http://docs.openstack.org/developer/dragonflow/specs/allowed_address_pairs.html for a functional description