commit 56a34f987dda9c3a829975758a6b88078b45c75f
author Igor Kanyuka <ifelmail@gmail.com> Fri Nov 14 06:27:29 2025 -0800
committer Igor Kanyuka <ifelmail@gmail.com> Tue Nov 18 09:52:45 2025 +0000
tree f363c3f8501e027c2897d56ea0199485e9a17928
parent 14e798cbdc0337638e514d7794e39ea08d595ff4

Expect HTTP 204 for PATCH user accounts

Changed:
  DBus code returns 204 instead of 200 for successful
  patch operations, see [1]. Expect 200 and 204.

Tested:
  Ran account service tests and made sure they succeeded.

[1] https://github.com/openbmc/bmcweb/commit/247ae89

Change-Id: Ifa60c71379e2d8328859b6e96152beef639676fb
Signed-off-by: Igor Kanyuka <ifelmail@gmail.com>

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 dc97acd..75a2adf 100644
--- a/redfish/account_service/test_user_account.robot
+++ b/redfish/account_service/test_user_account.robot
@@ -255,6 +255,7 @@
     # Update operator_user password using Redfish.
     ${payload}=  Create Dictionary  Password=NewTestPwd123
     Redfish.Patch  /redfish/v1/AccountService/Accounts/operator_user  body=&{payload}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
 
     # Verify users after updating
     Redfish Verify User  operator_user  NewTestPwd123  Operator        ${True}
@@ -309,6 +310,7 @@
     ...  AccountLockoutThreshold=${account_lockout_threshold}
     ...  AccountLockoutDuration=${account_lockout_duration}
     Redfish.Patch  ${REDFISH_ACCOUNTS_SERVICE_URI}  body=${payload}
+    ...  valid_status_codes=[${HTTP_OK},${HTTP_NO_CONTENT}]
 
     Redfish.Logout
 
@@ -331,6 +333,7 @@
     # Manually unlock the account before lockout threshold expires
     Redfish.Login
     Redfish.Patch  ${REDFISH_ACCOUNTS_URI}test_user  body=${payload}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     Redfish.Logout
 
     # Try redfish login with the recently unlocked account
@@ -354,6 +357,7 @@
 
     # Change password of 'readonly' user with admin user.
     Redfish.Patch  /redfish/v1/AccountService/Accounts/readonly_user  body={'Password': 'NewTestPwd123'}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
 
     # Verify modified user.
     Redfish Verify User  readonly_user  NewTestPwd123  ReadOnly  ${True}
@@ -379,6 +383,7 @@
 
     # Modify Role ID of Operator user.
     Redfish.Patch  /redfish/v1/AccountService/Accounts/operator_user  body={'RoleId': 'Administrator'}
+    ...  valid_status_codes=[${HTTP_OK},${HTTP_NO_CONTENT}]
 
     # Verify modified user.
     Redfish Verify User  operator_user  TestPwd123  Administrator  ${True}
@@ -1038,6 +1043,7 @@
 
     # Change to a valid password.
     Redfish.Patch  /redfish/v1/AccountService/Accounts/${user_name}  body={'Password': 'UserPwd1'}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
 
     # Verify login.
     Redfish.Logout