search

acquia / blt

Acquia's toolset for automating Drupal 8 and 9 development, testing, and deployment.
https://docs.acquia.com/blt/
GNU General Public License v2.0
442 stars 394 forks source link

blt setup:update:features-override-check is dependent on console window size #1443

Closed AMDAndy closed 7 years ago

AMDAndy commented 7 years ago

My system information:

Output of blt doctor:

Omitted. I can provide if needed.

When I run this command:

blt setup:update:features-override-check

I get the following output:

BUILD FINISHED; 11.8506 seconds

And I expected this to happen:

BUILD FAILED/home/andy/web/amd/./vendor/acquia/blt/phing/tasks/setup.xml:354:8: /home/andy/web/amd/./vendor/acquia/blt/phing/tasks/setup.xml:357:14: A feature in the ${bundle} bundle is overridden. You must re-export this feature to incorporate the changes.
; 11.7752 seconds

The features-override-check gives me a false negative but fails on Travis. If I widen my console window the test passes. Drush is trying to fit the word "Changed" into the last column and breaking it across multiple lines. The above test tries to use grep to find the word "changed" but is failing due to this.

grasmash commented 7 years ago

Great catch. Will roll a fix for this issue this week.