Compare commits
No commits in common. 'c9' and 'i8-beta-stream-3.9' have entirely different histories.
c9
...
i8-beta-st
@ -0,0 +1,68 @@
|
|||||||
|
From dcbc52a26e661fe879e30e352103ed0fbafba908 Mon Sep 17 00:00:00 2001
|
||||||
|
From: tigro <tigro@msvsphere-os.ru>
|
||||||
|
Date: Tue, 28 May 2024 17:42:37 +0300
|
||||||
|
Subject: [PATCH] Backport test_simple_xml function
|
||||||
|
|
||||||
|
---
|
||||||
|
src/lxml/tests/test_elementtree.py | 45 +++++++++++++++---------------
|
||||||
|
1 file changed, 22 insertions(+), 23 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/lxml/tests/test_elementtree.py b/src/lxml/tests/test_elementtree.py
|
||||||
|
index 96b043d..85a1386 100644
|
||||||
|
--- a/src/lxml/tests/test_elementtree.py
|
||||||
|
+++ b/src/lxml/tests/test_elementtree.py
|
||||||
|
@@ -4347,29 +4347,28 @@ class _XMLPullParserTest(unittest.TestCase):
|
||||||
|
self.assertEqual([(action, elem.tag) for action, elem in events],
|
||||||
|
expected)
|
||||||
|
|
||||||
|
- def test_simple_xml(self):
|
||||||
|
- for chunk_size in (None, 1, 5):
|
||||||
|
- #with self.subTest(chunk_size=chunk_size):
|
||||||
|
- parser = self.etree.XMLPullParser()
|
||||||
|
- self.assert_event_tags(parser, [])
|
||||||
|
- self._feed(parser, "<!-- comment -->\n", chunk_size)
|
||||||
|
- self.assert_event_tags(parser, [])
|
||||||
|
- self._feed(parser,
|
||||||
|
- "<root>\n <element key='value'>text</element",
|
||||||
|
- chunk_size)
|
||||||
|
- self.assert_event_tags(parser, [])
|
||||||
|
- self._feed(parser, ">\n", chunk_size)
|
||||||
|
- self.assert_event_tags(parser, [('end', 'element')])
|
||||||
|
- self._feed(parser, "<element>text</element>tail\n", chunk_size)
|
||||||
|
- self._feed(parser, "<empty-element/>\n", chunk_size)
|
||||||
|
- self.assert_event_tags(parser, [
|
||||||
|
- ('end', 'element'),
|
||||||
|
- ('end', 'empty-element'),
|
||||||
|
- ])
|
||||||
|
- self._feed(parser, "</root>\n", chunk_size)
|
||||||
|
- self.assert_event_tags(parser, [('end', 'root')])
|
||||||
|
- root = self._close_and_return_root(parser)
|
||||||
|
- self.assertEqual(root.tag, 'root')
|
||||||
|
+
|
||||||
|
+ def test_simple_xml(self, chunk_size=None):
|
||||||
|
+ parser = self.etree.XMLPullParser()
|
||||||
|
+ self.assert_event_tags(parser, [])
|
||||||
|
+ self._feed(parser, "<!-- comment -->\n", chunk_size)
|
||||||
|
+ self.assert_event_tags(parser, [])
|
||||||
|
+ self._feed(parser,
|
||||||
|
+ "<root>\n <element key='value'>text</element",
|
||||||
|
+ chunk_size)
|
||||||
|
+ self.assert_event_tags(parser, [])
|
||||||
|
+ self._feed(parser, ">\n", chunk_size)
|
||||||
|
+ self._feed(parser, "<element>text</element>tail\n", chunk_size)
|
||||||
|
+ self._feed(parser, "<empty-element/>\n", chunk_size)
|
||||||
|
+ self._feed(parser, "</root>\n", chunk_size)
|
||||||
|
+ self.assert_event_tags(parser, [
|
||||||
|
+ ('end', 'element'),
|
||||||
|
+ ('end', 'element'),
|
||||||
|
+ ('end', 'empty-element'),
|
||||||
|
+ ('end', 'root'),
|
||||||
|
+ ])
|
||||||
|
+ root = self._close_and_return_root(parser)
|
||||||
|
+ self.assertEqual(root.tag, 'root')
|
||||||
|
|
||||||
|
def test_feed_while_iterating(self):
|
||||||
|
parser = self.etree.XMLPullParser()
|
||||||
|
--
|
||||||
|
2.45.1
|
||||||
|
|
||||||
@ -1,104 +0,0 @@
|
|||||||
diff --git a/src/lxml/apihelpers.pxi b/src/lxml/apihelpers.pxi
|
|
||||||
index 5eb3416..88a031d 100644
|
|
||||||
--- a/src/lxml/apihelpers.pxi
|
|
||||||
+++ b/src/lxml/apihelpers.pxi
|
|
||||||
@@ -246,9 +246,10 @@ cdef dict _build_nsmap(xmlNode* c_node):
|
|
||||||
while c_node is not NULL and c_node.type == tree.XML_ELEMENT_NODE:
|
|
||||||
c_ns = c_node.nsDef
|
|
||||||
while c_ns is not NULL:
|
|
||||||
- prefix = funicodeOrNone(c_ns.prefix)
|
|
||||||
- if prefix not in nsmap:
|
|
||||||
- nsmap[prefix] = funicodeOrNone(c_ns.href)
|
|
||||||
+ if c_ns.prefix or c_ns.href:
|
|
||||||
+ prefix = funicodeOrNone(c_ns.prefix)
|
|
||||||
+ if prefix not in nsmap:
|
|
||||||
+ nsmap[prefix] = funicodeOrNone(c_ns.href)
|
|
||||||
c_ns = c_ns.next
|
|
||||||
c_node = c_node.parent
|
|
||||||
return nsmap
|
|
||||||
diff --git a/src/lxml/includes/xmlparser.pxd b/src/lxml/includes/xmlparser.pxd
|
|
||||||
index a196e34..45acfc8 100644
|
|
||||||
--- a/src/lxml/includes/xmlparser.pxd
|
|
||||||
+++ b/src/lxml/includes/xmlparser.pxd
|
|
||||||
@@ -144,6 +144,7 @@ cdef extern from "libxml/parser.h":
|
|
||||||
void* userData
|
|
||||||
int* spaceTab
|
|
||||||
int spaceMax
|
|
||||||
+ int nsNr
|
|
||||||
bint html
|
|
||||||
bint progressive
|
|
||||||
int inSubset
|
|
||||||
diff --git a/src/lxml/iterparse.pxi b/src/lxml/iterparse.pxi
|
|
||||||
index 4c20506..3da7485 100644
|
|
||||||
--- a/src/lxml/iterparse.pxi
|
|
||||||
+++ b/src/lxml/iterparse.pxi
|
|
||||||
@@ -419,7 +419,7 @@ cdef int _countNsDefs(xmlNode* c_node):
|
|
||||||
count = 0
|
|
||||||
c_ns = c_node.nsDef
|
|
||||||
while c_ns is not NULL:
|
|
||||||
- count += 1
|
|
||||||
+ count += (c_ns.href is not NULL)
|
|
||||||
c_ns = c_ns.next
|
|
||||||
return count
|
|
||||||
|
|
||||||
@@ -430,9 +430,10 @@ cdef int _appendStartNsEvents(xmlNode* c_node, list event_list) except -1:
|
|
||||||
count = 0
|
|
||||||
c_ns = c_node.nsDef
|
|
||||||
while c_ns is not NULL:
|
|
||||||
- ns_tuple = (funicode(c_ns.prefix) if c_ns.prefix is not NULL else '',
|
|
||||||
- funicode(c_ns.href))
|
|
||||||
- event_list.append( (u"start-ns", ns_tuple) )
|
|
||||||
- count += 1
|
|
||||||
+ if c_ns.href:
|
|
||||||
+ ns_tuple = (funicodeOrEmpty(c_ns.prefix),
|
|
||||||
+ funicode(c_ns.href))
|
|
||||||
+ event_list.append( (u"start-ns", ns_tuple) )
|
|
||||||
+ count += 1
|
|
||||||
c_ns = c_ns.next
|
|
||||||
return count
|
|
||||||
diff --git a/src/lxml/parser.pxi b/src/lxml/parser.pxi
|
|
||||||
index 3ed223b..f5ff6b2 100644
|
|
||||||
--- a/src/lxml/parser.pxi
|
|
||||||
+++ b/src/lxml/parser.pxi
|
|
||||||
@@ -569,6 +569,9 @@ cdef class _ParserContext(_ResolverContext):
|
|
||||||
self._c_ctxt.disableSAX = 0 # work around bug in libxml2
|
|
||||||
else:
|
|
||||||
xmlparser.xmlClearParserCtxt(self._c_ctxt)
|
|
||||||
+ # work around bug in libxml2 [2.9.10 .. 2.9.14]:
|
|
||||||
+ # https://gitlab.gnome.org/GNOME/libxml2/-/issues/378
|
|
||||||
+ self._c_ctxt.nsNr = 0
|
|
||||||
|
|
||||||
cdef int prepare(self, bint set_document_loader=True) except -1:
|
|
||||||
cdef int result
|
|
||||||
diff --git a/src/lxml/tests/test_etree.py b/src/lxml/tests/test_etree.py
|
|
||||||
index 42613dc..db1f560 100644
|
|
||||||
--- a/src/lxml/tests/test_etree.py
|
|
||||||
+++ b/src/lxml/tests/test_etree.py
|
|
||||||
@@ -1459,6 +1459,27 @@ class ETreeOnlyTestCase(HelperTestCase):
|
|
||||||
[1,2,1,4],
|
|
||||||
counts)
|
|
||||||
|
|
||||||
+ def test_walk_after_parse_failure(self):
|
|
||||||
+ # This used to be an issue because libxml2 can leak empty namespaces
|
|
||||||
+ # between failed parser runs. iterwalk() failed to handle such a tree.
|
|
||||||
+ parser = etree.XMLParser()
|
|
||||||
+
|
|
||||||
+ try:
|
|
||||||
+ etree.XML('''<anot xmlns="1">''', parser=parser)
|
|
||||||
+ except etree.XMLSyntaxError:
|
|
||||||
+ pass
|
|
||||||
+ else:
|
|
||||||
+ assert False, "invalid input did not fail to parse"
|
|
||||||
+
|
|
||||||
+ et = etree.XML('''<root> </root>''', parser=parser)
|
|
||||||
+ try:
|
|
||||||
+ ns = next(etree.iterwalk(et, events=('start-ns',)))
|
|
||||||
+ except StopIteration:
|
|
||||||
+ # This would be the expected result, because there was no namespace
|
|
||||||
+ pass
|
|
||||||
+ else:
|
|
||||||
+ assert False, "Found unexpected namespace '%s'" % ns
|
|
||||||
+
|
|
||||||
def test_itertext_comment_pi(self):
|
|
||||||
# https://bugs.launchpad.net/lxml/+bug/1844674
|
|
||||||
XML = self.etree.XML
|
|
||||||
Loading…
Reference in new issue