diff options
author | Lamont Granquist <lamont@scriptkiddie.org> | 2020-05-13 19:44:45 -0700 |
---|---|---|
committer | Lamont Granquist <lamont@scriptkiddie.org> | 2020-05-13 20:12:12 -0700 |
commit | cf139b4011d3ded5a1194a0d78fe80667eccdc6d (patch) | |
tree | f30c22ec5669a00bb8d1f9828c57919bd150a01e | |
parent | 9bd758ff794afe1eab15ec4aac644d49fcf4d5c1 (diff) | |
download | chef-cf139b4011d3ded5a1194a0d78fe80667eccdc6d.tar.gz |
Remove useless test
We broke this and they didn't fail, refactored the code and
fixed it and now they're failing. We've got coverage of this
in the functional tests now
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
-rw-r--r-- | lib/chef/provider/launchd.rb | 2 | ||||
-rw-r--r-- | spec/unit/provider/launchd_spec.rb | 42 |
2 files changed, 0 insertions, 44 deletions
diff --git a/lib/chef/provider/launchd.rb b/lib/chef/provider/launchd.rb index eb01661fe0..78ae823596 100644 --- a/lib/chef/provider/launchd.rb +++ b/lib/chef/provider/launchd.rb @@ -62,8 +62,6 @@ class Chef end action :delete do - # If you delete a service you want to make sure its not loaded or - # the service will be in memory and you wont be able to stop it. if ::File.exists?(path) manage_service(:disable) end diff --git a/spec/unit/provider/launchd_spec.rb b/spec/unit/provider/launchd_spec.rb index 69f0f315d0..6a729c2cb6 100644 --- a/spec/unit/provider/launchd_spec.rb +++ b/spec/unit/provider/launchd_spec.rb @@ -194,48 +194,6 @@ describe Chef::Provider::Launchd do end end - describe "with an :enable action" do - describe "and the file has been updated" do - before(:each) do - allow(provider).to receive( - :manage_plist - ).with(:create).and_return(true) - allow(provider).to receive( - :manage_service - ).with(:restart).and_return(true) - end - - it "should call manage_service with a :restart action" do - expect(provider.manage_service(:restart)).to be_truthy - end - - it "works with action enable" do - expect(run_resource_setup_for_action(:enable)).to be_truthy - provider.action_enable - end - end - - describe "and the file has not been updated" do - before(:each) do - allow(provider).to receive( - :manage_plist - ).with(:create).and_return(nil) - allow(provider).to receive( - :manage_service - ).with(:enable).and_return(true) - end - - it "should call manage_service with a :enable action" do - expect(provider.manage_service(:enable)).to be_truthy - end - - it "works with action enable" do - expect(run_resource_setup_for_action(:enable)).to be_truthy - provider.action_enable - end - end - end - describe "with an :delete action" do describe "and the ld file is present" do before(:each) do |