From: Fred Drake Date: Mon, 4 Dec 2000 16:30:40 +0000 (+0000) Subject: Add test cases for ConfigParser.remove_option() behavior. This includes X-Git-Tag: v2.1a1~665 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=3d5f7e83c74c1c2fa20cf23d38020bbefb69916e;p=python Add test cases for ConfigParser.remove_option() behavior. This includes coverage to ensure bug #124324 does not re-surface. --- diff --git a/Lib/test/test_cfgparser.py b/Lib/test/test_cfgparser.py index 00b84659b6..bd06d3bb85 100644 --- a/Lib/test/test_cfgparser.py +++ b/Lib/test/test_cfgparser.py @@ -1,6 +1,9 @@ import ConfigParser import StringIO +from test_support import TestFailed + + def basic(src): print print "Testing basic accessors..." @@ -25,6 +28,27 @@ def basic(src): else: print '__name__ "option" properly hidden by the API.' + # Make sure the right things happen for remove_option(); + # added to include check for SourceForge bug #123324: + if not cf.remove_option('Foo Bar', 'foo'): + raise TestFailed( + "remove_option() failed to report existance of option") + if cf.has_option('Foo Bar', 'foo'): + raise TestFailed("remove_option() failed to remove option") + if cf.remove_option('Foo Bar', 'foo'): + raise TestFailed( + "remove_option() failed to report non-existance of option" + " that was removed") + try: + cf.remove_option('No Such Section', 'foo') + except ConfigParser.NoSectionError: + pass + else: + raise TestFailed( + "remove_option() failed to report non-existance of option" + " that never existed") + + def interpolation(src): print print "Testing value interpolation..."