]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
[3.11] gh-100933: Improve `check_element` helper in `test_xml_etree` (GH-100934)...
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Wed, 8 Feb 2023 11:11:54 +0000 (03:11 -0800)
committerGitHub <noreply@github.com>
Wed, 8 Feb 2023 11:11:54 +0000 (12:11 +0100)
Items checked by this test are always `str` and `dict` instances.
(cherry picked from commit eb49d32b9af0b3b01a5588626179187f11d145c9)

Co-authored-by: Nikita Sobolev <mail@sobolevn.me>
Lib/test/test_xml_etree.py

index 63b6725af89704f66c6c0300e100cfdacb525597..abf3c62156a94e339d1b08bc1a8539b44f07c3a8 100644 (file)
@@ -203,25 +203,6 @@ class ElementTreeTest(unittest.TestCase):
     def test_interface(self):
         # Test element tree interface.
 
-        def check_string(string):
-            len(string)
-            for char in string:
-                self.assertEqual(len(char), 1,
-                        msg="expected one-character string, got %r" % char)
-            new_string = string + ""
-            new_string = string + " "
-            string[:0]
-
-        def check_mapping(mapping):
-            len(mapping)
-            keys = mapping.keys()
-            items = mapping.items()
-            for key in keys:
-                item = mapping[key]
-            mapping["key"] = "value"
-            self.assertEqual(mapping["key"], "value",
-                    msg="expected value string, got %r" % mapping["key"])
-
         def check_element(element):
             self.assertTrue(ET.iselement(element), msg="not an element")
             direlem = dir(element)
@@ -231,12 +212,12 @@ class ElementTreeTest(unittest.TestCase):
                 self.assertIn(attr, direlem,
                         msg='no %s visible by dir' % attr)
 
-            check_string(element.tag)
-            check_mapping(element.attrib)
+            self.assertIsInstance(element.tag, str)
+            self.assertIsInstance(element.attrib, dict)
             if element.text is not None:
-                check_string(element.text)
+                self.assertIsInstance(element.text, str)
             if element.tail is not None:
-                check_string(element.tail)
+                self.assertIsInstance(element.tail, str)
             for elem in element:
                 check_element(elem)