diff options
author | Ezio Melotti <ezio.melotti@gmail.com> | 2010-01-24 16:58:36 +0000 |
---|---|---|
committer | Ezio Melotti <ezio.melotti@gmail.com> | 2010-01-24 16:58:36 +0000 |
commit | b0f5adc3f4e098b1744eeac65fc830fb1df2b8ab (patch) | |
tree | e2936db9f8e1194464aeb7e9546d5a892f7239c7 /Lib/test/test_urllibnet.py | |
parent | f14c7fc33da8ba3aa28bbb7b0305ea12ab30be09 (diff) | |
download | cpython-git-b0f5adc3f4e098b1744eeac65fc830fb1df2b8ab.tar.gz |
use assert[Not]IsInstance where appropriate
Diffstat (limited to 'Lib/test/test_urllibnet.py')
-rw-r--r-- | Lib/test/test_urllibnet.py | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/Lib/test/test_urllibnet.py b/Lib/test/test_urllibnet.py index 7ab2aa2477..39286f9c91 100644 --- a/Lib/test/test_urllibnet.py +++ b/Lib/test/test_urllibnet.py @@ -71,10 +71,10 @@ class urlopenNetworkTests(unittest.TestCase): # Test both readline and readlines. open_url = self.urlopen("http://www.python.org/") try: - self.assertTrue(isinstance(open_url.readline(), basestring), - "readline did not return a string") - self.assertTrue(isinstance(open_url.readlines(), list), - "readlines did not return a list") + self.assertIsInstance(open_url.readline(), basestring, + "readline did not return a string") + self.assertIsInstance(open_url.readlines(), list, + "readlines did not return a list") finally: open_url.close() @@ -85,9 +85,9 @@ class urlopenNetworkTests(unittest.TestCase): info_obj = open_url.info() finally: open_url.close() - self.assertTrue(isinstance(info_obj, mimetools.Message), - "object returned by 'info' is not an instance of " - "mimetools.Message") + self.assertIsInstance(info_obj, mimetools.Message, + "object returned by 'info' is not an " + "instance of mimetools.Message") self.assertEqual(info_obj.getsubtype(), "html") def test_geturl(self): @@ -175,8 +175,8 @@ class urlretrieveNetworkTests(unittest.TestCase): # Make sure header returned as 2nd value from urlretrieve is good. file_location, header = self.urlretrieve("http://www.python.org/") os.unlink(file_location) - self.assertTrue(isinstance(header, mimetools.Message), - "header is not an instance of mimetools.Message") + self.assertIsInstance(header, mimetools.Message, + "header is not an instance of mimetools.Message") |