From 839e5eafcb351178983c2f603152091d03d60831 Mon Sep 17 00:00:00 2001 From: Victor Stinner Date: Sun, 14 Apr 2013 16:43:03 +0200 Subject: Issue #17702: use assertRaises() for the unit test --- Lib/test/test_os.py | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/Lib/test/test_os.py b/Lib/test/test_os.py index 83373be..33b9e5b 100644 --- a/Lib/test/test_os.py +++ b/Lib/test/test_os.py @@ -636,19 +636,13 @@ class EnvironTests(mapping_tests.BasicTestMappingProtocol): missing = 'missingkey' self.assertNotIn(missing, os.environ) - try: + with self.assertRaises(KeyError) as cm: os.environ[missing] - except KeyError as err: - self.assertIs(err.args[0], missing) - else: - self.fail("KeyError not raised") + self.assertIs(cm.exception.args[0], missing) - try: + with self.assertRaises(KeyError) as cm: del os.environ[missing] - except KeyError as err: - self.assertIs(err.args[0], missing) - else: - self.fail("KeyError not raised") + self.assertIs(cm.exception.args[0], missing) class WalkTests(unittest.TestCase): """Tests for os.walk().""" -- cgit v0.12