Corrected logic for Set_Host_Time_With_Both_And_NTP test case.

This change is required as user can not adjust the time with
NTP mode.

Re-enable this test case by removing this from skip lists.

Resolves openbmc/openbmc-test-automation#761

Change-Id: Ic906aa22da1e47f4a4f132a26d1de1c1b29850f2
Signed-off-by: Rahul Maheshwari <rahulmaheshwari@in.ibm.com>
diff --git a/test_lists/skip_test b/test_lists/skip_test
index d453880..84699a6 100644
--- a/test_lists/skip_test
+++ b/test_lists/skip_test
@@ -15,8 +15,6 @@
 -e Set_password_for_existing_user
 -e Set_password_with_empty_password_for_existing
 -e Create_existing_user
-#https://github.com/openbmc/openbmc/issues/928
--e Set_Host_Time_With_Both_And_NTP
 #https://github.com/openbmc/openbmc/issues/901
 -e Set_BMC_Time_With_Split_And_Manual
 #https://github.com/openbmc/openbmc/issues/1142
diff --git a/test_lists/skip_test_barreleye b/test_lists/skip_test_barreleye
index 9b65a82..78a3354 100644
--- a/test_lists/skip_test_barreleye
+++ b/test_lists/skip_test_barreleye
@@ -15,8 +15,6 @@
 -e Set_password_for_existing_user
 -e Set_password_with_empty_password_for_existing
 -e Create_existing_user
-#https://github.com/openbmc/openbmc/issues/928
--e Set_Host_Time_With_Both_And_NTP
 #https://github.com/openbmc/openbmc/issues/901
 -e Set_BMC_Time_With_Split_And_Manual
 #https://github.com/openbmc/openbmc/issues/1142
diff --git a/test_lists/skip_test_palmetto b/test_lists/skip_test_palmetto
index ebe80f7..45c611b 100644
--- a/test_lists/skip_test_palmetto
+++ b/test_lists/skip_test_palmetto
@@ -25,8 +25,6 @@
 -e Verify_Enabling_OCC_Turbo_Setting_Via_IPMI
 -e Verify_Disabling_OCC_Turbo_Setting_Via_IPMI
 -e Verify_Setting_OCC_Turbo_Via_REST
-#https://github.com/openbmc/openbmc/issues/928
--e Set_Host_Time_With_Both_And_NTP
 #https://github.com/openbmc/openbmc/issues/901
 -e Set_BMC_Time_With_Split_And_Manual
 #https://github.com/openbmc/openbmc/issues/1142
diff --git a/test_lists/skip_test_witherspoon b/test_lists/skip_test_witherspoon
index cd2173a..bac9b7f 100644
--- a/test_lists/skip_test_witherspoon
+++ b/test_lists/skip_test_witherspoon
@@ -21,8 +21,6 @@
 -e io_board_Fault
 -e Event_Logging
 -e Association
-#https://github.com/openbmc/openbmc/issues/928
--e Set_Host_Time_With_Both_And_NTP
 #https://github.com/openbmc/openbmc/issues/901
 -e Set_BMC_Time_With_Split_And_Manual
 #https://github.com/openbmc/openbmc/issues/1142
diff --git a/tests/test_time.robot b/tests/test_time.robot
index 0fd88e8..6744686 100644
--- a/tests/test_time.robot
+++ b/tests/test_time.robot
@@ -161,7 +161,7 @@
 
 Set Host Time With Both And NTP
     #Operation     Owner  Mode    Status  BMC Time   Host Time
-    Set Host Time  BOTH   NTP     ok      No Change  Set
+    Set Host Time  BOTH   NTP     error   No Change  Not Set
 
     [Documentation]  Set host time when time owner is both and time mode is
     ...              NTP.