commit 79fc7f0883398ca6695e2d6e5a200da8d8658289
author George Keishing <gkeishin@in.ibm.com> Fri May 02 00:37:49 2025 +0530
committer George Keishing <gkeishin@in.ibm.com> Wed May 07 07:48:07 2025 +0000
tree 2f6654394ba49cb70d4542bf634d51a280ffde76
parent 62b0c90e54c41276e2d41aff217f93c91d9df4a7

Fix depecrating robot syntax in test suites

Changes:
    - Run KeyWord If is deprecated since Robot
      Framework version 5.*, use IF instead
    - Keyword name Redfish.login does not follow case convention
    - TEMPLATE is in wrong place of Test Case.
      Recommended order of elements in Test Cases:
      DOCUMENTATION, TAGS, TIMEOUT, SETUP, TEMPLATE, KEYWORD, TEARDOWN

Tested:
     - Ran dry-run test
Change-Id: I7b861ea1f90b86f57a483b97d40d3104dc789cdb
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

redfish/account_service/test_ldap_configuration.robot

diff --git a/redfish/account_service/test_ldap_configuration.robot b/redfish/account_service/test_ldap_configuration.robot
index 074dc17..0338772 100644
--- a/redfish/account_service/test_ldap_configuration.robot
+++ b/redfish/account_service/test_ldap_configuration.robot
@@ -766,8 +766,11 @@
     ${power_status}=  Redfish.Get Attribute  /redfish/v1/Chassis/${CHASSIS_ID}  PowerState
     Return From Keyword If  '${power_status}' == '${expected_power_status}'
 
-    Run Keyword If  '${power_status}' == 'Off'  Redfish Power On
-    ...  ELSE  Redfish Power Off
+    IF  '${power_status}' == 'Off'
+        Redfish Power On
+    ELSE
+        Redfish Power Off
+    END
 
 Update LDAP User Role And Host Poweroff
     [Documentation]  Update LDAP user role and do host poweroff.