From: Russ Cox Date: Mon, 14 Mar 2016 01:34:46 +0000 (-0400) Subject: os/user: allow LookupGroupId to fail during test X-Git-Tag: go1.7beta1~1323 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=80a051712981fb232567813e11899b592ddd51de;p=gostls13.git os/user: allow LookupGroupId to fail during test On my Mac I am in group 5000 which apparently has no name (I suspect because it is an LDAP group and I cannot reach the LDAP server). Do not make the test fail in that case. Fixes #14806 Change-Id: I56b11a8e86b048abfb00812eaad37802fd2adcc5 Reviewed-on: https://go-review.googlesource.com/20710 Reviewed-by: Brad Fitzpatrick --- diff --git a/src/os/user/user_test.go b/src/os/user/user_test.go index 361c48a7bd..9d8d94d8da 100644 --- a/src/os/user/user_test.go +++ b/src/os/user/user_test.go @@ -104,7 +104,11 @@ func TestLookupGroup(t *testing.T) { g1, err := LookupGroupId(user.Gid) if err != nil { - t.Fatalf("LookupGroupId(%q): %v", user.Gid, err) + // NOTE(rsc): Maybe the group isn't defined. That's fine. + // On my OS X laptop, rsc logs in with group 5000 even + // though there's no name for group 5000. Such is Unix. + t.Logf("LookupGroupId(%q): %v", user.Gid, err) + return } if g1.Gid != user.Gid { t.Errorf("LookupGroupId(%q).Gid = %s; want %s", user.Gid, g1.Gid, user.Gid)