[Python-checkins] r66301 - in python/trunk/Lib: cgi.py test/test_cgi.py
facundo.batista
python-checkins at python.org
Mon Sep 8 02:20:29 CEST 2008
Author: facundo.batista
Date: Mon Sep 8 02:20:28 2008
New Revision: 66301
Log:
Issue 3801. Fixing a dumb error in the deprecated parse_qsl()
function. Tests added.
Modified:
python/trunk/Lib/cgi.py
python/trunk/Lib/test/test_cgi.py
Modified: python/trunk/Lib/cgi.py
==============================================================================
--- python/trunk/Lib/cgi.py (original)
+++ python/trunk/Lib/cgi.py Mon Sep 8 02:20:28 2008
@@ -189,7 +189,7 @@
"""Parse a query given as a string argument."""
warn("cgi.parse_qsl is deprecated, use urlparse.parse_qsl instead",
PendingDeprecationWarning)
- return urlparse.parse_qs(qs, keep_blank_values, strict_parsing)
+ return urlparse.parse_qsl(qs, keep_blank_values, strict_parsing)
def parse_multipart(fp, pdict):
"""Parse multipart input.
Modified: python/trunk/Lib/test/test_cgi.py
==============================================================================
--- python/trunk/Lib/test/test_cgi.py (original)
+++ python/trunk/Lib/test/test_cgi.py Mon Sep 8 02:20:28 2008
@@ -344,6 +344,17 @@
v = gen_result(data, environ)
self.assertEqual(result, v)
+ def test_deprecated_parse_qs(self):
+ # this func is moved to urlparse, this is just a sanity check
+ self.assertEqual({'a': ['A1'], 'B': ['B3'], 'b': ['B2']},
+ cgi.parse_qs('a=A1&b=B2&B=B3'))
+
+ def test_deprecated_parse_qsl(self):
+ # this func is moved to urlparse, this is just a sanity check
+ self.assertEqual([('a', 'A1'), ('b', 'B2'), ('B', 'B3')],
+ cgi.parse_qsl('a=A1&b=B2&B=B3'))
+
+
def test_main():
run_unittest(CgiTests)
More information about the Python-checkins
mailing list