From: Fred Drake <fdrake@acm.org>
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..."