We take a screenshot when tests fail, but it'd be nice to be able to manually take a screenshot to aid in debugging. Let's refactor out that code into a step that can be run manually too!

(We did this to help debug #2869464: All Chrome tests failing with "Could not open connection: An unknown server-side error" - moving patch here!)

CommentFileSizeAuthor
#2 panopoly_test-manual-screenshot-2874638-2.patch2.73 KBdsnopek

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Active » Needs review
StatusFileSize
new panopoly_test-manual-screenshot-2874638-2.patch2.73 KB

Here's the patch!

dsnopek credited cboyden.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

  • dsnopek committed 6ea772b on 7.x-1.x 
    Update Panopoly Test for Issue #2874638 by dsnopek, cboyden: Add a Behat...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Committed!

cboyden’s picture

cboyden commented

It would be nice to not depend on Imgur in situations where it's not needed. When testing locally, screenshots could be saved to a local directory.

Would it be possible to capture the screenshot, then if Imgur is not available, save it locally to a path that can be set in behat.yml? Our child distro has a very similar step that saves screenshots locally, then uses the Travis artifacts function to upload screenshots to AWS. It would be nice to combine forces here. @dsnopek if you think this is a good idea, I can work on an updated patch.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

Sure! That should probably be a new issue, though

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.