diff options
author | Thom May <thom@may.lt> | 2018-01-18 14:16:28 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-01-18 14:16:28 +0000 |
commit | d0a68f81fcd6fe8abe12dabfaaf16d995a933a76 (patch) | |
tree | 775c12e2c12543a47c071ba7f83388166f4d2ce2 /lib/chef | |
parent | 23d1aed242c7933f85e6258cba47b9dc4e7e4521 (diff) | |
parent | 6cac5affe524a2fc481a944bebef4135a16a34b7 (diff) | |
download | chef-d0a68f81fcd6fe8abe12dabfaaf16d995a933a76.tar.gz |
Merge pull request #6714 from MsysTechnologiesllc/dh/update_validation_on_idle
[MSYS-735]fix validation for idle_time
Diffstat (limited to 'lib/chef')
-rw-r--r-- | lib/chef/resource/windows_task.rb | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/lib/chef/resource/windows_task.rb b/lib/chef/resource/windows_task.rb index 14f5da46fa..10e8400c34 100644 --- a/lib/chef/resource/windows_task.rb +++ b/lib/chef/resource/windows_task.rb @@ -82,7 +82,7 @@ class Chef validate_create_frequency_modifier(frequency, frequency_modifier) validate_create_day(day, frequency) if day validate_create_months(months, frequency) if months - validate_idle_time(idle_time, frequency) if idle_time + validate_idle_time(idle_time, frequency) if ( !idle_time.nil? && ([:minute, :hourly, :daily, :weekly, :monthly].include? frequency)) || (idle_time.nil? || !(idle_time > 0 && idle_time <= 999)) && !([:minute, :hourly, :daily, :weekly, :monthly].include? frequency) end private @@ -199,7 +199,9 @@ class Chef unless [:on_idle].include?(frequency) raise ArgumentError, "idle_time property is only valid for tasks that run on_idle" end - + if idle_time.nil? + raise ArgumentError, "idle_time value should be set for :on_idle frequency." + end unless idle_time > 0 && idle_time <= 999 raise ArgumentError, "idle_time value #{idle_time} is invalid. Valid values for :on_idle frequency are 1 - 999." end |