diff options
-rw-r--r-- | ChangeLog | 7 | ||||
-rw-r--r-- | test/etc/test_etc.rb | 17 |
2 files changed, 18 insertions, 6 deletions
@@ -1,3 +1,10 @@ +Mon Aug 27 20:19:49 2012 Hiroshi Shirosaki <h.shirosaki@gmail.com> + + * test/etc/test_etc.rb (TestEtc#test_getgrgid): fix for non unique GID. + No unixen systems guarantee that GID is unique. Etc.getgrgid would + not return the first entry in the order of Etc.group for shared GID. + [ruby-core:47312] [Bug #6935] + Mon Aug 27 18:19:36 2012 Koichi Sasada <ko1@atdot.net> * include/ruby/ruby.h (rb_float_value): optimize it. diff --git a/test/etc/test_etc.rb b/test/etc/test_etc.rb index 5bc8db431c..c105122af1 100644 --- a/test/etc/test_etc.rb +++ b/test/etc/test_etc.rb @@ -76,13 +76,18 @@ class TestEtc < Test::Unit::TestCase end def test_getgrgid - groups = {} - Etc.group do |s| - groups[s.gid] ||= s + # group database is not unique on GID, and which entry will be + # returned by getgrgid() is not specified. + groups = Hash.new {[]} + # on MacOSX, same entries are returned from /etc/group and Open + # Directory. + Etc.group {|s| groups[s.gid] |= [s]} + groups.each_pair do |gid, s| + assert_include(s, Etc.getgrgid(gid)) end - groups.each_value do |s| - assert_equal(s, Etc.getgrgid(s.gid)) - assert_equal(s, Etc.getgrgid) if Process.egid == s.gid + s = groups[Process.egid] + unless s.empty? + assert_include(s, Etc.getgrgid) end end |