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: Increase Test coverage #756

Merged
merged 13 commits into from Apr 15, 2022
Merged

TST: Increase Test coverage #756

merged 13 commits into from Apr 15, 2022

Conversation

MartinThoma
Copy link
Member

No description provided.

@MartinThoma MartinThoma changed the title Black + xmp test TST: Increase Test coverage Apr 15, 2022
@codecov-commenter
Copy link

codecov-commenter commented Apr 15, 2022

Codecov Report

Merging #756 (ce82f40) into main (9d53ee8) will increase coverage by 3.59%.
The diff coverage is 47.36%.

@@            Coverage Diff             @@
##             main     #756      +/-   ##
==========================================
+ Coverage   62.63%   66.23%   +3.59%     
==========================================
  Files           9        9              
  Lines        3300     3311      +11     
  Branches      779      781       +2     
==========================================
+ Hits         2067     2193     +126     
+ Misses        985      882     -103     
+ Partials      248      236      -12     
Impacted Files Coverage Δ
PyPDF2/merger.py 72.47% <ø> (+12.54%) ⬆️
PyPDF2/pdf.py 69.69% <0.00%> (+0.70%) ⬆️
PyPDF2/generic.py 65.95% <16.66%> (+0.61%) ⬆️
PyPDF2/filters.py 38.82% <60.00%> (+16.39%) ⬆️
PyPDF2/utils.py 91.71% <100.00%> (+4.41%) ⬆️
PyPDF2/xmp.py 43.09% <0.00%> (+3.31%) ⬆️
PyPDF2/pagerange.py 98.11% <0.00%> (+3.77%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d53ee8...ce82f40. Read the comment docs.

@MartinThoma MartinThoma merged commit 012709f into main Apr 15, 2022
@MartinThoma MartinThoma deleted the bump-coverage branch April 15, 2022 10:29
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.

None yet

2 participants