From fa6cecbc68f4b2daa5452baeb869245335e1eb03 Mon Sep 17 00:00:00 2001 From: Senthil Kumaran Date: Sat, 11 Jan 2014 22:16:55 -0800 Subject: [PATCH] Adding test coverage for cgi.FieldStorage based on the scenario mentioned in issue #19097 --- Lib/test/test_cgi.py | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/Lib/test/test_cgi.py b/Lib/test/test_cgi.py index 7fdd482b94..349d5e9b40 100644 --- a/Lib/test/test_cgi.py +++ b/Lib/test/test_cgi.py @@ -5,6 +5,8 @@ import sys import tempfile import unittest +from collections import namedtuple + class HackedSysModule: # The regression test will have real values in sys.argv, which # will completely confuse the test of the cgi module @@ -232,6 +234,14 @@ class CgiTests(unittest.TestCase): # as long as the chunksize is 1 << 16. self.assertTrue(f.numcalls > 2) + def test_fieldstorage_invalid(self): + fs = cgi.FieldStorage() + self.assertFalse(fs) + self.assertRaises(TypeError, bool(fs)) + self.assertEqual(list(fs), list(fs.keys())) + fs.list.append(namedtuple('MockFieldStorage', 'name')('fieldvalue')) + self.assertTrue(fs) + def test_fieldstorage_multipart(self): #Test basic FieldStorage multipart parsing env = {'REQUEST_METHOD':'POST', 'CONTENT_TYPE':'multipart/form-data; boundary=---------------------------721837373350705526688164684', 'CONTENT_LENGTH':'558'} -- 2.50.1