summaryrefslogtreecommitdiffstats
path: root/test
diff options
context:
space:
mode:
authorJames Cammarata <jimi@sngx.net>2010-09-11 10:23:38 -0500
committerJames Turnbull <james@lovedthanlost.net>2010-09-14 09:22:19 +1000
commit9bdfe694bc6c60a48b45f8dd49c20c6da31445f7 (patch)
tree70e91e31b36bdc27e4338cde6d0a6656b9ffbbb0 /test
parent14b33402812756dc0cd3e84619afa211d449d59f (diff)
downloadpuppet-9bdfe694bc6c60a48b45f8dd49c20c6da31445f7.tar.gz
puppet-9bdfe694bc6c60a48b45f8dd49c20c6da31445f7.tar.xz
puppet-9bdfe694bc6c60a48b45f8dd49c20c6da31445f7.zip
Fix for Bug #4756 - Providers no longer respect missing features
Restored deleted lines from type.rb and reinstated unit tests
Diffstat (limited to 'test')
-rwxr-xr-xtest/ral/manager/attributes.rb5
1 files changed, 4 insertions, 1 deletions
diff --git a/test/ral/manager/attributes.rb b/test/ral/manager/attributes.rb
index 74a4d0708..6d0284d9e 100755
--- a/test/ral/manager/attributes.rb
+++ b/test/ral/manager/attributes.rb
@@ -229,7 +229,10 @@ class TestTypeAttributes < Test::Unit::TestCase
end
yes.each { |a| assert(resource.should(a), "Did not get value for #{a} in #{prov.name}") }
no.each do |a|
- # These may or may not get passed to the provider. We shouldn't care.
+ assert_nil(resource.should(a), "Got value for unsupported %s in %s" % [a, prov.name])
+ if Puppet::Util::Log.sendlevel?(:info)
+ assert(@logs.find { |l| l.message =~ /not managing attribute #{a}/ and l.level == :info }, "No warning about failed %s" % a)
+ end
end
@logs.clear