CommentFileSizeAuthor
#8 panopoly-behat-content-page-2153649-with-png.patch275.37 KBbwood
#6 panopoly-behat-content-page-2153649.patch3.06 KBbwood
#3 issue-2153649.patch2.69 KBlsolesen
contentpage-behat.patch2.69 KBlsolesen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

This test passes locally, but fails on Travis CI because the Tabs region is not defined. However, it seems that it is defined. Not sure what is going on?

https://travis-ci.org/lsolesen/panopoly/builds/15240231

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented

Seems to be a duplicate of #2152767: Add Behat test for content page

lsolesen’s picture

lsolesen CreditAttribution: lsolesen commented
Component: Code » Admin
FileSize
issue-2153649.patch2.69 KB

Rerolled against latest dev.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
CreditAttribution: dsnopek commented
Component: Admin » Tests / Continuous Integration
bwood’s picture

bwood CreditAttribution: bwood commented
Assigned: Unassigned » bwood

DrupalCon Austin Sprint. I'm on this.

bwood’s picture

bwood CreditAttribution: bwood commented
Related issues: +#2281709: Featured image ajax validation error persists after subsequent success
FileSize
panopoly-behat-content-page-2153649.patch3.06 KB

Refactored the tests a little. We revisit the content page to get rid of the validation error situation described in the related ticket 2281709.

My travis build passed: https://travis-ci.org/bwood/panopoly/builds/26981673

dsnopek’s picture

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

@bwood: Thanks! However, your screenshot.png isn't included in this patch. I think you need to use this command to get the binary data included:

git diff --binary --full-index
bwood’s picture

bwood CreditAttribution: bwood commented
FileSize
panopoly-behat-content-page-2153649-with-png.patch275.37 KB

Sorry about that. Here is a new patch which includes the .png

dsnopek’s picture

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

Thanks, @lsolesen and @bwood! I've committed it. :-)

  • Commit f4d577a on 7.x-1.x authored by bwood, committed by dsnopek: 
    Issue #2153649 by bwood, lsolesen: Add Behat test for content page.

Status: Fixed » Closed (fixed)

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