Problem/Motivation

We're currently getting a test failure, see https://www.drupal.org/pift-ci-job/83690:

Features.Drupal\Tests\features\Kernel\FeaturesGenerateTest
testExportWrite

fail: [Other] Line 71 of modules/features/tests/src/Kernel/FeaturesGenerateTest.php:
Failed asserting that false is true.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 2618390-4.patch4.65 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo created an issue. See original summary.

nedjo’s picture

The test passes for me locally.

mpotter’s picture

Hmm, I wonder if it doesn't have permission to write to the /modules directory on the CI environment?

I looked at trying to export a Feature to /tmp programatically, but we won't have any way currently to override that. Even setting the $package['directory'] in the test doesn't work because the directory is regenerated by PrepareFiles.

So, we either need a global path override, or a package specific override for the path if that is the issue with the test.

I'm surprised it doesn't give us the custom messages for the failing tests though. I added those to determine exactly which assert is failing, but all the CI says is "Failed asserting that false is true." which is very annoying. So hard to determine what's actually failing.

dawehner’s picture

I'm curious whether this fixes the issue ...

dawehner’s picture

Status: Active » Needs review

Do we have automatic testing enabled?

dawehner’s picture

Oh right we have a failing branch test ...

Status: Needs review » Needs work

The last submitted patch, 4: 2618390-4.patch, failed testing.

nedjo’s picture

Status: Needs work » Closed (duplicate)