commit f05a1036b3a513240f1a848264617eb293b0a2a0
author George Keishing <gkeishin@in.ibm.com> Mon Jan 27 11:08:51 2020 -0600
committer George Keishing <gkeishin@in.ibm.com> Tue Jan 28 05:52:22 2020 +0000
tree 1e5763866d01b54bc1698eb3610fb98e65dd5b44
parent 14455f46774f5d321ac695f34eac193da662ece1

Fix NTP patch request body construct

Change-Id: Iac7f1d00cdbf189243d8788936d3c122f2077fa1
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 7347f4f..e297b17 100644
--- a/redfish/managers/test_managers_bmc_time.robot
+++ b/redfish/managers/test_managers_bmc_time.robot
@@ -108,7 +108,7 @@
     [Documentation]  Verify NTP server set.
     [Tags]  Verify_NTP_Server_Set
 
-    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}
+    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}}
     ${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.
@@ -120,7 +120,7 @@
     [Documentation]  Verify NTP servers value not same for both primary and secondary server.
     [Tags]  Verify_NTP_Server_Value_Not_Duplicated
 
-    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTPServers': ['${ntp_server_1}', '${ntp_server_1}']}
+    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_1}']}}
     ${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.
@@ -130,7 +130,7 @@
     [Documentation]  Verify NTP server setting persist after BMC reboot.
     [Tags]  Verify_NTP_Server_Setting_Persist_After_BMC_Reboot
 
-    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}
+    Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}  body={'NTP':{'NTPServers': ['${ntp_server_1}', '${ntp_server_2}']}}
     Redfish OBMC Reboot (off)
     Redfish.Login
     ${network_protocol}=  Redfish.Get Properties  ${REDFISH_NW_PROTOCOL_URI}
@@ -226,6 +226,7 @@
     [Documentation]  Do the suite level teardown.
 
     Redfish.Patch  ${REDFISH_NW_PROTOCOL_URI}
-    ...  body={'NTPServers': ['${EMPTY}', '${EMPTY}']}
+    ...  body={'NTP':{'NTPServers': ['${EMPTY}', '${EMPTY}']}}
+    ...  valid_status_codes=[${HTTP_OK}, ${HTTP_NO_CONTENT}]
     Rest Set Time Owner
     Redfish.Logout