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

Fix failing tests #205

Merged
merged 6 commits into from
Dec 5, 2022
Merged

Fix failing tests #205

merged 6 commits into from
Dec 5, 2022

Conversation

patinthehat
Copy link
Collaborator

@patinthehat patinthehat commented Dec 5, 2022

This PR adds the pestphp/pest-plugin composer plugin to the list of allowed plugins in the composer.json file.

It also updates the test workflow to use the latest ghostscript version, which was causing tests to fail (#198). Also, this PR loads the PDF file after creating the Imagick instance.

Additionally, it implements some of the code from #199, authored by @bobvandevijver, to fix the failing tests:

  • Remove usages of Imagick pingImage method as it is unreliable
  • Lazy load numberOfPages until it is used

This PR should allow PRs #198 and #199 to be closed.

id:allow-pest-plugin/v1

@patinthehat patinthehat changed the title Allow Pest Composer Plugin Allow Pest Composer Plugin (fixes failing tests) Dec 5, 2022
@patinthehat patinthehat changed the title Allow Pest Composer Plugin (fixes failing tests) Allow Pest Composer Plugin Dec 5, 2022
@patinthehat patinthehat changed the title Allow Pest Composer Plugin Fix failing tests Dec 5, 2022
@freekmurze freekmurze merged commit 8e60264 into spatie:main Dec 5, 2022
@freekmurze
Copy link
Member

Thanks!

@bobvandevijver
Copy link
Contributor

Thanks, great to see that my changes have finally been "merged" to main, apparently my name was the blocker?

@patinthehat
Copy link
Collaborator Author

patinthehat commented Dec 5, 2022

@bobvandevijver Not at all, I simply merged several PRs into a single PR to streamline the process. I've updated the description of this PR to make it clear that you authored the code I merged. Thank you for the contribution! 👍

@bobvandevijver bobvandevijver mentioned this pull request Dec 13, 2023
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.

3 participants