commit 7735cbba148c5ab5c8551ad5e25b7b3dc81efe73
author Megha G N <Megha.G.N@ibm.com> Tue Nov 15 23:22:10 2022 -0600
committer George Keishing <gkeishin@gmail.com> Thu Nov 17 13:54:27 2022 +0000
tree 4433c0b67387d6262a46ec73b88c43015a64b5d5
parent 732cc3ff35631ae89189273b0383026a08804957

Added new test case to protocol settings for IPMI and SSH.

Changes :
    - Added test to enable SSH, Disable IPMI and check for
      persistency on BMC Reboot
    - Updated documentation in the test suite

Tested:
    - Ran successfully test_bmc_protocol_settings.robot

Change-Id: I426c64e1fdd6fe99856bc028a03f557cdd38aee4
Signed-off-by: Megha G N <Megha.G.N@ibm.com>

redfish/managers/test_bmc_protocol_settings.robot

diff --git a/redfish/managers/test_bmc_protocol_settings.robot b/redfish/managers/test_bmc_protocol_settings.robot
index 0674cd4..e096e66 100644
--- a/redfish/managers/test_bmc_protocol_settings.robot
+++ b/redfish/managers/test_bmc_protocol_settings.robot
@@ -231,6 +231,14 @@
     ${False}     ${False}
 
 
+Enable SSH And Disable IPMI Protocol And Check Persistency On BMC Reboot
+    [Documentation]  Set the SSH to True and IPMI protocol attributes to False, and verify
+    ...              the setting after BMC reboot.
+    [Tags]  Enable_SSH_And_Disable_IPMI_Protocol_And_Check_Persistency_On_BMC_Reboot
+
+    Set SSH And IPMI Protocol  ${True}  ${False}  ${True}
+
+
 *** Keywords ***
 
 Suite Setup Execution
@@ -243,7 +251,7 @@
     Sleep  ${NETWORK_TIMEOUT}s
 
 Is BMC LastResetTime Changed
-    [Documentation]  return fail if BMC last reset time is not changed
+    [Documentation]  return fail if BMC last reset time is not changed.
     [Arguments]  ${reset_time}
 
     ${last_reset_time}=  Redfish.Get Attribute  /redfish/v1/Managers/bmc  LastResetTime
@@ -251,7 +259,7 @@
 
 
 Redfish BMC Reboot
-    [Documentation]  Use Redfish API reboot BMC and wait for BMC ready
+    [Documentation]  Use Redfish API reboot BMC and wait for BMC ready.
 
     #  Get BMC last reset time for compare
     ${last_reset_time}=  Redfish.Get Attribute  /redfish/v1/Managers/bmc  LastResetTime
@@ -265,7 +273,7 @@
 
 Set SSH And IPMI Protocol
     [Documentation]  Set SSH and IPMI protocol state.
-    [Arguments]  ${ssh_state}  ${ipmi_state}
+    [Arguments]  ${ssh_state}  ${ipmi_state}  ${persistency_check}=${False}
 
     # Description of argument(s):
     # ssh_state     State of SSH to be set (e.g. True, False).
@@ -281,6 +289,7 @@
     # Wait for timeout for new values to take effect.
     Sleep  ${NETWORK_TIMEOUT}s
 
+    Run Keyword if  ${persistency_check} == ${True}  Redfish OBMC Reboot (off)
     Verify Protocol State  ${ssh_state}  ${ipmi_state}