test_urllib would set environment variable NO_PROXY without removing it afterwards.

diff --git a/Lib/test/test_urllib.py b/Lib/test/test_urllib.py
index 65687323cf64f4bcd3b095e2c4b9d0961e6d7aa5..a46f4219edfcc738dbe3dd457083d7a999cd12f2 100644 (file)
--- a/Lib/test/test_urllib.py
+++ b/Lib/test/test_urllib.py
@@ -130,10 +130,14 @@ class ProxyTests(unittest.TestCase):
             os.environ[k] = v
 
     def test_getproxies_environment_keep_no_proxies(self):
-        os.environ['NO_PROXY'] = 'localhost'
-        proxies = urllib.request.getproxies_environment()
-        # getproxies_environment use lowered case truncated (no '_proxy') keys
-        self.assertEquals('localhost', proxies['no'])
+        try:
+            os.environ['NO_PROXY'] = 'localhost'
+            proxies = urllib.request.getproxies_environment()
+            # getproxies_environment use lowered case truncated (no '_proxy') keys
+            self.assertEquals('localhost', proxies['no'])
+        finally:
+            # The old value will be restored by tearDown, if applicable.
+            del os.environ['NO_PROXY']
 
 
 class urlopen_HttpTests(unittest.TestCase):