Is it just Metatag or are all tests failing? See https://www.drupal.org/pift-ci-job/307429 for an example. Here's the error that's showing up:

00:00:36.644 cd /var/www/html && sudo -u www-data php /var/www/html/scripts/run-tests.sh --list --php /opt/phpenv/shims/php > /var/www/html/artifacts/testgroups.txt
00:00:36.646 Command created as exec id dede714c
00:00:37.371 PHP Parse error:  syntax error, unexpected '[', expecting ')' in /var/www/html/sites/all/modules/openedu/modules/contrib/scheduler/scheduler.test on line 604

I'm not sure why Metatag's tests are being affected by openedu, so is there a general problem with the test infrastructure right now?

CommentFileSizeAuthor
#3 Selection_405.png40.37 KBdave reid

Comments

DamienMcKenna created an issue. See original summary.

dave reid’s picture

This is probably caused by #2735443: Remove forks of contrib from project. Ugh.

dave reid’s picture

dave reid’s picture

Priority: Major » Critical

I've confirmed this has also affected any module that was included in http://cgit.drupalcode.org/openedu/tree/modules/contrib, that has a recent commit. For example, this has also affected the Media module.

dave reid’s picture

We should probably not allow any contrib modules included inside install profiles to override dependency resolution to Drupal.org versions.

dave reid’s picture

Title: All tests failing? » If install profile includes fork of contrib module, the profile is added as a test dependency, causing failures because install profile version is used instead of actual contrib module during tests
dave reid’s picture

Title: If install profile includes fork of contrib module, the profile is added as a test dependency, causing failures because install profile version is used instead of actual contrib module during tests » Test failures in contrib modules when other projects include forked versions of said contrib modules

I'm guessing this might not be unique to install profiles? What if my contrib module X included a forked version of the metatag module as well?

drumm’s picture

Assigned: Unassigned » drumm
Status: Active » Needs review

I changed OpenEdu from being a "module" to being a "distribution" and removed its components project_dependency has parsed.

drumm’s picture

Status: Needs review » Fixed
dave reid’s picture

Yep, that resolved it for Media too. Thanks drumm.

damienmckenna’s picture

Woo, I step away for lunch and you've fixed it! Thank you, drumm!

Status: Fixed » Closed (fixed)

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