[Python-checkins] r78718 - python/trunk/Lib/test/test_os.py

Sat Mar 6 08:35:20 CET 2010

Author: gregory.p.smith
Date: Sat Mar 6 08:35:19 2010
New Revision: 78718
Log:
Call setreuid and setregid in a subprocess to avoid altering the test runner's
process state. Should fix issue8045.
Modified:
 python/trunk/Lib/test/test_os.py
Modified: python/trunk/Lib/test/test_os.py
==============================================================================

--- python/trunk/Lib/test/test_os.py	(original)
+++ python/trunk/Lib/test/test_os.py	Sat Mar 6 08:35:19 2010
@@ -642,7 +642,14 @@
 self.assertRaises(os.error, os.setreuid, 0, 0)
 self.assertRaises(OverflowError, os.setreuid, 1<<32, 0)
 self.assertRaises(OverflowError, os.setreuid, 0, 1<<32)
- os.setreuid(-1, -1) # Does nothing, but it needs to accept -1
+
+ def test_setreuid_neg1(self):
+ # Needs to accept -1. We run this in a subprocess to avoid
+ # altering the test runner's process state (issue8045).
+ import subprocess
+ subprocess.check_call([
+ sys.executable, '-c',
+ 'import os,sys;os.setreuid(-1,-1);sys.exit(0)'])
 
 if hasattr(os, 'setregid'):
 def test_setregid(self):
@@ -650,7 +657,14 @@
 self.assertRaises(os.error, os.setregid, 0, 0)
 self.assertRaises(OverflowError, os.setregid, 1<<32, 0)
 self.assertRaises(OverflowError, os.setregid, 0, 1<<32)
- os.setregid(-1, -1) # Does nothing, but it needs to accept -1
+
+ def test_setregid_neg1(self):
+ # Needs to accept -1. We run this in a subprocess to avoid
+ # altering the test runner's process state (issue8045).
+ import subprocess
+ subprocess.check_call([
+ sys.executable, '-c',
+ 'import os,sys;os.setregid(-1,-1);sys.exit(0)'])
 else:
 class PosixUidGidTests(unittest.TestCase):
 pass
