Added extra debugging information.

[janw-me]

Also shouldn't the exit return 1?
As it is an error.

[schlessera]

@janw-me The idea behind the check for the features folder was that we can have unified composer test for all repositories, and it will then conditionally run Behat only if Behat tests are present. This enables unified GHA workflows that are automatically synced across all repositories.

Not sure anymore whether this is the best approach, but it is what we currently have, and changing it will require lots of coordination work again.

[janw-me]

@schlessera If this changes breaks anything drop it like no mic has ever been dropped before.

But in al seriousness I'm not sure I understand what you're suggesting.
"whether this is the best approach" Are you refering to the way behat is only run if a features folder is present?
Because if this is because of GHA that's fine as long as a message is thrown that new(/rare)-contributers help along the way.

[schlessera]

But in al seriousness I'm not sure I understand what you're suggesting.
"whether this is the best approach" Are you refering to the way behat is only run if a features folder is present?
Because if this is because of GHA that's fine as long as a message is thrown that new(/rare)-contributers help along the way.

Yes, I'm not sure the check is in the right place. The composer test command should probably have script that checks which tests to run, and then only run them conditionally, instead of each test doing such a check.

As you experienced yourself, this causes explicit calls to run a certain test suite to just silently fail.