Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Unified Diff: gmerge_test.py

Issue 6850002: Fix to replace current env with a blank one. (Closed) Base URL: http://git.chromium.org/git/dev-util.git@master
Patch Set: Created 9 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gmerge_test.py
diff --git a/gmerge_test.py b/gmerge_test.py
index 912d8cc0248fd8995e27f437dc3df732e10a18ac..c65d6317d0ae805050e59260e83f69f3bd64b943 100755
--- a/gmerge_test.py
+++ b/gmerge_test.py
@@ -30,7 +30,8 @@ class GMergeTest(unittest.TestCase):
merger.lsb_release)
def testPostData(self):
- original_use = os.environ.get('USE', '')
+ old_env = os.environ
+ os.environ = {}
os.environ['USE'] = 'a b c d +e'
gmerge.FLAGS = Flags({'accept_stable': 'blah'})
@@ -38,7 +39,7 @@ class GMergeTest(unittest.TestCase):
self.assertEqual(
'use=a+b+c+d+%2Be&pkg=package_name&board=x86-mario&accept_stable=blah',
merger.GeneratePackageRequest('package_name'))
- os.environ['USE'] = original_use
+ os.environ = old_env
if __name__ == '__main__':
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698