Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test fails with libxml2-2.10.4 #248

Closed
vcunat opened this issue Apr 26, 2023 · 4 comments · Fixed by #250
Closed

test fails with libxml2-2.10.4 #248

vcunat opened this issue Apr 26, 2023 · 4 comments · Fixed by #250

Comments

@vcunat
Copy link

vcunat commented Apr 26, 2023

Due to behavior changed in https://gitlab.gnome.org/GNOME/libxml2/-/commit/73210eeda5782da3414517cab20550d137e2dd25

=================================== FAILURES ===================================
_____________________ TestXMLNamespace.test_selector_html ______________________

self = <tests.test_pyquery.TestXMLNamespace testMethod=test_selector_html>

    def test_selector_html(self):
        expected = 'What'
        d = pq('blah', self.xml.split('?>', 1)[1], parser='html')
        val = d.text()
>       self.assertEqual(repr(val), repr(expected))
E       AssertionError: "''" != "'What'"
E       - ''
E       + 'What'

tests/test_pyquery.py:862: AssertionError
=========================== short test summary info ============================
FAILED tests/test_pyquery.py::TestXMLNamespace::test_selector_html - AssertionError: "''" != "'What'"
================= 1 failed, 149 passed, 1 deselected in 6.73s ==================
@mgorny
Copy link

mgorny commented Jul 31, 2023

Ping.

@gawel gawel reopened this Jul 31, 2023
@frenzymadness
Copy link

We have the same issue in Fedora Linux.

@gawel
Copy link
Owner

gawel commented Apr 2, 2024

I'm not sure it's the same issue but test are finally failing on gha (I wasn't able to reproduce the problem before that)

@gawel
Copy link
Owner

gawel commented Apr 2, 2024

I ended up removing the test... I think it's no longer possible to use the html parser with an xmlfile

see b0cc127

@gawel gawel closed this as completed Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants