chef#6638 fixed random_delay and execution_time_limit property not idempotent for windows_task resource

Signed-off-by: Vasu1105 <vasundhara.jagdale@msystechnologies.com>

1 files changed, 16 insertions, 2 deletions

diff --git a/lib/chef/provider/windows_task.rb b/lib/chef/provider/windows_task.rb
index 02e45cb9e0..dca0156d9c 100644
--- a/lib/chef/provider/windows_task.rb
+++ b/lib/chef/provider/windows_task.rb
@@ -229,8 +229,8 @@ class Chef
             current_resource.frequency_modifier != new_resource.frequency_modifier ||
             current_resource.frequency != new_resource.frequency ||
             current_resource.idle_time != new_resource.idle_time ||
-            current_resource.random_delay != new_resource.random_delay ||
-            !new_resource.execution_time_limit.include?(current_resource.execution_time_limit) ||
+            random_delay_updated? ||
+            execution_time_limit_updated? ||
             (new_resource.start_day && new_resource.start_day != "N/A" && start_day_updated?) ||
             (new_resource.start_time && new_resource.start_time != "N/A" && start_time_updated?)
         begin
@@ -243,6 +243,20 @@ class Chef
         false
       end
 
+      def random_delay_updated?
+        return false if current_resource.random_delay.nil? && new_resource.random_delay.nil?
+        return true if new_resource.random_delay.nil?
+        current_resource.random_delay = 0 if current_resource.random_delay.nil?
+        ISO8601::Duration.new(current_resource.random_delay) != ISO8601::Duration.new(new_resource.random_delay)
+      end
+
+      def execution_time_limit_updated?
+        return false if current_resource.execution_time_limit.nil? && new_resource.execution_time_limit.nil?
+        return true if new_resource.execution_time_limit.nil?
+        current_resource.execution_time_limit = 0 if current_resource.execution_time_limit.nil?
+        ISO8601::Duration.new(current_resource.execution_time_limit) != ISO8601::Duration.new(new_resource.execution_time_limit)
+      end
+
       def start_day_updated?
         current_day = DateTime.strptime(current_resource.start_day, convert_system_date_format_to_ruby_date_format)
         new_day = parse_day(new_resource.start_day)