commit fa2840d590c70688a1fc95d4db643c8518132736
author George Keishing <gkeishin@in.ibm.com> Tue Jan 28 01:41:30 2020 -0600
committer George Keishing <gkeishin@in.ibm.com> Wed Jan 29 15:43:16 2020 +0000
tree 3e06256263f1588ece08df2c85e71dccebec3e32
parent 9f84bc83d1aea9cd4d15ee4f40f873f7d330c54c

Update NTP enabled body and response code check

Change-Id: Idd9a8079c7a37abc864bbde5cb923e0ace988aaf
Signed-off-by: George Keishing <gkeishin@in.ibm.com>

redfish/managers/test_managers_bmc_time.robot

diff --git a/redfish/managers/test_managers_bmc_time.robot b/redfish/managers/test_managers_bmc_time.robot
index e297b17..5866df8 100644
--- a/redfish/managers/test_managers_bmc_time.robot
+++ b/redfish/managers/test_managers_bmc_time.robot
@@ -109,6 +109,7 @@
     [Tags]  Verify_NTP_Server_Set
 
     Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     ${network_protocol}=  Redfish.Get Properties  ${REDFISH_NW_PROTOCOL_URI}
     Should Contain  ${network_protocol["NTP"]["NTPServers"]}  ${ntp_server_1}
     ...  msg=NTP server value ${ntp_server_1} not stored.
@@ -121,6 +122,7 @@
     [Tags]  Verify_NTP_Server_Value_Not_Duplicated
 
     Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_1}']}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     ${network_protocol}=  Redfish.Get Properties  ${REDFISH_NW_PROTOCOL_URI}
     Should Contain X Times  ${network_protocol["NTP"]["NTPServers"]}  ${ntp_server_1}  1
     ...  msg=NTP primary and secondary server values should not be same.
@@ -131,6 +133,7 @@
     [Tags]  Verify_NTP_Server_Setting_Persist_After_BMC_Reboot
 
     Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     Redfish OBMC Reboot (off)
     Redfish.Login
     ${network_protocol}=  Redfish.Get Properties  ${REDFISH_NW_PROTOCOL_URI}
@@ -149,7 +152,8 @@
     Set Suite Variable  ${original_ntp}
     Rprint Vars  original_ntp
     # The following patch command should set the ["NTP"]["ProtocolEnabled"] property to "True".
-    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={u'NTPEnabled': ${True}}
+    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'ProtocolEnabled': ${True}}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     ${ntp}=  Redfish.Get Attribute  ${REDFISH_NW_PROTOCOL_URI}  NTP
     Rprint Vars  ntp
     Valid Value  ntp["ProtocolEnabled"]  valid_values=[True]
@@ -212,7 +216,8 @@
     Return From Keyword If  &{original_ntp} == &{EMPTY}
     Print Timen  Restore NTP Mode.
     Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}
-    ...  body={u'NTPEnabled': ${original_ntp["ProtocolEnabled"]}}
+    ...  body={'NTP':{'ProtocolEnabled': ${original_ntp["ProtocolEnabled"]}}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
 
 
 Suite Setup Execution