summaryrefslogtreecommitdiffstats
path: root/spec
diff options
context:
space:
mode:
authorMatt Robinson <matt@puppetlabs.com>2011-04-14 11:33:03 -0700
committerMatt Robinson <matt@puppetlabs.com>2011-04-14 11:51:06 -0700
commit174e87a9b150a06a4ff9d696a6008fc08b05568b (patch)
tree8b36561cabe518b65a67deb0a2fcf41a11dda672 /spec
parentcb552af357763310ca4a16bf514dd39fcbd203bf (diff)
downloadpuppet-174e87a9b150a06a4ff9d696a6008fc08b05568b.tar.gz
puppet-174e87a9b150a06a4ff9d696a6008fc08b05568b.tar.xz
puppet-174e87a9b150a06a4ff9d696a6008fc08b05568b.zip
(#4258) Fix pkgutil spec test to have the correct provider
The resource being used for testing didn't explicitly set the provider, so it ended up using whatever the default provider was on the system on which it was run. This was problematic when running the specs on a Mac since the default provider is pkgdmg and that provider doesn't seem to be upgradeable. So when you tried: @resource[:ensure] = :latest You got the error: 1) Puppet::Type::Package::ProviderPkgutil when installing should use a command without versioned package Failure/Error: @resource[:ensure] = :latest Puppet::Error: Parameter ensure failed: Provider must have features 'upgradeable' to set 'ensure' to 'latest' # ./lib/puppet/parameter.rb:171:in `fail' # ./lib/puppet/parameter.rb:257:in `validate' # ./lib/puppet/property.rb:300:in `should=' # ./lib/puppet/property.rb:300:in `each' # ./lib/puppet/property.rb:300:in `should=' # ./lib/puppet/property.rb:337:in `value=' # ./lib/puppet/type.rb:416:in `[]=' # ./spec/unit/provider/package/pkgutil_spec.rb:35 Reviewed-by: Daniel Pittman <daniel@puppetlabs.com>
Diffstat (limited to 'spec')
-rwxr-xr-xspec/unit/provider/package/pkgutil_spec.rb7
1 files changed, 5 insertions, 2 deletions
diff --git a/spec/unit/provider/package/pkgutil_spec.rb b/spec/unit/provider/package/pkgutil_spec.rb
index f1d21f450..5549b3f6d 100755
--- a/spec/unit/provider/package/pkgutil_spec.rb
+++ b/spec/unit/provider/package/pkgutil_spec.rb
@@ -6,8 +6,11 @@ provider = Puppet::Type.type(:package).provider(:pkgutil)
describe provider do
before(:each) do
- @resource = stub 'resource'
- @resource = Puppet::Type.type(:package).new(:name => "TESTpkg", :ensure => :present)
+ @resource = Puppet::Type.type(:package).new(
+ :name => "TESTpkg",
+ :ensure => :present,
+ :provider => :pkgutil
+ )
@provider = provider.new(@resource)
# Stub all file and config tests