On the latest build on Travis-CI:

https://travis-ci.org/panopoly/panopoly/jobs/85037939

... check-overridden.sh failed completely, causing the test to fail.

@mglaman recently made changes to MojoCart for the same problem:

http://cgit.drupalcode.org/mojocart/commit/?id=e8e4a144f8a5f68977c4f6031...

Looking at that commit, I think we just need to set DRUSH_ARGS="--root=$DRUPAL_TI_DRUPAL_DIR --uri=$DRUPAL_TI_WEBSERVER_URL:$DRUPAL_TI_WEBSERVER_PORT" when calling check-overridden.sh.

CommentFileSizeAuthor
#2 panopoly-check-overridden-2589991-2.patch688 bytesdsnopek
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dsnopek created an issue. See original summary.

dsnopek’s picture

Status: Active » Needs review
FileSize
688 bytes

Here's a first attempt at a patch!

And here's the test run on Travis-CI:

https://travis-ci.org/dsnopek/panopoly/builds/85103712

  • dsnopek committed 1b780d3 on 7.x-1.x
    Issue #2589991 by dsnopek: check-overridden.sh failing on Travis-CI
    
dsnopek’s picture

Status: Needs review » Fixed

Build on Travis got passed the check-overridden.sh part! So, committed. Let's see what Travis says on the main repo:

https://travis-ci.org/panopoly/panopoly/jobs/85106431

Status: Fixed » Closed (fixed)

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