chef 15.12.22-universal-mingw32 → 15.16.4-universal-mingw32
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/Gemfile +10 -10
- data/Rakefile +11 -17
- data/chef-universal-mingw32.gemspec +4 -4
- data/chef.gemspec +22 -4
- data/lib/chef/api_client/registration.rb +2 -2
- data/lib/chef/chef_fs/file_system/repository/base_file.rb +1 -0
- data/lib/chef/chef_fs/file_system/repository/chef_repository_file_system_cookbook_dir.rb +2 -2
- data/lib/chef/chef_fs/file_system/repository/chef_repository_file_system_cookbook_entry.rb +1 -1
- data/lib/chef/chef_fs/file_system/repository/chef_repository_file_system_root_dir.rb +2 -2
- data/lib/chef/chef_fs/file_system/repository/directory.rb +1 -1
- data/lib/chef/chef_fs/file_system/repository/file_system_entry.rb +1 -1
- data/lib/chef/deprecated.rb +4 -0
- data/lib/chef/environment.rb +2 -2
- data/lib/chef/exceptions.rb +3 -0
- data/lib/chef/http.rb +2 -1
- data/lib/chef/knife/bootstrap.rb +3 -3
- data/lib/chef/knife/bootstrap/train_connector.rb +1 -0
- data/lib/chef/knife/cookbook_download.rb +1 -1
- data/lib/chef/knife/cookbook_metadata.rb +1 -1
- data/lib/chef/knife/core/gem_glob_loader.rb +1 -1
- data/lib/chef/knife/core/hashed_command_loader.rb +3 -2
- data/lib/chef/knife/core/subcommand_loader.rb +1 -1
- data/lib/chef/knife/exec.rb +2 -2
- data/lib/chef/knife/ssh.rb +20 -1
- data/lib/chef/log.rb +1 -1
- data/lib/chef/mixin/openssl_helper.rb +26 -3
- data/lib/chef/mixin/template.rb +1 -0
- data/lib/chef/node_map.rb +5 -2
- data/lib/chef/provider/mount/solaris.rb +0 -1
- data/lib/chef/provider/package/dnf/dnf_helper.py +5 -0
- data/lib/chef/provider/package/freebsd/pkgng.rb +3 -1
- data/lib/chef/provider/package/yum/yum_helper.py +4 -0
- data/lib/chef/provider/package/zypper.rb +0 -1
- data/lib/chef/provider/service/arch.rb +2 -2
- data/lib/chef/provider/service/debian.rb +1 -1
- data/lib/chef/provider/service/gentoo.rb +2 -2
- data/lib/chef/provider/service/macosx.rb +2 -2
- data/lib/chef/provider/service/openbsd.rb +1 -1
- data/lib/chef/provider/service/redhat.rb +2 -2
- data/lib/chef/provider/service/upstart.rb +1 -1
- data/lib/chef/provider/yum_repository.rb +1 -1
- data/lib/chef/provider/zypper_repository.rb +1 -1
- data/lib/chef/resource.rb +2 -0
- data/lib/chef/resource/cron_access.rb +2 -2
- data/lib/chef/resource/cron_d.rb +2 -1
- data/lib/chef/resource/homebrew_cask.rb +3 -3
- data/lib/chef/resource/hostname.rb +18 -18
- data/lib/chef/resource/locale.rb +1 -1
- data/lib/chef/resource/lwrp_base.rb +7 -0
- data/lib/chef/resource/ssh_known_hosts_entry.rb +1 -1
- data/lib/chef/resource/sudo.rb +2 -2
- data/lib/chef/resource/windows_feature_powershell.rb +6 -2
- data/lib/chef/resource/windows_font.rb +2 -1
- data/lib/chef/role.rb +2 -2
- data/lib/chef/shell.rb +32 -1
- data/lib/chef/shell/shell_session.rb +2 -0
- data/lib/chef/util/diff.rb +1 -1
- data/lib/chef/util/dsc/configuration_generator.rb +1 -1
- data/lib/chef/util/powershell/cmdlet.rb +1 -1
- data/lib/chef/version.rb +2 -2
- data/lib/chef/version_string.rb +1 -1
- data/lib/chef/win32/file.rb +2 -2
- data/spec/functional/knife/ssh_spec.rb +4 -4
- data/spec/functional/resource/aix_service_spec.rb +0 -1
- data/spec/functional/resource/aixinit_service_spec.rb +7 -8
- data/spec/functional/resource/apt_package_spec.rb +0 -1
- data/spec/functional/resource/bff_spec.rb +2 -2
- data/spec/functional/resource/cookbook_file_spec.rb +1 -1
- data/spec/functional/resource/cron_spec.rb +0 -1
- data/spec/functional/resource/dsc_resource_spec.rb +1 -1
- data/spec/functional/resource/insserv_spec.rb +4 -5
- data/spec/functional/resource/link_spec.rb +17 -17
- data/spec/functional/resource/rpm_spec.rb +2 -2
- data/spec/functional/resource/user/dscl_spec.rb +1 -1
- data/spec/functional/resource/user/mac_user_spec.rb +1 -1
- data/spec/functional/resource/windows_certificate_spec.rb +3 -3
- data/spec/functional/resource/windows_font_spec.rb +49 -0
- data/spec/functional/run_lock_spec.rb +2 -1
- data/spec/functional/shell_spec.rb +5 -5
- data/spec/functional/util/powershell/cmdlet_spec.rb +1 -1
- data/spec/functional/version_spec.rb +1 -1
- data/spec/functional/win32/service_manager_spec.rb +1 -1
- data/spec/integration/recipes/accumulator_spec.rb +1 -1
- data/spec/integration/recipes/lwrp_inline_resources_spec.rb +2 -2
- data/spec/integration/recipes/lwrp_spec.rb +1 -1
- data/spec/integration/recipes/notifies_spec.rb +1 -1
- data/spec/integration/recipes/notifying_block_spec.rb +1 -1
- data/spec/integration/recipes/recipe_dsl_spec.rb +1 -1
- data/spec/integration/recipes/resource_converge_if_changed_spec.rb +2 -0
- data/spec/integration/recipes/resource_load_spec.rb +1 -0
- data/spec/integration/recipes/unified_mode_spec.rb +1 -1
- data/spec/scripts/ssl-serve.rb +1 -1
- data/spec/spec_helper.rb +10 -8
- data/spec/support/platform_helpers.rb +12 -42
- data/spec/support/platforms/win32/spec_service.rb +1 -1
- data/spec/support/shared/functional/directory_resource.rb +1 -1
- data/spec/support/shared/functional/execute_resource.rb +1 -1
- data/spec/support/shared/functional/file_resource.rb +2 -2
- data/spec/support/shared/functional/win32_service.rb +1 -1
- data/spec/support/shared/functional/windows_script.rb +3 -3
- data/spec/support/shared/integration/knife_support.rb +2 -5
- data/spec/unit/chef_fs/file_system/operation_failed_error_spec.rb +2 -4
- data/spec/unit/cookbook/gem_installer_spec.rb +2 -1
- data/spec/unit/dsl/platform_introspection_spec.rb +1 -0
- data/spec/unit/environment_spec.rb +7 -7
- data/spec/unit/event_dispatch/dispatcher_spec.rb +3 -0
- data/spec/unit/knife/bootstrap_spec.rb +14 -14
- data/spec/unit/knife/cookbook_download_spec.rb +4 -4
- data/spec/unit/knife/cookbook_metadata_from_file_spec.rb +1 -1
- data/spec/unit/knife/cookbook_upload_spec.rb +5 -6
- data/spec/unit/knife/core/hashed_command_loader_spec.rb +3 -3
- data/spec/unit/knife/ssh_spec.rb +2 -2
- data/spec/unit/knife/supermarket_share_spec.rb +5 -3
- data/spec/unit/lwrp_spec.rb +4 -4
- data/spec/unit/mixin/securable_spec.rb +0 -1
- data/spec/unit/property_spec.rb +5 -5
- data/spec/unit/provider/package/dnf/python_helper_spec.rb +1 -1
- data/spec/unit/provider/package/freebsd/pkgng_spec.rb +1 -1
- data/spec/unit/provider/package/rubygems_spec.rb +5 -10
- data/spec/unit/provider/package/windows_spec.rb +30 -53
- data/spec/unit/provider/service/arch_service_spec.rb +3 -2
- data/spec/unit/provider/service/debian_service_spec.rb +1 -1
- data/spec/unit/provider/service/gentoo_service_spec.rb +7 -7
- data/spec/unit/provider/service/macosx_spec.rb +3 -3
- data/spec/unit/provider/service/redhat_spec.rb +2 -2
- data/spec/unit/provider/service/upstart_service_spec.rb +3 -3
- data/spec/unit/provider/service/windows_spec.rb +2 -6
- data/spec/unit/provider/systemd_unit_spec.rb +28 -24
- data/spec/unit/provider_spec.rb +1 -0
- data/spec/unit/resource/windows_feature_powershell_spec.rb +30 -4
- data/spec/unit/resource/windows_package_spec.rb +1 -0
- data/spec/unit/role_spec.rb +11 -11
- data/spec/unit/run_context/cookbook_compiler_spec.rb +1 -1
- data/spec/unit/win32/security_spec.rb +4 -3
- metadata +48 -29
data/spec/unit/property_spec.rb
CHANGED
@@ -111,11 +111,11 @@ describe "Chef::Resource.property" do
|
|
111
111
|
|
112
112
|
with_property ":Straße" do
|
113
113
|
it "properties with UTF-8 in their name work" do
|
114
|
-
expect(resource.Straße).to eql(nil)
|
115
|
-
expect(resource.Straße "foo").to eql("foo")
|
116
|
-
expect(resource.Straße).to eql("foo")
|
117
|
-
expect(resource.Straße = "bar").to eql("bar")
|
118
|
-
expect(resource.Straße).to eql("bar")
|
114
|
+
expect(resource.Straße).to eql(nil) # rubocop: disable Naming/AsciiIdentifiers
|
115
|
+
expect(resource.Straße "foo").to eql("foo") # rubocop: disable Naming/AsciiIdentifiers
|
116
|
+
expect(resource.Straße).to eql("foo") # rubocop: disable Naming/AsciiIdentifiers
|
117
|
+
expect(resource.Straße = "bar").to eql("bar") # rubocop: disable Naming/AsciiIdentifiers
|
118
|
+
expect(resource.Straße).to eql("bar") # rubocop: disable Naming/AsciiIdentifiers
|
119
119
|
end
|
120
120
|
end
|
121
121
|
|
@@ -22,7 +22,7 @@ require "spec_helper"
|
|
22
22
|
describe Chef::Provider::Package::Dnf::PythonHelper do
|
23
23
|
let(:helper) { Chef::Provider::Package::Dnf::PythonHelper.instance }
|
24
24
|
|
25
|
-
it "propagates stacktraces on stderr from the forked subprocess" do
|
25
|
+
it "propagates stacktraces on stderr from the forked subprocess", :rhel do
|
26
26
|
allow(helper).to receive(:dnf_command).and_return("ruby -e 'raise \"your hands in the air\"'")
|
27
27
|
expect { helper.query(:whatprovides, "tcpdump") }.to raise_error(/your hands in the air/)
|
28
28
|
end
|
@@ -66,7 +66,7 @@ describe Chef::Provider::Package::Freebsd::Port do
|
|
66
66
|
end
|
67
67
|
|
68
68
|
it "should query pkg database" do
|
69
|
-
expect(@provider).to receive(:shell_out_compacted!).with("pkg", "info", "zsh", env: nil, returns: [0, 70], timeout: 900).and_return(@pkg_info)
|
69
|
+
expect(@provider).to receive(:shell_out_compacted!).with("pkg", "info", "zsh", env: nil, returns: [0, 1, 70], timeout: 900).and_return(@pkg_info)
|
70
70
|
expect(@provider.current_installed_version).to eq("3.1.7")
|
71
71
|
end
|
72
72
|
end
|
@@ -106,30 +106,25 @@ describe Chef::Provider::Package::Rubygems::CurrentGemEnvironment do
|
|
106
106
|
context "new default rubygems behavior" do
|
107
107
|
before do
|
108
108
|
Chef::Config[:rubygems_cache_enabled] = false
|
109
|
+
|
110
|
+
dep_installer = Gem::DependencyInstaller.new
|
111
|
+
expect(dep_installer).not_to receive(:find_gems_with_sources)
|
112
|
+
allow(@gem_env).to receive(:dependency_installer).and_return(dep_installer)
|
109
113
|
end
|
110
114
|
|
111
115
|
it "finds a matching gem candidate version on rubygems 2.0.0+" do
|
112
116
|
dep = Gem::Dependency.new("rspec", ">= 0")
|
113
|
-
dep_installer = Gem::DependencyInstaller.new
|
114
|
-
allow(@gem_env).to receive(:dependency_installer).and_return(dep_installer)
|
115
|
-
expect(dep_installer).not_to receive(:find_gems_with_sources).with(dep).and_call_original
|
116
117
|
expect(@gem_env.candidate_version_from_remote(dep)).to be_kind_of(Gem::Version)
|
117
118
|
end
|
118
119
|
|
119
120
|
it "gives the candidate version as nil if none is found" do
|
120
121
|
dep = Gem::Dependency.new("lksdjflksdjflsdkfj", ">= 0")
|
121
|
-
dep_installer = Gem::DependencyInstaller.new
|
122
|
-
allow(@gem_env).to receive(:dependency_installer).and_return(dep_installer)
|
123
|
-
expect(dep_installer).not_to receive(:find_gems_with_sources).with(dep).and_call_original
|
124
122
|
expect(@gem_env.candidate_version_from_remote(dep)).to be_nil
|
125
123
|
end
|
126
124
|
|
127
125
|
it "finds a matching gem from a specific gemserver when explicit sources are given (to a server that doesn't respond to api requests)" do
|
128
126
|
dep = Gem::Dependency.new("rspec", ">= 0")
|
129
|
-
|
130
|
-
allow(@gem_env).to receive(:dependency_installer).and_return(dep_installer)
|
131
|
-
expect(dep_installer).not_to receive(:find_gems_with_sources).with(dep).and_call_original
|
132
|
-
expect(@gem_env.candidate_version_from_remote(dep, "http://production.cf.rubygems.org")).to be_kind_of(Gem::Version)
|
127
|
+
expect(@gem_env.candidate_version_from_remote(dep, "https://rubygems.org")).to be_kind_of(Gem::Version)
|
133
128
|
end
|
134
129
|
end
|
135
130
|
|
@@ -42,7 +42,7 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
42
42
|
let(:cache_path) { 'c:\\cache\\' }
|
43
43
|
|
44
44
|
before(:each) do
|
45
|
-
allow(::File).to receive(:exist?).with(
|
45
|
+
allow(::File).to receive(:exist?).with(new_resource.source).and_return(true)
|
46
46
|
end
|
47
47
|
|
48
48
|
describe "load_current_resource" do
|
@@ -66,20 +66,22 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
66
66
|
|
67
67
|
it "sets the version to be installed" do
|
68
68
|
provider.load_current_resource
|
69
|
-
expect(
|
69
|
+
expect(new_resource.version).to eql("2.0")
|
70
70
|
end
|
71
71
|
end
|
72
72
|
|
73
|
-
context "when the source is
|
74
|
-
|
75
|
-
|
76
|
-
context "when the source has not been downloaded" do
|
73
|
+
context "when the source is not present it loads from cache" do
|
74
|
+
context "when the package is not installed" do
|
77
75
|
before(:each) do
|
78
|
-
allow(provider).to receive(:
|
76
|
+
allow(provider).to receive(:uri_scheme?).and_return(false)
|
77
|
+
allow(provider.package_provider).to receive(:get_product_property).and_return(nil)
|
78
|
+
allow(provider.package_provider).to receive(:get_installed_version).and_return(nil)
|
79
|
+
allow(provider.package_provider).to receive(:package_version).and_return(nil)
|
79
80
|
end
|
80
|
-
|
81
|
+
|
82
|
+
it "sets the current version nil" do
|
81
83
|
provider.load_current_resource
|
82
|
-
expect(provider.current_resource.version).to eql(
|
84
|
+
expect(provider.current_resource.version).to eql(nil)
|
83
85
|
end
|
84
86
|
end
|
85
87
|
|
@@ -233,9 +235,9 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
233
235
|
end
|
234
236
|
end
|
235
237
|
|
236
|
-
it "returns
|
237
|
-
|
238
|
-
expect(provider.installer_type).to eql(:
|
238
|
+
it "returns the resource's installer_type if it is set" do
|
239
|
+
new_resource.installer_type(:nsis)
|
240
|
+
expect(provider.installer_type).to eql(:nsis)
|
239
241
|
end
|
240
242
|
|
241
243
|
it "sets installer_type to inno if the source contains inno" do
|
@@ -275,7 +277,7 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
275
277
|
|
276
278
|
it "raises an error" do
|
277
279
|
allow(::Kernel).to receive(:open).and_yield(StringIO.new(""))
|
278
|
-
|
280
|
+
new_resource.installer_type(nil)
|
279
281
|
expect { provider.installer_type }.to raise_error(Chef::Exceptions::CannotDetermineWindowsInstallerType)
|
280
282
|
end
|
281
283
|
end
|
@@ -312,7 +314,10 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
312
314
|
let(:resource_source) { "https://foo.bar/calculator.exe" }
|
313
315
|
|
314
316
|
it "downloads the http resource" do
|
315
|
-
allow(
|
317
|
+
allow(provider).to receive(:uri_scheme?).and_return(true)
|
318
|
+
allow(provider).to receive(:installer_type).and_return(nil)
|
319
|
+
allow(File).to receive(:exist?).with("https\\foo.bar\\calculator.exe").and_return(false)
|
320
|
+
allow(provider).to receive(:compile_and_converge_action)
|
316
321
|
expect(provider).to receive(:download_source_file)
|
317
322
|
provider.run_action(:install)
|
318
323
|
end
|
@@ -400,7 +405,7 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
400
405
|
let(:resource_source) { "C:/a_missing_file.exe" }
|
401
406
|
let(:installer_type) { nil }
|
402
407
|
before do
|
403
|
-
allow(::File).to receive(:exist?).with(
|
408
|
+
allow(::File).to receive(:exist?).with(new_resource.source).and_return(false)
|
404
409
|
provider.load_current_resource
|
405
410
|
end
|
406
411
|
|
@@ -414,51 +419,23 @@ describe Chef::Provider::Package::Windows, :windows_only do
|
|
414
419
|
Chef::Config[:why_run] = false
|
415
420
|
end
|
416
421
|
end
|
417
|
-
end
|
418
422
|
|
419
|
-
|
420
|
-
|
421
|
-
|
422
|
-
allow(provider).to receive(:checksum).and_return("jiie00u3bbs92vsbhvgvklb2lasgh20ah")
|
423
|
-
end
|
423
|
+
it "does not raise an error with a valid checksum" do
|
424
|
+
expect(Chef::Digester).to receive(:checksum_for_file).with(new_resource.source).and_return("abcdef1234567890")
|
425
|
+
expect(provider).to receive(:install_package)
|
424
426
|
|
425
|
-
|
426
|
-
expect { provider.send(:validate_content!) }.to_not raise_error
|
427
|
-
end
|
428
|
-
end
|
429
|
-
end
|
430
|
-
|
431
|
-
shared_context "invalid checksum" do
|
432
|
-
context "checksum is invalid" do
|
433
|
-
before do
|
434
|
-
allow(provider).to receive(:checksum).and_return("kiie30u3bbs92vsbhvgvklb2lasgh20ah")
|
435
|
-
end
|
427
|
+
new_resource.checksum("abcdef1234567890")
|
436
428
|
|
437
|
-
|
438
|
-
expect { provider.send(:validate_content!) }.to raise_error(
|
439
|
-
Chef::Exceptions::ChecksumMismatch
|
440
|
-
)
|
441
|
-
end
|
429
|
+
provider.run_action(:install)
|
442
430
|
end
|
443
|
-
end
|
444
431
|
|
445
|
-
|
446
|
-
|
447
|
-
|
448
|
-
end
|
449
|
-
|
450
|
-
context "checksum is in lowercase" do
|
451
|
-
include_context "valid checksum"
|
452
|
-
include_context "invalid checksum"
|
453
|
-
end
|
432
|
+
it "raises an error with an invalid checksum" do
|
433
|
+
expect(Chef::Digester).to receive(:checksum_for_file).with(new_resource.source).and_return("abcdef1234567890")
|
434
|
+
expect(provider).not_to receive(:install_package)
|
454
435
|
|
455
|
-
|
456
|
-
before do
|
457
|
-
new_resource.checksum = new_resource.checksum.upcase
|
458
|
-
end
|
436
|
+
new_resource.checksum("ffffffffffffffff")
|
459
437
|
|
460
|
-
|
461
|
-
include_context "invalid checksum"
|
438
|
+
expect { provider.run_action(:install) }.to raise_error(Chef::Exceptions::Package)
|
462
439
|
end
|
463
440
|
end
|
464
441
|
end
|
@@ -37,7 +37,8 @@ describe Chef::Provider::Service::Arch, "load_current_resource" do
|
|
37
37
|
|
38
38
|
@provider = Chef::Provider::Service::Arch.new(@new_resource, @run_context)
|
39
39
|
|
40
|
-
allow(::File).to receive(:
|
40
|
+
allow(::File).to receive(:exist?).with("/etc/rc.d/chef").and_return(false)
|
41
|
+
allow(::File).to receive(:exist?).with("/etc/rc.conf").and_return(true)
|
41
42
|
allow(::File).to receive(:read).with("/etc/rc.conf").and_return("DAEMONS=(network apache sshd)")
|
42
43
|
end
|
43
44
|
|
@@ -106,7 +107,7 @@ describe Chef::Provider::Service::Arch, "load_current_resource" do
|
|
106
107
|
end
|
107
108
|
|
108
109
|
it "should fail if file /etc/rc.conf does not exist" do
|
109
|
-
allow(::File).to receive(:
|
110
|
+
allow(::File).to receive(:exist?).with("/etc/rc.conf").and_return(false)
|
110
111
|
expect { @provider.load_current_resource }.to raise_error(Chef::Exceptions::Service)
|
111
112
|
end
|
112
113
|
|
@@ -47,7 +47,7 @@ describe Chef::Provider::Service::Debian do
|
|
47
47
|
|
48
48
|
describe "load_current_resource" do
|
49
49
|
it "ensures /usr/sbin/update-rc.d is available" do
|
50
|
-
expect(File).to receive(:
|
50
|
+
expect(File).to receive(:exist?).with("/usr/sbin/update-rc.d").and_return(false)
|
51
51
|
|
52
52
|
@provider.define_resource_requirements
|
53
53
|
expect do
|
@@ -32,16 +32,16 @@ describe Chef::Provider::Service::Gentoo do
|
|
32
32
|
allow(Chef::Resource::Service).to receive(:new).and_return(@current_resource)
|
33
33
|
@status = double("Status", exitstatus: 0, stdout: @stdout)
|
34
34
|
allow(@provider).to receive(:shell_out).and_return(@status)
|
35
|
-
allow(File).to receive(:
|
36
|
-
allow(File).to receive(:
|
37
|
-
allow(File).to receive(:
|
35
|
+
allow(File).to receive(:exist?).with("/etc/init.d/chef").and_return(true)
|
36
|
+
allow(File).to receive(:exist?).with("/sbin/rc-update").and_return(true)
|
37
|
+
allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(false)
|
38
38
|
allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return(false)
|
39
39
|
end
|
40
40
|
# new test: found_enabled state
|
41
41
|
#
|
42
42
|
describe "load_current_resource" do
|
43
43
|
it "should raise Chef::Exceptions::Service if /sbin/rc-update does not exist" do
|
44
|
-
expect(File).to receive(:
|
44
|
+
expect(File).to receive(:exist?).with("/sbin/rc-update").and_return(false)
|
45
45
|
@provider.define_resource_requirements
|
46
46
|
expect { @provider.process_resource_requirements }.to raise_error(Chef::Exceptions::Service)
|
47
47
|
end
|
@@ -65,7 +65,7 @@ describe Chef::Provider::Service::Gentoo do
|
|
65
65
|
|
66
66
|
describe "and the file exists and is readable" do
|
67
67
|
before do
|
68
|
-
allow(File).to receive(:
|
68
|
+
allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(true)
|
69
69
|
allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return(true)
|
70
70
|
end
|
71
71
|
it "should set enabled to true" do
|
@@ -76,7 +76,7 @@ describe Chef::Provider::Service::Gentoo do
|
|
76
76
|
|
77
77
|
describe "and the file exists but is not readable" do
|
78
78
|
before do
|
79
|
-
allow(File).to receive(:
|
79
|
+
allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(true)
|
80
80
|
allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return(false)
|
81
81
|
end
|
82
82
|
|
@@ -88,7 +88,7 @@ describe Chef::Provider::Service::Gentoo do
|
|
88
88
|
|
89
89
|
describe "and the file does not exist" do
|
90
90
|
before do
|
91
|
-
allow(File).to receive(:
|
91
|
+
allow(File).to receive(:exist?).with("/etc/runlevels/default/chef").and_return(false)
|
92
92
|
allow(File).to receive(:readable?).with("/etc/runlevels/default/chef").and_return("foobarbaz")
|
93
93
|
end
|
94
94
|
|
@@ -86,7 +86,7 @@ describe Chef::Provider::Service::Macosx do
|
|
86
86
|
.with(/(#{su_cmd} '#{cmd}'|#{cmd})/, default_env: false)
|
87
87
|
.and_return(double("Status",
|
88
88
|
stdout: launchctl_stdout, exitstatus: 0))
|
89
|
-
allow(File).to receive(:
|
89
|
+
allow(File).to receive(:exist?).and_return([true], [])
|
90
90
|
allow(provider).to receive(:shell_out!)
|
91
91
|
.with(/plutil -convert xml1 -o/, default_env: false)
|
92
92
|
.and_return(double("Status", stdout: plutil_stdout))
|
@@ -112,7 +112,7 @@ describe Chef::Provider::Service::Macosx do
|
|
112
112
|
|
113
113
|
before do
|
114
114
|
allow(Dir).to receive(:glob).and_return([])
|
115
|
-
allow(File).to receive(:
|
115
|
+
allow(File).to receive(:exist?).and_return([true], [])
|
116
116
|
allow(provider).to receive(:shell_out!)
|
117
117
|
.with(/plutil -convert xml1 -o/)
|
118
118
|
.and_raise(Mixlib::ShellOut::ShellCommandFailed)
|
@@ -168,7 +168,7 @@ describe Chef::Provider::Service::Macosx do
|
|
168
168
|
describe "running unsupported actions" do
|
169
169
|
before do
|
170
170
|
allow(Dir).to receive(:glob).and_return([(plist).to_s], [])
|
171
|
-
allow(File).to receive(:
|
171
|
+
allow(File).to receive(:exist?).and_return([true], [])
|
172
172
|
end
|
173
173
|
it "should throw an exception when reload action is attempted" do
|
174
174
|
expect { provider.run_action(:reload) }.to raise_error(Chef::Exceptions::UnsupportedAction)
|
@@ -21,7 +21,7 @@ require "ostruct"
|
|
21
21
|
|
22
22
|
shared_examples_for "define_resource_requirements_common" do
|
23
23
|
it "should raise an error if /sbin/chkconfig does not exist" do
|
24
|
-
allow(File).to receive(:
|
24
|
+
allow(File).to receive(:exist?).with("/sbin/chkconfig").and_return(false)
|
25
25
|
allow(@provider).to receive(:shell_out).with("/sbin/service chef status").and_raise(Errno::ENOENT)
|
26
26
|
allow(@provider).to receive(:shell_out!).with("/sbin/chkconfig --list chef", returns: [0, 1]).and_raise(Errno::ENOENT)
|
27
27
|
@provider.load_current_resource
|
@@ -55,7 +55,7 @@ describe "Chef::Provider::Service::Redhat" do
|
|
55
55
|
@provider = Chef::Provider::Service::Redhat.new(@new_resource, @run_context)
|
56
56
|
@provider.action = :start
|
57
57
|
allow(Chef::Resource::Service).to receive(:new).and_return(@current_resource)
|
58
|
-
allow(File).to receive(:
|
58
|
+
allow(File).to receive(:exist?).with("/sbin/chkconfig").and_return(true)
|
59
59
|
end
|
60
60
|
|
61
61
|
describe "while not in why run mode" do
|
@@ -74,7 +74,7 @@ describe Chef::Provider::Service::Upstart do
|
|
74
74
|
@status = double("Status", exitstatus: 0, stdout: "", stderr: "")
|
75
75
|
allow(@provider).to receive(:shell_out).and_return(@status)
|
76
76
|
|
77
|
-
allow(::File).to receive(:
|
77
|
+
allow(::File).to receive(:exist?).and_return(true)
|
78
78
|
allow(::File).to receive(:open).and_return(true)
|
79
79
|
end
|
80
80
|
|
@@ -181,13 +181,13 @@ describe Chef::Provider::Service::Upstart do
|
|
181
181
|
end
|
182
182
|
|
183
183
|
it "should assume disable when no job configuration file is found" do
|
184
|
-
allow(::File).to receive(:
|
184
|
+
allow(::File).to receive(:exist?).and_return(false)
|
185
185
|
expect(@current_resource).to receive(:running).with(false)
|
186
186
|
@provider.load_current_resource
|
187
187
|
end
|
188
188
|
|
189
189
|
it "should track state when the upstart configuration file fails to load" do
|
190
|
-
expect(File).to receive(:
|
190
|
+
expect(File).to receive(:exist?).and_return false
|
191
191
|
@provider.load_current_resource
|
192
192
|
expect(@provider.instance_variable_get("@config_file_found")).to eq(false)
|
193
193
|
end
|
@@ -22,12 +22,8 @@ require "spec_helper"
|
|
22
22
|
describe Chef::Provider::Service::Windows, "load_current_resource" do
|
23
23
|
include_context "Win32"
|
24
24
|
|
25
|
-
before
|
26
|
-
Chef::ReservedNames::Win32::Security
|
27
|
-
end
|
28
|
-
|
29
|
-
after(:all) do
|
30
|
-
Chef::ReservedNames::Win32.send(:remove_const, :Security) unless windows?
|
25
|
+
before do
|
26
|
+
stub_const("Chef::ReservedNames::Win32::Security", Class.new) unless windows?
|
31
27
|
end
|
32
28
|
|
33
29
|
let(:logger) { double("Mixlib::Log::Child").as_null_object }
|
@@ -220,31 +220,35 @@ describe Chef::Provider::SystemdUnit do
|
|
220
220
|
expect(current_resource.content).to eq(nil)
|
221
221
|
end
|
222
222
|
|
223
|
-
|
224
|
-
|
225
|
-
|
226
|
-
|
227
|
-
.
|
228
|
-
|
229
|
-
|
230
|
-
|
231
|
-
|
232
|
-
|
233
|
-
|
234
|
-
.
|
235
|
-
|
236
|
-
|
237
|
-
|
223
|
+
# A password is required when specifying a user on Windows. Since systemd resources
|
224
|
+
# won't actually run on Windows, skip these tests rather than code a workaround.
|
225
|
+
unless windows?
|
226
|
+
it "loads the user unit content if the file exists and user is set" do
|
227
|
+
new_resource.user("joe")
|
228
|
+
allow(File).to receive(:exist?)
|
229
|
+
.with(unit_path_user)
|
230
|
+
.and_return(true)
|
231
|
+
allow(File).to receive(:read)
|
232
|
+
.with(unit_path_user)
|
233
|
+
.and_return(unit_content_string)
|
234
|
+
expect(File).to receive(:exist?)
|
235
|
+
.with(unit_path_user)
|
236
|
+
expect(File).to receive(:read)
|
237
|
+
.with(unit_path_user)
|
238
|
+
provider.load_current_resource
|
239
|
+
expect(current_resource.content).to eq(unit_content_string)
|
240
|
+
end
|
238
241
|
|
239
|
-
|
240
|
-
|
241
|
-
|
242
|
-
|
243
|
-
|
244
|
-
|
245
|
-
|
246
|
-
|
247
|
-
|
242
|
+
it "does not load the user unit if the file does not exist and user is set" do
|
243
|
+
new_resource.user("joe")
|
244
|
+
allow(File).to receive(:exist?)
|
245
|
+
.with(unit_path_user)
|
246
|
+
.and_return(false)
|
247
|
+
expect(File).to_not receive(:read)
|
248
|
+
.with(unit_path_user)
|
249
|
+
provider.load_current_resource
|
250
|
+
expect(current_resource.content).to eq(nil)
|
251
|
+
end
|
248
252
|
end
|
249
253
|
end
|
250
254
|
|