summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEzio Melotti <ezio.melotti@gmail.com>2010-02-22 16:01:22 (GMT)
committerEzio Melotti <ezio.melotti@gmail.com>2010-02-22 16:01:22 (GMT)
commitd2a577d6234c2f09617fe8ba216156f78fade1ab (patch)
tree3cf304a4ac5db361522d9f3608153db4caae8610
parentf9793a36a490a2702da5b3055b757dc9882bbaa8 (diff)
downloadcpython-d2a577d6234c2f09617fe8ba216156f78fade1ab.zip
cpython-d2a577d6234c2f09617fe8ba216156f78fade1ab.tar.gz
cpython-d2a577d6234c2f09617fe8ba216156f78fade1ab.tar.bz2
Merged revisions 78316 via svnmerge from
svn+ssh://pythondev@svn.python.org/python/branches/py3k ........ r78316 | ezio.melotti | 2010-02-22 17:59:01 +0200 (Mon, 22 Feb 2010) | 1 line #7310: fix the repr() of os.environ ........
-rw-r--r--Lib/os.py10
-rw-r--r--Lib/test/test_os.py8
-rw-r--r--Misc/NEWS2
3 files changed, 20 insertions, 0 deletions
diff --git a/Lib/os.py b/Lib/os.py
index 5b8aa25..d72f32b 100644
--- a/Lib/os.py
+++ b/Lib/os.py
@@ -387,22 +387,32 @@ class _Environ(MutableMapping):
self.data = data = {}
for key, value in environ.items():
data[keymap(key)] = str(value)
+
def __getitem__(self, key):
return self.data[self.keymap(key)]
+
def __setitem__(self, key, value):
value = str(value)
self.putenv(key, value)
self.data[self.keymap(key)] = value
+
def __delitem__(self, key):
self.unsetenv(key)
del self.data[self.keymap(key)]
+
def __iter__(self):
for key in self.data:
yield key
+
def __len__(self):
return len(self.data)
+
+ def __repr__(self):
+ return 'environ({!r})'.format(self.data)
+
def copy(self):
return dict(self)
+
def setdefault(self, key, value):
if key not in self:
self[key] = value
diff --git a/Lib/test/test_os.py b/Lib/test/test_os.py
index 907f943..d7f763f 100644
--- a/Lib/test/test_os.py
+++ b/Lib/test/test_os.py
@@ -400,6 +400,14 @@ class EnvironTests(mapping_tests.BasicTestMappingProtocol):
for key, value in self._reference().items():
self.assertEqual(os.environ.get(key), value)
+ # Issue 7310
+ def test___repr__(self):
+ """Check that the repr() of os.environ looks like environ({...})."""
+ env = os.environ
+ self.assertTrue(isinstance(env.data, dict))
+ self.assertEqual(repr(env), 'environ({!r})'.format(env.data))
+
+
class WalkTests(unittest.TestCase):
"""Tests for os.walk()."""
diff --git a/Misc/NEWS b/Misc/NEWS
index 835bb92..9f1b145 100644
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -79,6 +79,8 @@ Core and Builtins
Library
-------
+- Issue #7310: fix the __repr__ of os.environ to show the environment variables.
+
- Issue #7970: email.Generator.flatten now correctly flattens message/rfc822
messages parsed by email.Parser.HeaderParser.