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

lxml as BeautifulSoup parser #256

Open
yhay81 opened this issue Feb 5, 2024 · 2 comments
Open

lxml as BeautifulSoup parser #256

yhay81 opened this issue Feb 5, 2024 · 2 comments

Comments

@yhay81
Copy link

yhay81 commented Feb 5, 2024

Feature Request

I would like to propose the addition of an option to select lxml as the parser for BeautifulSoup responses, instead of the current fixed default of html.parser.

It will improve parsing speed for large or complex documents.

I believe this enhancement would make webtest even more versatile and powerful for a wide range of testing scenarios. I am looking forward to your thoughts on this proposal and am happy to contribute to the discussion or assist with preliminary testing.

Thank you for considering my suggestion and for your ongoing efforts to improve the webtest library.

@gawel
Copy link
Member

gawel commented Mar 4, 2024

Hi, sorry for the delay. Feel free to provide a PR. It will be merged if it doesn't change the current behavior and don't break tests :)

@brondsem
Copy link

You can pass parser_features='lxml', into the TestApp constructor. That's working for me

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

No branches or pull requests

3 participants