-
Notifications
You must be signed in to change notification settings - Fork 435
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
pytest integration #415
base: master
Are you sure you want to change the base?
pytest integration #415
Conversation
What's the reason for this? |
Mostly to leverage the pytest ecosystem (coverage, performance etc). Pytest interact well with the major IDEs, which is a nice bonus. I dropped this pull request to show some proof of concept and if there is any interest. |
This reverts commit d87a23e.
@nicholasserra now the test runs an all major platform / python versions https://github.com/cav71/python-markdown2/actions/runs/1904662952 and there is a branch ready with the coverage for it https://github.com/cav71/python-markdown2/tree/coverage-support |
This is "preliminary" pytest port of the testlib.py code, leveraging pytest and its ecosystem.
It supports running all (unchanged) tests (tm-cases, markdowntest-cases, php-markdown-cases and php-markdown-extra-cases).
pytest itself provides extensive support for code quality (coverage, unitest xml etc.).
This change also integrates the workflow actions.