Skip to content
This repository was archived by the owner on Apr 11, 2025. It is now read-only.

TODO Mock pdfium unavailable for lattice test #264

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bosd
Copy link
Collaborator

@bosd bosd commented Nov 2, 2024

To improve coverage of the pdfium backend, would like to add a mock the availability.

What is the proper way to do it?

@bosd bosd added the help wanted Extra attention is needed label Nov 2, 2024
@stefan6419846
Copy link

I personally would avoid modifying sys.modules etc. on the main thread and instead move this to a corresponding subprocess (example: https://github.com/py-pdf/pypdf/blob/98aa9742e757ec428e1953ba7f47c6d7c44b331a/tests/test_filters.py#L248-L291). Nevertheless, this might require further configuration for coverage.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants