commit a83fb47f255593a277d94c7ada30823168c2a800
author Sandhya Somashekar <sansomas@in.ibm.com> Thu Aug 29 02:26:21 2019 -0500
committer Sandhya Somashekar <sansomas@in.ibm.com> Thu Aug 29 02:33:09 2019 -0500
tree 882e8b30648197d692885b5e48f332fecbbf3197
parent c0e91ecabfbb0ac9c8a712219d9efa0f634579fb

Small fix - variable usage change

Signed-off-by: Sandhya Somashekar <sansomas@in.ibm.com>
Change-Id: I0a83598ae9e4b403c7c3bd261c4b33d741a3c1f7

redfish/account_service/test_user_account.robot

diff --git a/redfish/account_service/test_user_account.robot b/redfish/account_service/test_user_account.robot
index ffc7dbb..51982c0 100644
--- a/redfish/account_service/test_user_account.robot
+++ b/redfish/account_service/test_user_account.robot
@@ -23,6 +23,32 @@
     ${resp} =  Redfish_utils.Get Attribute  /redfish/v1/AccountService  ServiceEnabled
     Should Be Equal As Strings  ${resp}  ${True}
 
+Verify Redfish User Persistence After Reboot
+    [Documentation]  Verify Redfish user persistence after reboot.
+    [Tags]  Verify_Redfish_User_Persistence_After_Reboot
+
+    # Create Redfish users.
+    Redfish Create User  admin_user     TestPwd123  Administrator   ${True}
+    Redfish Create User  operator_user  TestPwd123  Operator        ${True}
+    Redfish Create User  user_user      TestPwd123  User            ${True}
+    Redfish Create User  callback_user  TestPwd123  Callback        ${True}
+
+    # Reboot BMC.
+    Redfish OBMC Reboot (off)  stack_mode=normal
+    Redfish.Login
+
+    # Verify users after reboot.
+    Redfish Verify User  admin_user     TestPwd123  Administrator   ${True}
+    Redfish Verify User  operator_user  TestPwd123  Operator        ${True}
+    Redfish Verify User  user_user      TestPwd123  User            ${True}
+    Redfish Verify User  callback_user  TestPwd123  Callback        ${True}
+
+    # Delete created users.
+    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}admin_user
+    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}operator_user
+    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}user_user
+    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}callback_user
+
 Redfish Create and Verify Users
     [Documentation]  Create Redfish users with various roles.
     [Tags]  Redfish_Create_and_Verify_Users
@@ -67,31 +93,6 @@
     user_user      TestPwd123  User            ${False}
     callback_user  TestPwd123  Callback        ${False}
 
-Verify Redfish User Persistence After Reboot
-    [Documentation]  Verify Redfish user persistence after reboot.
-    [Tags]  Verify_Redfish_User_Persistence_After_Reboot
-
-    # Create Redfish users.
-    Redfish Create User  admin_user     TestPwd123  Administrator   ${True}
-    Redfish Create User  operator_user  TestPwd123  Operator        ${True}
-    Redfish Create User  user_user      TestPwd123  User            ${True}
-    Redfish Create User  callback_user  TestPwd123  Callback        ${True}
-
-    # Reboot BMC.
-    Redfish OBMC Reboot (off)  stack_mode=normal
-    Redfish.Login
-
-    # Verify users after reboot.
-    Redfish Verify User  admin_user     TestPwd123  Administrator   ${True}
-    Redfish Verify User  operator_user  TestPwd123  Operator        ${True}
-    Redfish Verify User  user_user      TestPwd123  User            ${True}
-    Redfish Verify User  callback_user  TestPwd123  Callback        ${True}
-
-    # Delete created users.
-    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}admin_user
-    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}operator_user
-    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}user_user
-    Redfish.Delete  ${REDFISH_ACCOUNTS_URI}callback_user
 
 Verify User Creation With Invalid Role Id
     [Documentation]  Verify user creation with invalid role ID.
@@ -312,7 +313,7 @@
     ...  ELSE
     ...    Redfish.Login  ${username}  ${password}
 
-    Run Keyword If  ${enabled} == False
+    Run Keyword If  ${enabled} == ${False}
     ...  Redfish.Login
 
     # Validate Role ID of created user.