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

TST: Rewrite JS tests to pytest #746

Merged
merged 2 commits into from
Apr 14, 2022
Merged

Conversation

MasterOdin
Copy link
Member

This rewrites the JS tests in Tests/tests.py to use the "pytest style". The other unittest class in this file was removed as it's duplicated in Tests/test_workflows.py. The end result is simplifying out the pytest run line so that it just operates over the Tests directory, and everything utilizes the pytest style of writing tests.

Signed-off-by: Matthew Peveler <[email protected]>
@MasterOdin MasterOdin force-pushed the mpeveler/chore-pytest-javascript branch from e6a6d0c to dee4e7e Compare April 14, 2022 14:45
Signed-off-by: Matthew Peveler <[email protected]>
@MartinThoma MartinThoma merged commit 0ea2301 into main Apr 14, 2022
@MartinThoma MartinThoma deleted the mpeveler/chore-pytest-javascript branch April 14, 2022 16:23
@MartinThoma
Copy link
Member

Very cool! Thank you 🙏

MartinThoma added a commit that referenced this pull request Apr 15, 2022
Security (SEC):

- ContentStream_readInlineImage had potential infinite loop (#740)

Bug fixes (BUG):

- Fix merging encrypted files (#757)
- CCITTFaxDecode decodeParms can be an ArrayObject (#756)

Robustness improvements (ROBUST):

- title sometimes None (#744)

Documentation (DOC):

- Adjust short description of the package

Tests and Test setup (TST):

- Rewrite JS tests from unittest to pytest (#746)
- Increase Test coverage, mainly with filters (#756)
- Add test for inline images (#758)

Developer Experience Improvements (DEV):

- Remove unused Travis-CI configuration (#747)
- Show code coverage (#754, #755)
- Add mutmut (#760)

Miscellaneous:

- STY: Closing file handles, explicit exports, ... (#743)

All changes: 1.27.4...1.27.5
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 this pull request may close these issues.

2 participants