-
Notifications
You must be signed in to change notification settings - Fork 3
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 Error Catching and Screenshot Functionality #191
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works for me. Full screen captured, whatever is there, not just the browser window. Leaving this for Ben's final approval.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, using pillow was a good idea
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, good addition to the tool
Fixes code written to catch exceptions and failures in tests to screenshot the entire screen and what is captured by the driver. Also ensures that the INTERNAL ERROR does not show up anymore (before an exception was being raised due to failures, now it will simply log and error and move on.)
Note that on Linux CI: wayland machines prevent screenshotting (its a feature...) python-pillow/Pillow#6312 so screenshotting the entire screen is not possible on Linux CI without other libraries. Perhaps this is a come back to feature.
Additional work was done in GHA to ensure that headed tests run if the setup runs correctly and does not depend on the previous unheaded tests to pass anymore.