From 9cbbf0d65cacb37d561d1d3ac93840def2dad36c Mon Sep 17 00:00:00 2001 From: Bryan McLellan Date: Mon, 13 Oct 2014 16:31:40 -0400 Subject: windows_package installer_type should be a symbol Fixes #1997 --- lib/chef/resource/windows_package.rb | 2 +- spec/unit/resource/windows_package_spec.rb | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/chef/resource/windows_package.rb b/lib/chef/resource/windows_package.rb index 8bd41e0cb7..4163b2f98d 100644 --- a/lib/chef/resource/windows_package.rb +++ b/lib/chef/resource/windows_package.rb @@ -44,7 +44,7 @@ class Chef set_or_return( :installer_type, arg, - :kind_of => [ String ] + :kind_of => [ Symbol ] ) end diff --git a/spec/unit/resource/windows_package_spec.rb b/spec/unit/resource/windows_package_spec.rb index c9ef8d910c..306869bdc8 100644 --- a/spec/unit/resource/windows_package_spec.rb +++ b/spec/unit/resource/windows_package_spec.rb @@ -34,9 +34,9 @@ describe Chef::Resource::WindowsPackage, "initialize", :windows_only do expect(resource.provider).to eql(Chef::Provider::Package::Windows) end - it "supports setting installer_type" do - resource.installer_type("msi") - expect(resource.installer_type).to eql("msi") + it "supports setting installer_type as a symbol" do + resource.installer_type(:msi) + expect(resource.installer_type).to eql(:msi) end # String, Integer -- cgit v1.2.1