summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLamont Granquist <lamont@scriptkiddie.org>2020-05-13 19:44:45 -0700
committerLamont Granquist <lamont@scriptkiddie.org>2020-05-13 20:12:12 -0700
commitcf139b4011d3ded5a1194a0d78fe80667eccdc6d (patch)
treef30c22ec5669a00bb8d1f9828c57919bd150a01e
parent9bd758ff794afe1eab15ec4aac644d49fcf4d5c1 (diff)
downloadchef-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.rb2
-rw-r--r--spec/unit/provider/launchd_spec.rb42
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