openstack/swift
Code Issues Proposed changes

Drop group comparison from drop_privileges test

Browse Source 
Drop the group comparison from drop_privileges test as it isn't
valid since os.setgroups() is mocked.

Change-Id: Ida15e72ae4ecdc2d6ce0d37bd99c2d86bd4e5ddc
Closes-Bug: #1724342
This commit is contained in:
Corey Bryant 2017-10-17 15:04:45 -04:00 committed by Clay Gerrard
parent ae2a75ad52
commit 0f91c862e1
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/unit/common/test_utils.py
View File

@ -25,7 +25,6 @@ import eventlet.debug
import eventlet.event
import eventlet.patcher
import functools
import grp
import logging
import platform
import os
@ -2144,10 +2143,6 @@ log_name = %(yarr)s'''
import pwd
self.assertEqual(pwd.getpwnam(user)[5], utils.os.environ['HOME'])
groups = [g.gr_gid for g in grp.getgrall() if user in g.gr_mem]
groups.append(pwd.getpwnam(user).pw_gid)
self.assertEqual(set(groups), set(os.getgroups()))
# reset; test same args, OSError trying to get session leader
utils.os = MockOs(called_funcs=required_func_calls,
raise_funcs=('setsid',))