Skip to content

Used PHP Behat configuration as behat.php when the edition ships one - #145

Merged
ViniTou merged 6 commits into
mainfrom
behat-php-config
Sep 7, 2026
Merged

Used PHP Behat configuration as behat.php when the edition ships one#145
ViniTou merged 6 commits into
mainfrom
behat-php-config

Conversation

@ViniTou

@ViniTou ViniTou commented Aug 28, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12374

Description:

Behat 4 drops YAML configuration; ibexa/behat 6.0 ships behat_ibexa_<edition>.php next to the YAML files (ibexa/behat#195) and the 6.0 recipe copies both to the project root (ibexa/recipes-dev#258). The 6.0.x-dev and stable prepare scripts now copy the .php file to behat.php when it exists and fall back to behat.yaml otherwise, so older versions are unaffected.

Note on discovery: the friends-of-behat/symfony-extension Flex recipe leaves a behat.yml.dist in the project root, which outranks behat.php in Behat's auto-discovery. Instead of deleting recipe files here, bin/ibexabehat now defaults to --config=behat.php when the file exists (ibexa/behat#195), so the configuration is always selected explicitly.

Also fixes a pre-existing bash syntax error in both scripts (elif [[ ... ]] then without ;, rejected by bash -n).

For QA:

CI only. Verified via ibexa/behat#195 with ci-scripts-branch: behat-php-config.

Documentation:

N/A

@sonarqubecloud

sonarqubecloud Bot commented Sep 7, 2026

Copy link
Copy Markdown

@alongosz alongosz changed the title Use PHP Behat configuration as behat.php when the edition ships one Used PHP Behat configuration as behat.php when the edition ships one Sep 7, 2026
@ViniTou
ViniTou merged commit f00b423 into main Sep 7, 2026
6 checks passed
@ViniTou
ViniTou deleted the behat-php-config branch September 7, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants