summaryrefslogtreecommitdiff
path: root/Lib/test
diff options
context:
space:
mode:
authorRaymond Hettinger <python@rcn.com>2016-09-18 21:45:11 -0700
committerRaymond Hettinger <python@rcn.com>2016-09-18 21:45:11 -0700
commit7eb1becc2502723ebcdcc56972e41c03ef80d6af (patch)
tree904f60ee0ff01a8be376642190613344ef297459 /Lib/test
parent96b2dd5997735f417526758cbe3ab497850baf71 (diff)
downloadcpython-git-7eb1becc2502723ebcdcc56972e41c03ef80d6af.tar.gz
Issue #28189: dictitems_contains no longer swallows compare errors.
(Patch by Xiang Zhang)
Diffstat (limited to 'Lib/test')
-rw-r--r--Lib/test/test_dictviews.py26
1 files changed, 26 insertions, 0 deletions
diff --git a/Lib/test/test_dictviews.py b/Lib/test/test_dictviews.py
index ab23ca1cf7..0c996526d8 100644
--- a/Lib/test/test_dictviews.py
+++ b/Lib/test/test_dictviews.py
@@ -209,6 +209,32 @@ class DictSetTest(unittest.TestCase):
self.assertRaises(TypeError, copy.copy, d.values())
self.assertRaises(TypeError, copy.copy, d.items())
+ def test_compare_error(self):
+ class Exc(Exception):
+ pass
+
+ class BadEq:
+ def __hash__(self):
+ return 7
+ def __eq__(self, other):
+ raise Exc
+
+ k1, k2 = BadEq(), BadEq()
+ v1, v2 = BadEq(), BadEq()
+ d = {k1: v1}
+
+ self.assertIn(k1, d)
+ self.assertIn(k1, d.keys())
+ self.assertIn(v1, d.values())
+ self.assertIn((k1, v1), d.items())
+
+ self.assertRaises(Exc, d.__contains__, k2)
+ self.assertRaises(Exc, d.keys().__contains__, k2)
+ self.assertRaises(Exc, d.items().__contains__, (k2, v1))
+ self.assertRaises(Exc, d.items().__contains__, (k1, v2))
+ with self.assertRaises(Exc):
+ v2 in d.values()
+
def test_pickle(self):
d = {1: 10, "a": "ABC"}
for proto in range(pickle.HIGHEST_PROTOCOL + 1):