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

Improve code coverage with unit tests #395

Closed
Lucas-C opened this issue Apr 19, 2022 · 1 comment
Closed

Improve code coverage with unit tests #395

Lucas-C opened this issue Apr 19, 2022 · 1 comment

Comments

@Lucas-C
Copy link
Member

Lucas-C commented Apr 19, 2022

Current code coverage can be visualized here: https://app.codecov.io/gh/py-pdf/fpdf2

Goal: raise code coverage by one point

Our test strategy & tools (pytest) are described in the documentation

There are several minor cases (exceptions, unfrequent code paths...) that are not currently covered.
The codecov.io report allows to easily spot code lines not already covered by unit tests.
Example: rect_clip(), cf. issue #277

Any PR that raises code coverage is welcome 😊

Introducing hypothesis to cover the most-used FPDF methods would also be a very nice addition.


By contributing those improvements, you, as a benevolent FLOSS developper, will help the fpdf2 users community by making this library more robust & reliable.
Moreover, by working on this task, you will learn about PDFs syntax and the lifecycle & structure of a popular Python library.
You will also be added into the contributors list & map.

@Lucas-C Lucas-C pinned this issue Apr 19, 2022
@Lucas-C Lucas-C changed the title Significantly improve code coverage with unit tests Improve code coverage with unit tests Apr 22, 2022
@Lucas-C Lucas-C unpinned this issue May 6, 2022
@cgkoutzigiannis cgkoutzigiannis mentioned this issue May 11, 2022
6 tasks
@Lucas-C
Copy link
Member Author

Lucas-C commented May 24, 2024

Given that code coverage has improved a lot since I created this issue 2 years ago, I'm going to close this now.

@Lucas-C Lucas-C closed this as completed May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant