diff options
author | James Cammarata <jimi@sngx.net> | 2010-09-11 10:23:38 -0500 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2010-09-14 09:22:19 +1000 |
commit | 9bdfe694bc6c60a48b45f8dd49c20c6da31445f7 (patch) | |
tree | 70e91e31b36bdc27e4338cde6d0a6656b9ffbbb0 /test | |
parent | 14b33402812756dc0cd3e84619afa211d449d59f (diff) | |
download | puppet-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-x | test/ral/manager/attributes.rb | 5 |
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 |