From: Jozef Behran Date: Tue, 19 Jan 2016 16:38:53 +0000 (+0100) Subject: Rename "first_case_ok" to "netconf_is_ready" X-Git-Tag: release/beryllium-sr1~202 X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?p=integration%2Ftest.git;a=commitdiff_plain;h=07addcfd03f8d59fa445e8565c0f2ef05729d719 Rename "first_case_ok" to "netconf_is_ready" The new name gives much better meaning to the variable, improving code readability quite a lot. Additionally, I want to add some more test cases which shall not run if netconf is detected to be ready. The code of these new test cases is going to be a lot more readable if the variable has this new name. Change-Id: I8254443aad3d0ea9e039fa1c4051bf56ce5053a3 Signed-off-by: Jozef Behran --- diff --git a/csit/suites/netconf/ready/netconfready.robot b/csit/suites/netconf/ready/netconfready.robot index 6d7d02354e..125e9c275a 100644 --- a/csit/suites/netconf/ready/netconfready.robot +++ b/csit/suites/netconf/ready/netconfready.robot @@ -30,7 +30,7 @@ Resource ${CURDIR}/../../../libraries/Utils.robot Variables ${CURDIR}/../../../variables/Variables.py *** Variables *** -${first_case_ok} False +${netconf_is_ready} False ${NETCONFREADY_WAIT} 60s ${DEBUG_LOGGING_FOR_EVERYTHING} False @@ -39,13 +39,13 @@ Check_Whether_Netconf_Connector_Works [Documentation] Make one request to netconf-connector and see if it works. [Tags] exclude Check_Netconf_Connector - BuiltIn.Set_Suite_Variable ${first_case_ok} True + BuiltIn.Set_Suite_Variable ${netconf_is_ready} True [Teardown] Utils.Report_Failure_Due_To_Bug 4708 Wait_For_Netconf_Connector [Documentation] Attempt to wait for the netconf-connector for configurable time. [Tags] critical - BuiltIn.Run_Keyword_Unless ${first_case_ok} BuiltIn.Wait_Until_Keyword_Succeeds ${NETCONFREADY_WAIT} 1s Check_Netconf_Connector + BuiltIn.Run_Keyword_Unless ${netconf_is_ready} BuiltIn.Wait_Until_Keyword_Succeeds ${NETCONFREADY_WAIT} 1s Check_Netconf_Connector [Teardown] Utils.Report_Failure_Due_To_Bug 4583 Check_Whether_Netconf_Connector_Can_Pretty_Print