Skip to content

Commit

Permalink
Fix the refactored test to not override parameters.
Browse files Browse the repository at this point in the history
Based on Serhiy's code review.  (thanks!)

Confirmed it still passes as root and non-root on Linux.
  • Loading branch information
gpshead committed Dec 4, 2023
1 parent 52a475c commit ce31462
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions Lib/test/test_subprocess.py
Original file line number Diff line number Diff line change
Expand Up @@ -2073,8 +2073,6 @@ def test_extra_groups_empty_list(self):
self._test_extra_groups_impl(gid=os.getegid(), group_list=[])

def _test_extra_groups_impl(self, *, gid, group_list):
gid = os.getegid()
group_list = [65534 if gid != 65534 else 65533]
name_group = _get_test_grp_name()

if grp is not None:
Expand Down

0 comments on commit ce31462

Please sign in to comment.