diff options
author | James Turnbull <james@lovedthanlost.net> | 2008-07-29 08:07:56 +1000 |
---|---|---|
committer | James Turnbull <james@lovedthanlost.net> | 2008-07-29 08:08:36 +1000 |
commit | 97987a705da7b8126569b1f5b7c3676ad0220f66 (patch) | |
tree | c6d7cf5ae961eb478115f11cdea2660ea497b75a /test/util/posixtest.rb | |
parent | fe99828511afbf701aa03f8dbf6d725020539602 (diff) | |
download | puppet-97987a705da7b8126569b1f5b7c3676ad0220f66.tar.gz puppet-97987a705da7b8126569b1f5b7c3676ad0220f66.tar.xz puppet-97987a705da7b8126569b1f5b7c3676ad0220f66.zip |
Feature #1241 : Improve performance of group lookups
Diffstat (limited to 'test/util/posixtest.rb')
-rwxr-xr-x | test/util/posixtest.rb | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/test/util/posixtest.rb b/test/util/posixtest.rb index 34d68e3a2..f64a95d18 100755 --- a/test/util/posixtest.rb +++ b/test/util/posixtest.rb @@ -25,16 +25,29 @@ class TestPosixUtil < Test::Unit::TestCase def test_get_posix_field {:group => nonrootgroup, :passwd => nonrootuser}.each do |space, obj| id = Puppet::Util.idfield(space) - [obj.name, obj.send(id), obj.send(id).to_s].each do |test| + [obj.name, obj.send(id)].each do |test| value = nil assert_nothing_raised do value = get_posix_field(space, :name, test) end - assert_equal(obj.name, value, "did not get correct value from get_posix_field") + assert_equal(obj.name, value, "did not get correct value from get_posix_field (known to be broken on some platforms)") end end end + def test_search_posix_field + {:group => nonrootgroup, :passwd => nonrootuser}.each do |space, obj| + id = Puppet::Util.idfield(space) + [obj.name, obj.send(id)].each do |test| + value = nil + assert_nothing_raised do + value = search_posix_field(space, :name, test) + end + assert_equal(obj.name, value, "did not get correct value from search_posix_field") + end + end + end + def test_get_provider_value user = nonrootuser obj = mk_posix_resource(:user, user) |