diff options
author | NAshwini <ashwini.nehate@msystechnologies.com> | 2017-09-13 11:55:08 +0000 |
---|---|---|
committer | NAshwini <ashwini.nehate@msystechnologies.com> | 2017-09-20 17:56:46 +0530 |
commit | e295ddfad36a9c8fb35c65fe9099f75a17f1a3b0 (patch) | |
tree | 114ea888e67861bef3012df8238e0c077da1f22d | |
parent | a48cf1391c85294684a633f38ccd55433e09ad1e (diff) | |
download | chef-ash/add_none_frequency_to_windows_task_resource.tar.gz |
Fix spec and review commentsash/add_none_frequency_to_windows_task_resource
Signed-off-by: NAshwini <ashwini.nehate@msystechnologies.com>
-rw-r--r-- | lib/chef/provider/windows_task.rb | 4 | ||||
-rw-r--r-- | spec/unit/provider/windows_task_spec.rb | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lib/chef/provider/windows_task.rb b/lib/chef/provider/windows_task.rb index 7dbb4fa8c3..f637035e5b 100644 --- a/lib/chef/provider/windows_task.rb +++ b/lib/chef/provider/windows_task.rb @@ -306,8 +306,8 @@ class Chef if new_resource.frequency == :none doc.root.elements.delete(xml_element_mapping["random_delay"]) - cwd_element = REXML::Element.new(xml_element_mapping["random_delay"]) - doc.root.elements.add(cwd_element) + trigger_element = REXML::Element.new(xml_element_mapping["random_delay"]) + doc.root.elements.add(trigger_element) end options.each do |option| diff --git a/spec/unit/provider/windows_task_spec.rb b/spec/unit/provider/windows_task_spec.rb index 038d37248b..5346f921b5 100644 --- a/spec/unit/provider/windows_task_spec.rb +++ b/spec/unit/provider/windows_task_spec.rb @@ -149,7 +149,7 @@ describe Chef::Provider::WindowsTask do new_resource.random_delay "" allow(provider).to receive(:task_need_update?).and_return(true) allow(provider).to receive(:basic_validation).and_return(true) - allow(provider).to receive(:run_schtasks).with("CREATE", { "F" => "", "SC" => :once, "ST" => "00:00", "SD" => "", "TR" => nil, "RU" => "SYSTEM" }).twice + allow(provider).to receive(:run_schtasks).and_return("CREATE", { "F" => "", "SC" => :once, "ST" => "00:00", "SD" => "12/12/2012", "TR" => nil, "RU" => "SYSTEM" }) expect(provider).to receive(:update_task_xml) provider.run_action(:create) expect(new_resource).to be_updated_by_last_action |